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
etcher: 1.5.60 -> 1.5.86 #86228
etcher: 1.5.60 -> 1.5.86 #86228
Conversation
@GrahamcOfBorg build etcher |
# We'll use our Nixpkgs electron_3 instead | ||
rm $out/opt/balenaEtcher/balena-etcher-electron | ||
substituteInPlace $out/share/applications/balena-etcher-electron.desktop \ | ||
--replace 'Exec=/opt/balenaEtcher/balena-etcher-electron' 'Exec=${pname}' |
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.
Isn't an absolute path, i.e. $out/bin/etcher
safer here?
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.
Maybe, but the vast majority of desktop files on my system have only relative names in them:
find -name "*.desktop" -type f -exec grep Exec {} \;
Not sure what is the policy in nixpkgs.
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
Motivation for this change
Version update.
This also bumps dependency from very old Electron 3 to relatively new Electron 7.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)