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
perlPackages.Apprainbarf: init at 1.4 #95872
Conversation
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.
To make rainbarf
run on darwin:
nativeBuildInputs = stdenv.lib.optional stdenv.isDarwin shortenPerlShebang;
postInstall = stdenv.lib.optionalString stdenv.isDarwin ''
shortenPerlShebang $out/bin/rainbarf
'';
2558d94
to
e8bf306
Compare
Thank you for the kind review! 🙏 I added the suggested changes, please take a look again. :) |
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.
Pls change commit message to perlPackages.Apprainbarf: init at 1.4
Signed-off-by: Sascha Grunert <sgrunert@suse.com>
e8bf306
to
1683f3a
Compare
Done, thank you for the hint 👍 |
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 :) Thx for the PR
(To be pedantic: maybe update the PR name as well)
@GrahamcOfBorg build perlPackages.Apprainbarf |
@stigtsp I'm sure one day we'll get the bot building |
Sounds like @stigtsp tested this on |
Motivation for this change
A perl package I love, please let this get into nixpkgs.
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)