Bug 264281 - security/py-fido2: Unbreak and extend FreeBSD support
Summary: security/py-fido2: Unbreak and extend FreeBSD support
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: freebsd-python (Nobody)
URL: https://github.com/Yubico/python-fido...
Keywords:
Depends on:
Blocks:
 
Reported: 2022-05-27 10:11 UTC by Michael Gmelin
Modified: 2022-06-07 11:58 UTC (History)
1 user (show)

See Also:
koobs: maintainer-feedback+
koobs: merge-quarterly?


Attachments
Patch to fix and extend FreeBSD support in py-fido2 (8.98 KB, patch)
2022-05-27 10:11 UTC, Michael Gmelin
koobs: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Gmelin freebsd_committer freebsd_triage 2022-05-27 10:11:04 UTC
Created attachment 234260 [details]
Patch to fix and extend FreeBSD support in py-fido2

Upstream broke FreeBSD support a while ago by accident, when removing the dependency on py-uhid-freebsd.

This patch, which was already accepted by highly supportive and very responsive upstream, brings back FreeBSD support with classic uhid(4) and adds support for the new hidraw(4), usbhid(3) based driver.

Patch suitable for use with `git am`.
Comment 1 Kubilay Kocak freebsd_committer freebsd_triage 2022-05-28 00:56:13 UTC
Thanks for this Michael. Can you confirm this passes poudriere and `make test` on at least the default python version and one non default version

Happy to accept on that QA. Great stuff sorting this out upstream
Comment 2 Michael Gmelin freebsd_committer freebsd_triage 2022-05-28 07:36:59 UTC
(In reply to Kubilay Kocak from comment #1)

I can confirm that all of these pass on amd64:

Python 3.8 (default):
[x] poudriere testport
[x] poudriere bulk
[x] make test
[x] make test with an actual device plugged in (runs one more test)

Python 3.9:
[x] poudriere testport
[x] poudriere bulk
[x] make test
Comment 3 Kubilay Kocak freebsd_committer freebsd_triage 2022-06-02 02:08:05 UTC
(In reply to Michael Gmelin from comment #2)

Thank you, all yours champ!
Comment 4 Kubilay Kocak freebsd_committer freebsd_triage 2022-06-02 02:08:41 UTC
Comment on attachment 234260 [details]
Patch to fix and extend FreeBSD support in py-fido2

Approved by: koobs (python, maintainer)
MFH: 2022Q2
Comment 5 commit-hook freebsd_committer freebsd_triage 2022-06-07 11:54:45 UTC
A commit in branch main references this bug:

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

commit ce57b8b96961901188c60319459cfb5fcea13f03
Author:     Michael Gmelin <grembo@FreeBSD.org>
AuthorDate: 2022-05-27 09:27:56 +0000
Commit:     Michael Gmelin <grembo@FreeBSD.org>
CommitDate: 2022-06-07 11:51:57 +0000

    security/py-fido2: Fix and extend FreeBSD support

    This unbreaks FreeBSD support in general and adds support
    for FreeBSD 13's optional hidraw(4) driver.

    See https://github.com/Yubico/python-fido2/pull/139

    PR:             264281
    Approved by:    koobs (python, maintainer)

 security/py-fido2/Makefile                         |   1 +
 .../files/patch-fido2_hid_freebsd.py (new)         | 222 +++++++++++++++++++++
 2 files changed, 223 insertions(+)
Comment 6 commit-hook freebsd_committer freebsd_triage 2022-06-07 11:57:46 UTC
A commit in branch 2022Q2 references this bug:

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

commit afdc136ee424cd6159b4e77d3b56d5a9f32669c9
Author:     Michael Gmelin <grembo@FreeBSD.org>
AuthorDate: 2022-05-27 09:27:56 +0000
Commit:     Michael Gmelin <grembo@FreeBSD.org>
CommitDate: 2022-06-07 11:56:27 +0000

    security/py-fido2: Fix and extend FreeBSD support

    This unbreaks FreeBSD support in general and adds support
    for FreeBSD 13's optional hidraw(4) driver.

    See https://github.com/Yubico/python-fido2/pull/139

    PR:             264281
    Approved by:    koobs (python, maintainer)

    (cherry picked from commit ce57b8b96961901188c60319459cfb5fcea13f03)

 security/py-fido2/Makefile                         |   1 +
 .../files/patch-fido2_hid_freebsd.py (new)         | 222 +++++++++++++++++++++
 2 files changed, 223 insertions(+)