-
-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
gradle: fix update-deps.sh to use Nix bash #329272
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.
LGTM other than a nit
3067b1e
to
c6222b6
Compare
Nit is addressed :) Thanks for the review |
Result of 1 package failed to build:
2 packages built:
|
c6222b6
to
2f2724e
Compare
pkgs/tools/security/ghidra/extensions/gnudisassembler/default.nix
Outdated
Show resolved
Hide resolved
2f2724e
to
c526be6
Compare
Result of 1 package failed to build:
2 packages built:
|
pkgs/tools/security/ghidra/extensions/gnudisassembler/default.nix
Outdated
Show resolved
Hide resolved
c526be6
to
ab8a524
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
ab8a524
to
6e7ba87
Compare
6e7ba87
to
d61380e
Compare
d61380e
to
aac69e0
Compare
This fixes an issue on non NixOS Linux distributions where their system bash might be outside of /tmp, /nix, and /run. Thanks to @puckipedia for the suggestion
This is required as Ghidra plugins don't have a consistent layout and have to be widely configurable
aac69e0
to
5c354f1
Compare
Result of 3 packages built:
|
Description of changes
Fixes two issues I had with the gradle refactor and my external Ghidra plugins.
Things done
I only checked that this works for my specific use case. I didn't update all Gradle packages to see if this change somehow breaks something.
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.