-
Notifications
You must be signed in to change notification settings - Fork 2.6k
[beta] Don't set MAKEFLAGS for build scripts #4276
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
Conversation
r? @matklad (rust_highfive has picked a reviewer for you, use r? to override) |
@bors: r+ p=1 |
📌 Commit a9237f9 has been approved by |
Should we document |
Yes I can do that as a follow-up |
⌛ Testing commit a9237f9530fe30a557b2ab844afc1c62b91e5dcc with merge aec5bbe2822f8b167f998bafa1dbda3ce2136897... |
💔 Test failed - status-travis |
@bors: r+ |
📌 Commit 5186849 has been approved by |
⌛ Testing commit 5186849 with merge 468dcf6e1217a9546b1528a5f2a9719a7ac90e6c... |
💔 Test failed - status-appveyor |
Needed by rust-lang/rust#42938 I've now ripgrepped for "panicked at" and found no further test that hardcodes the "filename:line$" format.
@bors: r+ |
📌 Commit 931aec1 has been approved by |
[beta] Don't set MAKEFLAGS for build scripts Closes #4156 Closes rust-lang/rust#42635
☀️ Test successful - status-appveyor, status-travis |
Closes #4156
Closes rust-lang/rust#42635