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
trezord: 2.0.12 -> 2.0.19 and nixos/trezord: revised and updated udev rules #46658
Conversation
maintainers."1000101"
Missed semicolon
You're right, thank you so much. |
@1000101 Can you give a short description of the purpose of the changes to the udev rules? I'm no udev expert, so it's not obvious to me what's going on. I'll be happy to merge after this, if it's tested and working. :) |
Model T was not working with 660 rights.
Hi Sarah! Actually, I'm from the company that is releasing the original source code and since more of us are using NixOS, we'd like to contribute and maintain the package up-to-date.
Thanks a lot for asking. I was quite confident that everything is right, but since you've asked I've tested it again and found out that one model of our devices is not quite happy since it wants to use plugdev group (which is not defined by default) instead of dialout. I've committed a fix and tested rebuild on 18.03 by using the local pkgs. Both our devices are now fully functional. Thank you so much, Sarah, I appreciate it! |
Motivation for this change
I'd like to maintain the package up-to-date as we're the company releasing the original code.
Things done
Updated version from 2.0.14 to 2.0.19. Reviewed and updated the service policies and rights.
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)