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
stumpish: init at 0.0.1 #44016
stumpish: init at 0.0.1 #44016
Conversation
''; | ||
|
||
meta = with stdenv.lib; { | ||
homepage = https://github.com/stumpwm/stumwm-contrib; |
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.
typo
chmod +x $out/bin/stumpish | ||
''; | ||
|
||
postFixup = '' |
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.
Using a patch file with substituteAll
would probably be cleaner.
sha256 = "1dxzsnir3158p8y2128s08r9ca0ywr9mcznivmhn1lycw8mg4nfl"; | ||
}; | ||
|
||
phases = [ "buildPhase" "installPhase" "postFixup" ]; |
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.
Manually setting phases is frowned upon #28910
@jtojnar I updated |
''; | ||
|
||
installPhase = '' | ||
cp $src/util/stumpish/stumpish $out/bin |
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.
You need to drop $src/
. It contains the unpatched file.
|
||
installPhase = '' | ||
cp $src/util/stumpish/stumpish $out/bin | ||
chmod +x $out/bin/stumpish |
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.
This should not be necessary.
@jtojnar Is this better? |
It does seem so. |
@jtojnar Thanks! |
Do you think we can merge this now? |
@jtojnar Thanks! |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)