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

Bring test of nightly in line with tests #4010

Merged
merged 1 commit into from
May 9, 2017

Conversation

jonhoo
Copy link
Contributor

@jonhoo jonhoo commented May 9, 2017

#4000 passes -Zincremental to rustc only on nightly, but uses a different mechanism for detecting nightly than cargotest does. This PR brings the two in line, which should hopefully fix the build failure observed in rust-lang/rust#41830 (comment).

@rust-highfive
Copy link

r? @alexcrichton

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

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented May 9, 2017

📌 Commit e0e4a1d has been approved by alexcrichton

@jonhoo
Copy link
Contributor Author

jonhoo commented May 9, 2017

It looks like homu isn't processing approved PRs for cargo or rustup-rs. Do you know why that is @Mark-Simulacrum?

@alexcrichton
Copy link
Member

@bors: retry

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented May 9, 2017

💡 This pull request was already approved, no need to approve it again.

@bors
Copy link
Collaborator

bors commented May 9, 2017

📌 Commit e0e4a1d has been approved by alexcrichton

@alexcrichton alexcrichton force-pushed the better-incremental-nightly-test branch from e0e4a1d to 213a2bf Compare May 9, 2017 15:22
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented May 9, 2017

📌 Commit 213a2bf has been approved by alexcrichton

@alexcrichton
Copy link
Member

@bors: retry

@jonhoo
Copy link
Contributor Author

jonhoo commented May 9, 2017

@alexcrichton it doesn't look like the issue is the approval, but rather that homu isn't taking jobs off of the queue. Jobs in the rust repo seem to progress just fine..

@bors
Copy link
Collaborator

bors commented May 9, 2017

⌛ Testing commit 213a2bf with merge cf17c9f...

bors added a commit that referenced this pull request May 9, 2017
…richton

Bring test of nightly in line with tests

#4000 passes `-Zincremental` to `rustc` only on nightly, but uses a different mechanism for detecting nightly than [cargotest does](https://github.com/rust-lang/cargo/blob/9bf9bddd9297cfb5098be6146d85be551c6d4eff/tests/cargotest/lib.rs#L37). This PR brings the two in line, which should hopefully fix the build failure observed in rust-lang/rust#41830 (comment).
jonhoo added a commit to jonhoo/rust that referenced this pull request May 9, 2017
@bors
Copy link
Collaborator

bors commented May 9, 2017

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

@bors bors merged commit 213a2bf into rust-lang:master May 9, 2017
@ehuss ehuss added this to the 1.19.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