Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure all links to udev rules file point to recent version #389

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

cypheon
Copy link
Contributor

@cypheon cypheon commented Jan 23, 2025

The udev rules file 41-nitrokey.rules is crucial to the devices' functioning under Linux. The canonical and maintained version of these rules seems to be the one at https://github.com/Nitrokey/nitrokey-udev-rules

The documentation contained several links to older, unmaintained versions at different URLs, which are missing some recent fixes, e.g. nitrokey-udev-rules PR #6.

Make sure the documentation always links to the maintained version at https://raw.githubusercontent.com/Nitrokey/nitrokey-udev-rules/main/41-nitrokey.rules

The old URLs that have been replaced where:
https://raw.githubusercontent.com/Nitrokey/libnitrokey/master/data/41-nitrokey.rules
https://www.nitrokey.com/sites/default/files/41-nitrokey.rules

Now all URLs point to either https://raw.githubusercontent.com/Nitrokey/nitrokey-udev-rules/main/41-nitrokey.rules or https://raw.githubusercontent.com/Nitrokey/nitrokey-udev-rules/refs/heads/main/41-nitrokey.rules, which both lead to the same file/version.

The udev rules file `41-nitrokey.rules` is crucial to the devices'
functioning under Linux.  The canonical and maintained version of these rules seems to
be the one at https://github.com/Nitrokey/nitrokey-udev-rules

The documentation contained several links to older, unmaintained
versions at different URLs, which are missing some recent fixes, e.g.
[nitrokey-udev-rules PR Nitrokey#6](Nitrokey/nitrokey-udev-rules#6).

Make sure the documentation always links to the maintained version at
https://raw.githubusercontent.com/Nitrokey/nitrokey-udev-rules/main/41-nitrokey.rules
@daringer
Copy link
Contributor

nice - many thanks!

@daringer daringer merged commit 50c3362 into Nitrokey:main Jan 27, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants