Skip to content

Conversation

@TimNN
Copy link
Contributor

@TimNN TimNN commented May 6, 2017

This was missing from #41739, thanks @tedhorst for noticing!

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@Mark-Simulacrum
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented May 6, 2017

📌 Commit f5e3427 has been approved by Mark-Simulacrum

@TimNN
Copy link
Contributor Author

TimNN commented May 6, 2017

@bors retry

(this should abort the other build which will fail anyway)

Edit: apparently this needs to go to the other build.

@bors
Copy link
Collaborator

bors commented May 6, 2017

⌛ Testing commit f5e3427 with merge acbe4ad...

bors added a commit that referenced this pull request May 6, 2017
Trigger llvm rebuild

This was missing from #41739, thanks @tedhorst for noticing!
@bors
Copy link
Collaborator

bors commented May 6, 2017

💔 Test failed - status-travis

@TimNN
Copy link
Contributor Author

TimNN commented May 6, 2017

@bors
Copy link
Collaborator

bors commented May 6, 2017

⌛ Testing commit f5e3427 with merge fae255f...

@bors
Copy link
Collaborator

bors commented May 6, 2017

⌛ Testing commit f5e3427 with merge 2cf6af1...

bors added a commit that referenced this pull request May 6, 2017
Trigger llvm rebuild

This was missing from #41739, thanks @tedhorst for noticing!
@bors
Copy link
Collaborator

bors commented May 6, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: Mark-Simulacrum
Pushing 2cf6af1 to master...

@bors bors merged commit f5e3427 into rust-lang:master May 6, 2017
@TimNN TimNN deleted the trigger-llvm branch May 7, 2017 17:33
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