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
treewide: unzip buildInputs to nativeBuildInputs (2) #115228
Conversation
70bdbf5
to
4f0cdb6
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.
not sure about the modules.
Based on the comments, I think the modules are fine. Those are a small rebuild if not, too. |
Reverted #115228 for kodi to avoid conflict. It does not look like unzip would be used but not investigating now to speed up merge conflict resolution.
@@ -177,7 +177,7 @@ in stdenv.mkDerivation { | |||
libogg libvorbis flac libxslt systemd | |||
lzo libcdio libmodplug libass libbluray | |||
sqlite libmysqlclient avahi lame | |||
curl bzip2 zip unzip glxinfo | |||
curl bzip2 zip glxinfo |
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.
Looks like this has not been removed from the argument list.
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.
on staging
it is there and unused. On staging-next it is also in
buildInputs`.
Motivation for this change
Helps with cross-compilation. See #112302.
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)