Bug 265458 - comms/uarduno: Remove devclass from DRIVER_MODULE on recent main
Summary: comms/uarduno: Remove devclass from DRIVER_MODULE on recent main
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks: 264969
  Show dependency treegraph
 
Reported: 2022-07-26 16:39 UTC by John Baldwin
Modified: 2022-08-03 22:54 UTC (History)
1 user (show)

See Also:
bobf: maintainer-feedback+
bobf: maintainer-feedback+


Attachments
the actual patch referenced by submitter (1.10 KB, patch)
2022-07-26 23:08 UTC, Bob Frazier
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description John Baldwin freebsd_committer freebsd_triage 2022-07-26 16:39:53 UTC
Patch is at https://reviews.freebsd.org/D35947 (could not add maintainer as a reviewer in phab so opening a bug as well)
Comment 1 Bob Frazier 2022-07-26 22:56:57 UTC
The patch looks fine to me, with the assumption that these changes are needed for it to build with FBSD 14.  Should I apply this upstream?

(I'm attaching it to the bug as wel)
Comment 2 Bob Frazier 2022-07-26 23:08:52 UTC
Created attachment 235500 [details]
the actual patch referenced by submitter

patch file submitted to phab and referenced by bug submitter (tested)
Comment 3 Bob Frazier 2022-07-26 23:18:52 UTC
just to clarify I did a quick test-build with FBSD 12.2 to make sure that the patch did not affect anything below FBSD 14 (it does not appear to).  I am pretty sure it would not after looking it over, so now it's just a matter of testing in FBSD 14 which I'll need to do at some point (just not today).

If you need anything specific please let me know (thanks).  This port is actually  needed for earlier FBSD versions that had no ACM (modem class) serial driver, and uses an editable list of USB vendor/product IDs (corresponding to Arduinos that use an AVR USB serial driver, like the Uno and Mega).  Since the USB class drivers were added (in 11 I think) it is still useful but its need is less important.  I had considered marking it for FBSD versions prior to 11 but it builds/runs ok so I'll continue to maintain it.
Comment 4 John Baldwin freebsd_committer freebsd_triage 2022-08-03 22:37:12 UTC
Yes, please apply this upstream.  I plan to remove the compat shims from 14.0 but am fixing ports first.
Comment 5 commit-hook freebsd_committer freebsd_triage 2022-08-03 22:49:55 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=de3758e2e34c09c0b12343f1346aabf1834c8224

commit de3758e2e34c09c0b12343f1346aabf1834c8224
Author:     John Baldwin <jhb@FreeBSD.org>
AuthorDate: 2022-08-03 22:45:23 +0000
Commit:     John Baldwin <jhb@FreeBSD.org>
CommitDate: 2022-08-03 22:45:23 +0000

    comms/uarduno: Remove devclass from DRIVER_MODULE on recent main.

    PR:             265458
    Reviewed by:    Bob Frazier <bobf@mrp3.com> (maintainer)
    Differential Revision:  https://reviews.freebsd.org/D35947

 comms/uarduno/files/patch-uarduno.c (new) | 37 +++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)