Skip to content

Conversation

alexcrichton
Copy link
Member

Just applied a patch to help fix the tests.

shepmaster and others added 5 commits July 27, 2016 19:33
Previously, we would bail from the fingerprint computation if the old
and new mtimes had changed in *any* way. This caused some issues of
rebuilding with filesystems that do not preserve nanosecond
granularity (rust-lang#2874).
@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Contributor

bors commented Nov 2, 2016

📌 Commit db6db92 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Nov 2, 2016

⌛ Testing commit db6db92 with merge 4b7b567...

@rust-highfive
Copy link

r? @brson

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

@alexcrichton alexcrichton deleted the mtimes-in-the-past branch November 2, 2016 16:35
@alexcrichton alexcrichton restored the mtimes-in-the-past branch November 2, 2016 16:40
@alexcrichton alexcrichton reopened this Nov 2, 2016
@alexcrichton
Copy link
Member Author

Gah oops...

@bors: retry clean

@bors
Copy link
Contributor

bors commented Nov 2, 2016

⌛ Testing commit db6db92 with merge 717adc8...

bors added a commit that referenced this pull request Nov 2, 2016
Continuation of #2880

Just applied a patch to help fix the tests.
@bors
Copy link
Contributor

bors commented Nov 2, 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: alexcrichton
Pushing 717adc8 to master...

@bors bors merged commit db6db92 into rust-lang:master Nov 2, 2016
@alexcrichton alexcrichton deleted the mtimes-in-the-past branch November 2, 2016 23:23
@ehuss ehuss added this to the 1.14.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.

6 participants