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: 1.2.1 -> 2.0.12 #38704
trezord: 1.2.1 -> 2.0.12 #38704
Conversation
pkgs/servers/trezord/default.nix
Outdated
|
||
stdenv.mkDerivation rec { | ||
name = "trezord-${version}"; | ||
buildInputs = [clang]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather than using clang
to work around the issue you can use hardeningDisable = [ "fortify" ];
.
See https://github.com/NixOS/nixpkgs/blob/b81ca435e6367ddbca29846bfb6ae3b538e67a6c/pkgs/applications/altcoins/go-ethereum.nix for an example
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
f8b600a
to
4d2bfa3
Compare
The old trezord is obsolete and no longer functions with Trezor's password manager app.
4d2bfa3
to
a60e174
Compare
@roconnor-blockstream Thanks for the change. Everythink looks fine now :) I want to test this with some actual Trezor hardware before merging so it's gonna have to wait until tomorrow (~12h). I'll also backport this change to |
Awesome! Testing now. |
works great! |
@GrahamcOfBorg eval |
A bit slow to backport, but here it is: 24eb7a9 |
The old trezord is obsolete and no longer functions with Trezor's password manager app.
Motivation for this change
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)