-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Regression 1.33.0: Cargo doesn't resolve yanked crates when using local registry #6741
Labels
C-bug
Category: bug
Comments
hugwijst
added a commit
to hugwijst/cargo
that referenced
this issue
Mar 13, 2019
bors
added a commit
that referenced
this issue
Mar 13, 2019
Fix resolving yanked crates when using a local registry. Fixes #6741.
Can we backport this into beta? Getting a 1.33.1 is probably a long shot. We'll need to skip 1.33.0 internally, as having commits that cannot be successfully build anymore due to an update to the local registry is unacceptable. |
Backporting seems reasonable to me! @hugwijst would you be up for sending a PR to the rust-1.34.0 branch of this repository? |
hugwijst
added a commit
to hugwijst/cargo
that referenced
this issue
Mar 15, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem
When using a local registry, building a crate that depends on a yanked crate will fail with error message:
error: failed to select a version for the requirement
.The crate builds correctly when using the crates.io remote registry.
Steps
On 1.33.0 or nightly:
cargo init test-lib
cd test-lib
echo 'libc = "0.1.11"' >> Cargo.toml
cargo local-registry -s Cargo.lock registry
mkdir .cargo
.cargo/config
:cargo check
error: failed to select a version for the requirement `libc = "= 0.1.11"`
.Possible Solution(s)
Possibly introduced by #6655, as a local registry isn't provided the
yanked_whitelist
when constructed (RegistrySource::local
insrc/cargo/sources/registry/mod.rs
).Notes
Output of
cargo version
:cargo 1.33.0 (f099fe94b 2019-02-12)
.The text was updated successfully, but these errors were encountered: