Skip to content
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

cargo-chef isn't local registry aware #74

Closed
prestontw opened this issue Aug 5, 2021 · 1 comment
Closed

cargo-chef isn't local registry aware #74

prestontw opened this issue Aug 5, 2021 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@prestontw
Copy link
Contributor

Link to minimal repo: https://github.com/prestontw/cargo-chef-vendored-repo

I've vendored the dependencies of a small Rocket app using cargo vendor. I've then updated .cargo/config.toml so that cargo build uses the local dependencies. Trying to run cargo chef errors with checksum changes (since cargo chef has replaced all of the lib.rs files in the dependencies with empty files?).

I'm thinking if we make cargo chef aware of .cargo/config.toml, we will know not to skelefy these vendored files.

Aside: using cargo-local-registry is fine since it stores dependencies in zipped files, so cargo chef doesn't skelefy them.

@LukeMathWalker
Copy link
Owner

.cargo/config.toml is already part of the recipe file, so it should be possible to make this work without any changes to the CLI parameters.

@LukeMathWalker LukeMathWalker added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Aug 30, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants