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.

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

[beta] Don't set MAKEFLAGS for build scripts #4276

Merged
merged 3 commits into from
Jul 13, 2017

Conversation

alexcrichton
Copy link
Member

Closes #4156
Closes rust-lang/rust#42635

@rust-highfive
Copy link

r? @matklad

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

@rust-highfive
Copy link

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.19.0. Please double check that you specified the right target!

@alexcrichton alexcrichton changed the title Don't set MAKEFLAGS for build scripts [beta] Don't set MAKEFLAGS for build scripts Jul 12, 2017
@alexcrichton
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented Jul 12, 2017

📌 Commit a9237f9 has been approved by alexcrichton

@matklad
Copy link
Member

matklad commented Jul 12, 2017

Should we document CARGO_MAKEFLAGS ?

@alexcrichton
Copy link
Member Author

Yes I can do that as a follow-up

@bors
Copy link
Collaborator

bors commented Jul 12, 2017

⌛ Testing commit a9237f9530fe30a557b2ab844afc1c62b91e5dcc with merge aec5bbe2822f8b167f998bafa1dbda3ce2136897...

@bors
Copy link
Collaborator

bors commented Jul 12, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Collaborator

bors commented Jul 12, 2017

📌 Commit 5186849 has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented Jul 12, 2017

⌛ Testing commit 5186849 with merge 468dcf6e1217a9546b1528a5f2a9719a7ac90e6c...

@bors
Copy link
Collaborator

bors commented Jul 12, 2017

💔 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.
@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Collaborator

bors commented Jul 13, 2017

📌 Commit 931aec1 has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented Jul 13, 2017

⌛ Testing commit 931aec1 with merge e7a185b...

bors added a commit that referenced this pull request Jul 13, 2017
[beta] Don't set MAKEFLAGS for build scripts

Closes #4156
Closes rust-lang/rust#42635
@bors
Copy link
Collaborator

bors commented Jul 13, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing e7a185b to rust-1.19.0...

@bors bors merged commit 931aec1 into rust-lang:rust-1.19.0 Jul 13, 2017
@alexcrichton alexcrichton deleted the revert2 branch July 18, 2018 15:56
@ehuss ehuss added this to the 1.19.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants