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

Fix the lockfile-compat test #4762

Merged
merged 2 commits into from
Nov 29, 2017
Merged

Fix the lockfile-compat test #4762

merged 2 commits into from
Nov 29, 2017

Conversation

alexcrichton
Copy link
Member

The newest version of tar tweaks the checksum here slightly as the tarball is
slightly different, so this just updates the test to pull the checksum from the
publication rather than hardcoding it.

The newest version of `tar` tweaks the checksum here slightly as the tarball is
slightly different, so this just updates the test to pull the checksum from the
publication rather than hardcoding it.
@rust-highfive
Copy link

r? @matklad

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

@alexcrichton
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented Nov 29, 2017

📌 Commit a0dfbb7 has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented Nov 29, 2017

⌛ Testing commit a0dfbb7 with merge 9831549308ee063b26ecfe0514c7a727adfe8ece...

@bors
Copy link
Collaborator

bors commented Nov 29, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Collaborator

bors commented Nov 29, 2017

📌 Commit 5c7482a has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented Nov 29, 2017

⌛ Testing commit 5c7482a with merge 0f9da74...

bors added a commit that referenced this pull request Nov 29, 2017
Fix the lockfile-compat test

The newest version of `tar` tweaks the checksum here slightly as the tarball is
slightly different, so this just updates the test to pull the checksum from the
publication rather than hardcoding it.
@bors
Copy link
Collaborator

bors commented Nov 29, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 0f9da74 to master...

@bors bors merged commit 5c7482a into rust-lang:master Nov 29, 2017
nrc added a commit to nrc/rust that referenced this pull request Nov 29, 2017
Pulls in rust-lang/cargo#4762 fixing a broken test which prevented the RLS update
@alexcrichton alexcrichton deleted the fix-tests branch December 18, 2017 19:47
@ehuss ehuss added this to the 1.24.0 milestone Feb 6, 2022
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 this pull request may close these issues.

5 participants