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

Ignore summaries in downloaded crates #3217

Merged
merged 1 commit into from
Oct 26, 2016

Conversation

alexcrichton
Copy link
Member

Unfortunately historical Cargo bugs have made it such that the index sometimes
differs from the actual crate we download. Let's respect the index, however,
which should be our source of truth.

Closes #3214

@rust-highfive
Copy link

r? @brson

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

@brson
Copy link
Contributor

brson commented Oct 20, 2016

@bors r+

@bors
Copy link
Contributor

bors commented Oct 20, 2016

📌 Commit 41ff55e has been approved by brson

@bors
Copy link
Contributor

bors commented Oct 20, 2016

⌛ Testing commit 41ff55e with merge 51eec2c...

bors added a commit that referenced this pull request Oct 20, 2016
Ignore summaries in downloaded crates

Unfortunately historical Cargo bugs have made it such that the index sometimes
differs from the actual crate we download. Let's respect the index, however,
which should be our source of truth.

Closes #3214
@bors
Copy link
Contributor

bors commented Oct 20, 2016

💔 Test failed - cargo-mac-64

@alexcrichton
Copy link
Member Author

@bors: r=brson

On Thu, Oct 20, 2016 at 3:21 PM, bors [email protected] wrote:

💔 Test failed - cargo-mac-64
https://buildbot.rust-lang.org/builders/cargo-mac-64/builds/1862


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#3217 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD95AnkZqSSLewlJZQIAogQb6L2Tumbks5q1-lMgaJpZM4KcqGj
.

@bors
Copy link
Contributor

bors commented Oct 25, 2016

📌 Commit d4633fd has been approved by brson

@bors
Copy link
Contributor

bors commented Oct 25, 2016

⌛ Testing commit d4633fd with merge 2277ba6...

@bors
Copy link
Contributor

bors commented Oct 25, 2016

💔 Test failed - cargo-win-msvc-64

Unfortunately historical Cargo bugs have made it such that the index sometimes
differs from the actual crate we download. Let's respect the index, however,
which should be our source of truth.

Closes rust-lang#3214
@alexcrichton
Copy link
Member Author

@bors: r=brson

@bors
Copy link
Contributor

bors commented Oct 26, 2016

📌 Commit c8a5dcd has been approved by brson

@bors
Copy link
Contributor

bors commented Oct 26, 2016

⌛ Testing commit c8a5dcd with merge be010b5...

bors added a commit that referenced this pull request Oct 26, 2016
Ignore summaries in downloaded crates

Unfortunately historical Cargo bugs have made it such that the index sometimes
differs from the actual crate we download. Let's respect the index, however,
which should be our source of truth.

Closes #3214
@bors
Copy link
Contributor

bors commented Oct 26, 2016

☀️ Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: brson
Pushing be010b5 to master...

@bors bors merged commit c8a5dcd into rust-lang:master Oct 26, 2016
@alexcrichton alexcrichton deleted the fix-regression branch November 2, 2016 16:35
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.

4 participants