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
fetchCargoTarball: support vendoring extra manifests #104281
fetchCargoTarball: support vendoring extra manifests #104281
Conversation
b516955
to
1c2f8f5
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.
Can you add some documentation for this? I assume that https://github.com/NixOS/nixpkgs/blob/master/doc/languages-frameworks/rust.section.md would be the right place but I am not 100% sure.
This is helpful in a situation where not all crates in the given repository are listed in the toplevel Cargo.toml but are required during the build-process.
1c2f8f5
to
162971c
Compare
My use-case for this has ceased. It might still make sense to merge this since it's a feature offered by |
I marked this as stale due to inactivity. → More info |
Please add a test case so that we can ensure your feature continues working whenever we refactor the code. |
I marked this as stale due to inactivity. → More info |
i haven't been using this myself so i won't pursue this PR any longer |
Motivation for this change
This is helpful in a situation where not all crates in the given
repository are listed in the toplevel Cargo.toml but are required during
the build-process.
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)