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.

Sign up for GitHub

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

Jump to bottom

Update compiler_builtins submodule for probestack fix #43116

Merged
merged 1 commit into from
Jul 11, 2017
Merged

Update compiler_builtins submodule for probestack fix #43116

merged 1 commit into from
Jul 11, 2017

Conversation

alexcrichton
Copy link
Member

Closes #43102

@rust-highfive
Copy link
Collaborator

r? @aturon

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

@carols10cents carols10cents added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 10, 2017
@sfackler
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Jul 10, 2017

📌 Commit b8b4f56 has been approved by sfackler

@est31
Copy link
Member

est31 commented Jul 10, 2017

@bors p=1 this affects many people, servo people included

@bors
Copy link
Contributor

bors commented Jul 11, 2017

⌛ Testing commit b8b4f56 with merge 53b5778dea32cf398763ba8af1fa6f3e86f0768f...

@bors
Copy link
Contributor

bors commented Jul 11, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: sfackler
Pushing 53b5778dea32cf398763ba8af1fa6f3e86f0768f to master...

@bors
Copy link
Contributor

bors commented Jul 11, 2017

👀 Test was successful, but fast-forwarding failed: 422 Update is not a fast forward

@aidanhs
Copy link
Member

aidanhs commented Jul 11, 2017

@bors retry

(seen before in #43104, retry fixed)

@arielb1 arielb1 added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 11, 2017
@kennytm
Copy link
Member

kennytm commented Jul 11, 2017

cc servo/homu#24.

@bors
Copy link
Contributor

bors commented Jul 11, 2017

⌛ Testing commit b8b4f56 with merge e6076044bce04c07913135cc9170659b299bff46...

@bors
Copy link
Contributor

bors commented Jul 11, 2017

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

@bors
Copy link
Contributor

bors commented Jul 11, 2017

👀 Test was successful, but fast-forwarding failed: 422 Update is not a fast forward

@est31
Copy link
Member

est31 commented Jul 11, 2017

@bors retry

let's try again...

@aidanhs
Copy link
Member

aidanhs commented Jul 11, 2017

Hmm, I can't see any manual merging so not entirely sure what's going on. Will bring it up at next infra meeting.

@bors
Copy link
Contributor

bors commented Jul 11, 2017

⌛ Testing commit b8b4f56 with merge 9475ae4...

bors added a commit that referenced this pull request Jul 11, 2017
@bors
Auto merge of #43116 - alexcrichton:probestack, r=sfackler
9475ae4 
Update compiler_builtins submodule for probestack fix

Closes #43102
@bors
Copy link
Contributor

bors commented Jul 11, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: sfackler
Pushing 9475ae4 to master...

@bors bors merged commit b8b4f56 into rust-lang:master Jul 11, 2017
@alexcrichton alexcrichton deleted the probestack branch July 24, 2017 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@aturon aturon

Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@alexcrichton @rust-highfive @sfackler @bors @est31 @aidanhs @kennytm @carols10cents @aturon @arielb1