Bug 2491207 - Review Request: python-pynitrokey - Python client for Nitrokey devices
Summary: Review Request: python-pynitrokey - Python client for Nitrokey devices
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Zbigniew Jędrzejewski-Szmek
QA Contact: Fedora Extras Quality Assurance
URL: https://github.com/Nitrokey/pynitrokey
Whiteboard:
Depends On: 2491197 2491198 2491200 2491201
Blocks:
TreeView+ depends on / blocked
 
Reported: 2026-06-21 17:27 UTC by Daniel Milnes
Modified: 2026-07-20 01:56 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2026-07-11 15:30:37 UTC
Type: ---
Embargoed:
zbyszek: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 10622211 to 10622652 (913 bytes, patch)
2026-06-21 21:46 UTC, Fedora Review Service
no flags Details | Diff

Description Daniel Milnes 2026-06-21 17:27:18 UTC
Spec URL: https://download.copr.fedorainfracloud.org/results/thebeanogamer/python3-pynitrokey/srpm-builds/10622205/python-pynitrokey.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/thebeanogamer/python3-pynitrokey/srpm-builds/10622205/python-pynitrokey-0.12.3-1.src.rpm
Description: A command line interface for the Nitrokey FIDO2, Nitrokey Start, Nitrokey 3 and NetHSM.
Fedora Account System Username: thebeanogamer

This package depends on several other packages, which are linked to this ticket as blockers.

Comment 1 Fedora Review Service 2026-06-21 17:29:58 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/10622211
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2491207-python-pynitrokey/fedora-rawhide-x86_64/10622211-python-pynitrokey/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 2 Zbigniew Jędrzejewski-Szmek 2026-06-21 21:06:50 UTC
> BuildRequires: python3dist(pytest)
> BuildRequires: python3dist(oath)
The right column could be indented by one line more to match the rest of the file.

Hmm, should the main package be called 'python3-pynitrokey' or just 'nitropy'?
I guess the existing name is better. But please add 'Provides: nitropy' so that
people can easily install using the executable name.

It'd also be nice to extend the %description for the binary package with
some hints what nitropy is good for. Something that'd tell people what
this package is useful for, and also make it easier if they're trying to
deal with TOTP and such.

> Requires:       /usr/lib/udev/rules.d/41-nitrokey.rules
This is not allowed (and it also doesn't work with dnf5):
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_file_and_directory_dependencies:
> Packages MAY include such dependencies for paths inside one of the following directories:
> /usr/bin
> /etc

What was the error that rpmlint gave?

Looks reasonable in general.

Comment 3 Daniel Milnes 2026-06-21 21:44:41 UTC
> The right column could be indented by one line more to match the rest of the file.
Done

> But please add 'Provides: nitropy' so that people can easily install using the executable name.
Done

> It'd also be nice to extend the %description for the binary package with
> some hints what nitropy is good for. Something that'd tell people what
> this package is useful for, and also make it easier if they're trying to
> deal with TOTP and such.
Expanded it a bit

> This is not allowed (and it also doesn't work with dnf5):
Oh interesting. Explains some errors I was having on a non-Fedora project...

> What was the error that rpmlint gave?
python3-pynitrokey.noarch: E: explicit-lib-dependency libnitrokey.
libnitrokey doesn't have a `pkgconfig(` provides or similar that could be used instead.

Spec URL: https://download.copr.fedorainfracloud.org/results/thebeanogamer/python3-pynitrokey/srpm-builds/10622651/python-pynitrokey.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/thebeanogamer/python3-pynitrokey/srpm-builds/10622651/python-pynitrokey-0.12.3-1.src.rpm

Comment 4 Fedora Review Service 2026-06-21 21:46:37 UTC
Created attachment 2146179 [details]
The .spec file difference from Copr build 10622211 to 10622652

Comment 5 Fedora Review Service 2026-06-21 21:46:40 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/10622652
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2491207-python-pynitrokey/fedora-rawhide-x86_64/10622652-python-pynitrokey/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 6 Zbigniew Jędrzejewski-Szmek 2026-06-21 21:57:22 UTC
> python3-pynitrokey.noarch: E: explicit-lib-dependency libnitrokey.
The rpmlint error is meant to prevent manual dependencies being added for
shared libraries, but here the dependency is on the package for other files
that are in it. So please just use 'Requires: libnitrokey'.
(rpmlint is often wrong. Don't take it what is says as gospel.)

+ package name is OK
+ latest version
+ license is specified correctly (MIT + Apache 2.0)
+ license is acceptable for Fedora
+ builds and installs OK
+ R/BR/P appear correct (modulo the issue discussed above).

Standard pyproject packaging is used, nothing out of the ordinary.

The udev rules provided by libnitrokey look reasonable too.

Package is APPROVED. Please fix the libnitrokey dep when importing.

Comment 7 Fedora Admin user for bugzilla script actions 2026-06-21 22:17:43 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/python-pynitrokey
Monitoring:
Wasn't able to create project in Anitya. You can create it manually on: `https://release-monitoring.org`

Comment 8 Fedora Update System 2026-07-11 15:26:35 UTC
FEDORA-2026-cfc6f24990 (python-pynitrokey-0.12.3-2.fc45) has been submitted as an update to Fedora 45.
https://bodhi.fedoraproject.org/updates/FEDORA-2026-cfc6f24990

Comment 9 Fedora Update System 2026-07-11 15:30:37 UTC
FEDORA-2026-cfc6f24990 (python-pynitrokey-0.12.3-2.fc45) has been pushed to the Fedora 45 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 10 Fedora Update System 2026-07-19 15:40:37 UTC
FEDORA-2026-389c1e6a79 (python-pynitrokey-0.12.3-2.fc44) has been submitted as an update to Fedora 44.
https://bodhi.fedoraproject.org/updates/FEDORA-2026-389c1e6a79

Comment 11 Fedora Update System 2026-07-19 19:58:01 UTC
FEDORA-2026-d42f000478 (python-pynitrokey-0.12.3-2.fc43) has been submitted as an update to Fedora 43.
https://bodhi.fedoraproject.org/updates/FEDORA-2026-d42f000478

Comment 12 Fedora Update System 2026-07-20 01:38:40 UTC
FEDORA-2026-389c1e6a79 has been pushed to the Fedora 44 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2026-389c1e6a79 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2026-389c1e6a79

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 13 Fedora Update System 2026-07-20 01:56:05 UTC
FEDORA-2026-d42f000478 has been pushed to the Fedora 43 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2026-d42f000478 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2026-d42f000478

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.


Note You need to log in before you can comment on or make changes to this bug.