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
nix-index: Wrap an unwrapped derivation #106959
Conversation
The main concern with wrapping in this way is that it makes it much harder to override the Rust "unwrapped" derivation. Would it be more helpful to use "--suffix" instead of "--prefix" so the user can use their own Nix version? |
Essentially allow overriding the `nix` package used in the wrapping, without rebuilding the rust package.
98ee3c4
to
4a87b31
Compare
I think it depends on what the maintainer meant when they wrapped it. If it was meant to ensure compatibility, it would defeat the purpose to use
I modified the changes so now it is possible to override the unwrapped derivation as well, with examples in the comments of |
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
Does the nix version used by nix-index even matter? |
It probably does not, as nix-index only relies on |
I marked this as stale due to inactivity. → More info |
Essentially allow overriding the
nix
package used in the wrapping,without rebuilding the rust package.
Motivation for this change
Fix #106515 - cc @poscat0x04 .
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)