-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Rustbuild regression fixes #43430
Rustbuild regression fixes #43430
Conversation
@bors p=100 (once this is reviewed) |
@bors: r+ |
📌 Commit 003bbfc has been approved by |
⌛ Testing commit 003bbfceb6ee9ea90eb7725d7cdcff6344402000 with merge d05e8853f3f061b1a9a08f317ecbb632b10d8cc9... |
@bors r- retry |
@bors r+ p=100 |
📌 Commit bcc610a has been approved by |
⌛ Testing commit bcc610a76f15a21dbcac065f1e6cdf5581141842 with merge 6e5f4acdf4346fce513f96d080e6f61ca7e79ae5... |
💔 Test failed - status-travis |
Fuchsia failed to build LLVM at stage1 (why?). Legit.
|
bcc610a
to
9bad3dc
Compare
@bors r=alexcrichton I believe the last commit makes dist and install equivalent to what the old build system ran, which should hopefully resolve our issues. |
📌 Commit 9bad3dc has been approved by |
@Mark-Simulacrum The PR CI failed to build bootstrap.
|
⌛ Testing commit 9bad3dcb9671b6dd597d3d9e1ae529cc2775e680 with merge 3c4fc0f3f66785c410b8ee3245aa26b23d50ef38... |
💔 Test failed - status-appveyor |
9bad3dc
to
fef9b48
Compare
@bors r=alexcrichton |
📌 Commit fef9b48 has been approved by |
⌛ Testing commit fef9b48 with merge 2ad68c47023a625d693c69b88e1d9f0a476bf48b... |
💔 Test failed - status-travis |
@bors retry
|
Rustbuild regression fixes I believe this will fix #43427. This PR also avoids running pretty tests by-default. r? @alexcrichton
☀️ Test successful - status-appveyor, status-travis |
I believe this will fix #43427.
This PR also avoids running pretty tests by-default.
r? @alexcrichton