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

Fix command_line_flags mis-generation of rust-target's flags. #1186

Merged
merged 1 commit into from Dec 9, 2017

Conversation

Projects
None yet
4 participants
@manaskarekar
Copy link
Contributor

manaskarekar commented Dec 9, 2017

Trying to address issue #1182.

r? @fitzgen

Fix command_line_flags mis-generation of rust-target's flags.
Expand tests for testing the rust-target flag generated by builder.

@manaskarekar manaskarekar force-pushed the manaskarekar:issue-1182 branch from ede3961 to 23d9f12 Dec 9, 2017

@emilio

This comment has been minimized.

Copy link
Collaborator

emilio commented Dec 9, 2017

@bors-servo r+

Thanks!

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 9, 2017

📌 Commit 23d9f12 has been approved by emilio

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 9, 2017

⌛️ Testing commit 23d9f12 with merge c3ec58c...

bors-servo added a commit that referenced this pull request Dec 9, 2017

Auto merge of #1186 - manaskarekar:issue-1182, r=emilio
Fix command_line_flags mis-generation of rust-target's flags.

Trying to address issue  #1182.

r? @fitzgen
@emilio

emilio approved these changes Dec 9, 2017

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 9, 2017

☀️ Test successful - status-travis
Approved by: emilio
Pushing c3ec58c to master...

@bors-servo bors-servo merged commit 23d9f12 into rust-lang:master Dec 9, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.