Skip to content

Conversation

ubsan and others added 30 commits June 24, 2016 15:28
This is a partial fix for issue rust-lang#34808.

Most of the wording was copied verbatim from the RFC. Main difference is that I moved the actual template to the top of the section.

It also makes the error explanations the longest section in the guidelines doc for now.
@steveklabnik
Copy link
Contributor Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented Jul 25, 2016

📌 Commit e3e138b has been approved by steveklabnik

@rust-highfive
Copy link
Contributor

r? @alexcrichton

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

@bors
Copy link
Collaborator

bors commented Jul 25, 2016

⌛ Testing commit e3e138b with merge 8c1bca5...

@bors
Copy link
Collaborator

bors commented Jul 25, 2016

💔 Test failed - auto-win-gnu-32-opt-rustbuild

@arielb1
Copy link
Contributor

arielb1 commented Jul 25, 2016

@bors retry

command did not execute successfully: "git" "submodule" "update"
expected success, got: exit code: 1


Unable to find current revision in submodule path 'src/llvm'

@bors
Copy link
Collaborator

bors commented Jul 25, 2016

⌛ Testing commit e3e138b with merge c513608...

bors added a commit that referenced this pull request Jul 25, 2016
Rollup of 13 pull requests

- Successful merges: #34461, #34609, #34732, #34850, #34894, #34935, #34974, #34990, #34995, #35001, #35009, #35010, #35028
- Failed merges:
@bors
Copy link
Collaborator

bors commented Jul 25, 2016

💔 Test failed - auto-win-gnu-32-opt

@steveklabnik
Copy link
Contributor Author

@bors: retry

@bors
Copy link
Collaborator

bors commented Jul 25, 2016

⌛ Testing commit e3e138b with merge 8483c2f...

@bors
Copy link
Collaborator

bors commented Jul 25, 2016

💔 Test failed - auto-win-gnu-32-opt

@alexcrichton
Copy link
Member

@bors: retry

On Mon, Jul 25, 2016 at 2:37 PM, bors [email protected] wrote:

💔 Test failed - auto-win-gnu-32-opt
https://buildbot.rust-lang.org/builders/auto-win-gnu-32-opt/builds/5065


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

@bors
Copy link
Collaborator

bors commented Jul 26, 2016

⌛ Testing commit e3e138b with merge 10a7bc0...

@bors
Copy link
Collaborator

bors commented Jul 26, 2016

💔 Test failed - auto-win-msvc-64-cargotest

@alexcrichton
Copy link
Member

@bors: retry

On Tue, Jul 26, 2016 at 2:53 AM, bors [email protected] wrote:

💔 Test failed - auto-win-msvc-64-cargotest
https://buildbot.rust-lang.org/builders/auto-win-msvc-64-cargotest/builds/1279


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#35036 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD95DOgoiRSfSTLxtpr-ST1-f4wRoMYks5qZdkRgaJpZM4JUbPA
.

@bors
Copy link
Collaborator

bors commented Jul 26, 2016

⌛ Testing commit e3e138b with merge 751f2a4...

@bors
Copy link
Collaborator

bors commented Jul 26, 2016

💔 Test failed - auto-win-gnu-32-opt-rustbuild

@alexcrichton
Copy link
Member

Ok that seems legitimate.

cc @vadimcn though, the failure to do a backtrace seems worrisome.

@vadimcn
Copy link
Contributor

vadimcn commented Jul 26, 2016

#35014 should fix that

@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.