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] Backports #68189

Merged
merged 8 commits into from
Jan 14, 2020
Merged

[Beta] Backports #68189

merged 8 commits into from
Jan 14, 2020

Conversation

jonas-schievink
Copy link
Contributor

I did not include #67134 and #67289 since they seem to be on beta already.

jsgf and others added 8 commits January 13, 2020 22:19
PR #66512 added the ability to set argv[0] on
Command. As a side effect, it changed the Debug output to print both the program and
argv[0], which in practice results in stuttery output ("echo echo foo").

This PR reverts the behaviour to the the old one, so that the command is only printed
once - unless arg0 has been set. In that case it emits "[command] arg0 arg1 ...".
@rust-highfive
Copy link
Collaborator

r? @varkor

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

@rust-highfive
Copy link
Collaborator

⚠️ Warning ⚠️

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 13, 2020
@jonas-schievink
Copy link
Contributor Author

r? @Mark-Simulacrum

@Mark-Simulacrum
Copy link
Member

@bors r+ rollup=never (beta branch) p=1

@bors
Copy link
Contributor

bors commented Jan 13, 2020

📌 Commit 289cd17 has been approved by Mark-Simulacrum

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 13, 2020
@jonas-schievink
Copy link
Contributor Author

@bors p=1

@bors
Copy link
Contributor

bors commented Jan 14, 2020

⌛ Testing commit 289cd17 with merge 965ce16...

bors added a commit that referenced this pull request Jan 14, 2020
[Beta] Backports

I did not include #67134 and #67289 since they seem to be on beta already.

* Fix up Command Debug output when arg0 is specified. #67219
* Do not ICE on unnamed future #67289
* Don't suppress move errors for union fields #67314
* Reenable static linking of libstdc++ on windows-gnu #67410
* Use the correct type for static qualifs #67621
* Treat extern statics just like statics in the "const pointer to static" representation #67630
* Do not ICE on lifetime error involving closures #67687
@bors
Copy link
Contributor

bors commented Jan 14, 2020

☀️ Test successful - checks-azure
Approved by: Mark-Simulacrum
Pushing 965ce16 to beta...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jan 14, 2020
@bors bors merged commit 289cd17 into rust-lang:beta Jan 14, 2020
@jonas-schievink jonas-schievink deleted the beta-next branch January 28, 2020 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants