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

Update the cc crate #71882

Merged
merged 1 commit into from
May 10, 2020
Merged

Update the cc crate #71882

merged 1 commit into from
May 10, 2020

Conversation

alexcrichton
Copy link
Member

Pulls in updated MSVC detection logic landed in rust-lang/cc-rs#488

Pulls in updated MSVC detection logic landed in rust-lang/cc-rs#488
@rust-highfive
Copy link
Collaborator

r? @Mark-Simulacrum

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 4, 2020
@alexcrichton
Copy link
Member Author

Given bugs like rust-lang/cc-rs#493 I think this may also be a good candidate for backporting since it should be pretty low-risk and may be good to accelerate getting this out to users.

@Mark-Simulacrum
Copy link
Member

Looks like the diff here is rust-lang/cc-rs@1.0.50...1.0.52 -- that all looks reasonable to me. I agree that the MSVC version detection logic being broken on at least some systems it seems good to backport this so I'm going to go ahead and nominate.

@bors r+

@bors
Copy link
Contributor

bors commented May 4, 2020

📌 Commit 3318187 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 May 4, 2020
@Mark-Simulacrum Mark-Simulacrum added beta-nominated Nominated for backporting to the compiler in the beta channel. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels May 4, 2020
@pnkfelix
Copy link
Member

pnkfelix commented May 7, 2020

discussed at T-compiler meeting. Accepted for beta-backport.

@pnkfelix pnkfelix added the beta-accepted Accepted for backporting to the compiler in the beta channel. label May 7, 2020
@Mark-Simulacrum Mark-Simulacrum removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label May 9, 2020
RalfJung added a commit to RalfJung/rust that referenced this pull request May 10, 2020
…lacrum

Update the `cc` crate

Pulls in updated MSVC detection logic landed in rust-lang/cc-rs#488
This was referenced May 10, 2020
@Dylan-DPC-zz
Copy link

@bors p=1

bors added a commit to rust-lang-ci/rust that referenced this pull request May 10, 2020
Rollup of 4 pull requests

Successful merges:

 - rust-lang#71840 (Rework MIR drop tree lowering)
 - rust-lang#71882 (Update the `cc` crate)
 - rust-lang#71945 (Sort "implementations on foreign types" section in the sidebar)
 - rust-lang#72043 (Add missing backtick in E0569 explanation)

Failed merges:

r? @ghost
@bors
Copy link
Contributor

bors commented May 10, 2020

⌛ Testing commit 3318187 with merge 9912925...

@bors bors merged commit 37a9192 into rust-lang:master May 10, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request May 11, 2020
…ulacrum

[beta] backports

This backport rollup includes the following:
 * [beta] Update cargo rust-lang#71967
 * Backport 1.43.1 release notes to master rust-lang#71914
 * Update the `cc` crate rust-lang#71882
 * resolve: Relax fresh binding disambiguation slightly to fix regression rust-lang#71846
 * Do not try to find binop method on RHS `TyErr` rust-lang#71810
 * Quick and dirty fix of the unused_braces lint rust-lang#71517
 * Update stdarch submodule rust-lang#71495
 * normalize field projection ty to fix broken MIR issue rust-lang#71488
 * Update openssl-src to 1.1.1g rust-lang#71430
 * fix error code in E0751.md rust-lang#71426
 * Don't fuse Chain in its second iterator rust-lang#71404
 * Remove some `Vec` allocations to improve performance rust-lang#71268
 * [CI] Use the latest Python available on Windows rust-lang#71995

It also switches the bootstrap compiler to 1.43.1.
@alexcrichton alexcrichton deleted the update-cc branch July 23, 2020 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants