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
sameboy: init at 0.13.6 #98177
sameboy: init at 0.13.6 #98177
Conversation
Tested it using |
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.
A few comments, otherwise LGTM 👍
installPhase = '' | ||
pushd build/bin/SDL | ||
install -Dm755 sameboy $out/bin/sameboy | ||
rm sameboy |
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.
There's no need to remove this manually, the build env gets garbage collected in its entirety when it's not required anymore.
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.
That's not the role of this rm
. The goal here is to avoid *
matching the binary itself, copying only the rest. Should I add a comment to this effect? Or maybe there's a more idiomatic way of achieving this?
250bc84
to
2700090
Compare
Motivation for this change
Adds a new package.
Things done
sandbox
innix.conf
on non-NixOS linux)./result/bin/
)nix path-info -S
before and after)