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
electrum: update for the Qt5 change (#65399) #67727
Conversation
c5fa22f
to
8053c68
Compare
8053c68
to
e568ff5
Compare
@@ -102,7 +104,8 @@ python3Packages.buildPythonApplication rec { | |||
"Exec=$out/bin/electrum %u" \ | |||
--replace 'Exec=sh -c "PATH=\"\\$HOME/.local/bin:\\$PATH\"; electrum --testnet %u"' \ | |||
"Exec=$out/bin/electrum --testnet %u" | |||
''; | |||
|
|||
'' + (if enableQt then "wrapQtApp $out/bin/electrum" else ""); |
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 think this should really happen in the postFixup
phase
postFixup = lib.optionalString enableQt ''
wrapQtApp $out/bin/electrum
'';
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.
e568ff5
to
da39cb1
Compare
da39cb1
to
3d5144a
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.
LGTM, thanks for fixing this.
This fixes
electrum
, in line with the Qt update (#65399).Things 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 @ehmry @joachifm