Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Require cargo install --vers takes a semver version #3338

Merged
merged 1 commit into from
Dec 2, 2016

Conversation

alexcrichton
Copy link
Member

Historically Cargo accidentally took a semver version requirement, so let's
start issuing warnings about how this is now legacy behavior.

Closes #3321

Historically Cargo accidentally took a semver version *requirement*, so let's
start issuing warnings about how this is now legacy behavior.

Closes rust-lang#3321
@rust-highfive
Copy link

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@brson
Copy link
Contributor

brson commented Dec 1, 2016

@bors r+

@bors
Copy link
Contributor

bors commented Dec 1, 2016

📌 Commit d91c9b4 has been approved by brson

@bors
Copy link
Contributor

bors commented Dec 2, 2016

⌛ Testing commit d91c9b4 with merge 787591c...

bors added a commit that referenced this pull request Dec 2, 2016
Require `cargo install --vers` takes a semver version

Historically Cargo accidentally took a semver version *requirement*, so let's
start issuing warnings about how this is now legacy behavior.

Closes #3321
@bors
Copy link
Contributor

bors commented Dec 2, 2016

☀️ Test successful - status-appveyor, status-travis
Approved by: brson
Pushing 787591c to master...

@bors bors merged commit d91c9b4 into rust-lang:master Dec 2, 2016
@alexcrichton alexcrichton deleted the correct-vers-flag branch December 2, 2016 17:42
@ehuss ehuss added this to the 1.15.0 milestone Feb 6, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
5 participants