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

Regression in version strings accepted by cargo #9543

Closed
weiznich opened this issue Jun 4, 2021 · 2 comments · Fixed by rust-lang/rust#85983 or #9544
Closed

Regression in version strings accepted by cargo #9543

weiznich opened this issue Jun 4, 2021 · 2 comments · Fixed by rust-lang/rust#85983 or #9544

Comments

@weiznich
Copy link
Contributor

weiznich commented Jun 4, 2021

Code

I tried this code: diesel 1.4.x

I expected to see this happen: Compilation succeeds

Instead, this happened: Compilation fails with a dependency resolution error

    Updating crates.io index
 Downloading crates ...
error: failed to download `derive-error-chain v0.10.1`
Error: failed to download `derive-error-chain v0.10.1`
Caused by:
  unable to get packages from source

Caused by:
  failed to parse manifest at `C:\Users\runneradmin\.cargo\registry\src\github.com-1ecc6299db9ec823\derive-error-chain-0.10.1\Cargo.toml`

Caused by:
  failed to parse the version requirement `0.11.x` for dependency `syn`

Steps to reproduce

git clone https://github.com/diesel-rs/diesel
cd diesel
git checkout 1.4.x
cargo +nightly test --manifest-path diesel/Cargo.toml --no-default-features --features "sqlite extras libsqlite3-sys/bundled"

This is likely related to #9508 and rust-lang/rust#85983

Version it worked on

It most recently worked on: cargo 1.53.0-beta (4369396ce 2021-04-27)

Version with regression

cargo 1.54.0-nightly (0cecbd673 2021-06-01)

@rustbot modify labels: +regression-from-stable-to-nightly -regression-untriaged

@ehuss ehuss transferred this issue from rust-lang/rust Jun 4, 2021
@rustbot

This comment has been minimized.

@ehuss
Copy link
Contributor

ehuss commented Jun 4, 2021

Transferred to the cargo repository, as that is where cargo lives. This will be fixed by rust-lang/rust#85983.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants