-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Allow supplying an error destination via the compiler driver #36760
Conversation
r? @eddyb (rust_highfive has picked a reviewer for you, use r? to override) |
@bors r+ |
📌 Commit 85743df has been approved by |
Failing with:
|
@bors r- |
r? @eddyb |
☔ The latest upstream changes (presumably #36678) made this pull request unmergeable. Please resolve the merge conflicts. |
rebased, ping @eddyb for r? |
@bors r+ |
📌 Commit 80d9d80 has been approved by |
⌛ Testing commit 80d9d80 with merge a50a754... |
💔 Test failed - auto-win-gnu-64-opt |
@bors: retry On Tue, Sep 27, 2016 at 7:33 PM, bors notifications@github.com wrote:
|
⌛ Testing commit 80d9d80 with merge 476607e... |
looks like real test failure |
💔 Test failed - auto-win-msvc-64-opt-rustbuild |
@bors: retry On Tue, Sep 27, 2016 at 7:59 PM, bors notifications@github.com wrote:
|
⌛ Testing commit 80d9d80 with merge 6a2f6fc... |
Allows replacing stderr with a buffer from the client. Also, some refactoring around run_compiler.
@bors r=eddyb |
📌 Commit e8a4db2 has been approved by |
⌛ Testing commit e8a4db2 with merge ae94afc... |
💔 Test failed - auto-win-msvc-64-opt-rustbuild |
Hm I haven't seen that before, but sure does look spurious. Let's see if it actually is! @bors: r+ |
💡 This pull request was already approved, no need to approve it again.
|
📌 Commit e8a4db2 has been approved by |
@bors retry |
Allow supplying an error destination via the compiler driver Allows replacing stderr with a buffer from the client. Also, some refactoring around run_compiler.
Allows replacing stderr with a buffer from the client.
Also, some refactoring around run_compiler.