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
python3Packages.trezor_agent: fix startup error #71290
Conversation
, wheel | ||
, pinentry | ||
}: | ||
|
||
buildPythonPackage rec{ | ||
buildPythonApplication rec{ |
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.
I would keep this as buildPythonPackages while it's in the python-modules directory. If you wanted to do a top-level export, then you can just do toPythonApplication python3Packages.trezor_agent
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.
Alright, sure. I'll revert that part.
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.
9ebbe92
to
be6a20a
Compare
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.
nix-review
passes on NixOS (python38.cytoolz is broken on master)
diff LGTM
executable is fixed
[2 built (4 failed), 144 copied (342.9 MiB), 69.2 MiB DL]
error: build of '/nix/store/466m47fbkqcl76l9w1i2nlqfgggpw73x-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/71290
1 package failed to build:
python38Packages.trezor_agent
1 package were build:
python37Packages.trezor_agent
[nix-shell:~/.cache/nix-review/pr-71290]$ ./results/python37Packages.trezor_agent/bin/trezor-agent --help
usage: trezor-agent [-h] [-v] [--version] [-e CURVE] [--timeout TIMEOUT]
[--debug] [--log-file LOG_FILE] [--sock-path SOCK_PATH]
[--pin-entry-binary PIN_ENTRY_BINARY]
[--passphrase-entry-binary PASSPHRASE_ENTRY_BINARY]
[--cache-expiry-seconds CACHE_EXPIRY_SECONDS]
[-d | -f | -s | -c | --mosh]
identity [ARGUMENT [ARGUMENT ...]]
...
Motivation for this change
python3Packages.trezor_agent
fails on startup on 19.09 withThings done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @np @mmahut