Skip to content

rustc: Link statically to the MSVCRT #39837

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Feb 18, 2017

Conversation

alexcrichton
Copy link
Member

This commit changes all MSVC rustc binaries to be compiled with
-C target-feature=+crt-static to link statically against the MSVCRT instead of
dynamically (as it does today). This also necessitates compiling LLVM in a
different fashion, ensuring it's compiled with /MT instead of /MD.

cc #37406

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@brson
Copy link
Contributor

brson commented Feb 15, 2017

@bors r+

@bors
Copy link
Collaborator

bors commented Feb 15, 2017

📌 Commit b2c4e74 has been approved by brson

@brson brson added the relnotes Marks issues that should be documented in the release notes of the next release. label Feb 15, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 15, 2017
rustc: Link statically to the MSVCRT

This commit changes all MSVC rustc binaries to be compiled with
`-C target-feature=+crt-static` to link statically against the MSVCRT instead of
dynamically (as it does today). This also necessitates compiling LLVM in a
different fashion, ensuring it's compiled with `/MT` instead of `/MD`.

cc rust-lang#37406
bors added a commit that referenced this pull request Feb 16, 2017
Rollup of 11 pull requests

- Successful merges: #39775, #39793, #39804, #39824, #39834, #39837, #39839, #39840, #39843, #39844, #39846
- Failed merges:
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 16, 2017
rustc: Link statically to the MSVCRT

This commit changes all MSVC rustc binaries to be compiled with
`-C target-feature=+crt-static` to link statically against the MSVCRT instead of
dynamically (as it does today). This also necessitates compiling LLVM in a
different fashion, ensuring it's compiled with `/MT` instead of `/MD`.

cc rust-lang#37406
@frewsxcv
Copy link
Member

@bors r-

I'm going to guess these errors are relevant https://ci.appveyor.com/project/rust-lang/rust/build/1.0.1974

@alexcrichton
Copy link
Member Author

They are indeed! @frewsxcv feel free to leave my PRs out of rollups, I tend to have a propensity to cause breakage...

This commit changes all MSVC rustc binaries to be compiled with
`-C target-feature=+crt-static` to link statically against the MSVCRT instead of
dynamically (as it does today). This also necessitates compiling LLVM in a
different fashion, ensuring it's compiled with `/MT` instead of `/MD`.

cc rust-lang#37406
@alexcrichton
Copy link
Member Author

@bors: r=brson

@bors
Copy link
Collaborator

bors commented Feb 16, 2017

📌 Commit c02c44d has been approved by brson

@bors
Copy link
Collaborator

bors commented Feb 16, 2017

⌛ Testing commit c02c44d with merge 715ba66...

@bors
Copy link
Collaborator

bors commented Feb 16, 2017

💔 Test failed - status-appveyor

@alexcrichton
Copy link
Member Author

alexcrichton commented Feb 16, 2017 via email

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Feb 16, 2017
rustc: Link statically to the MSVCRT

This commit changes all MSVC rustc binaries to be compiled with
`-C target-feature=+crt-static` to link statically against the MSVCRT instead of
dynamically (as it does today). This also necessitates compiling LLVM in a
different fashion, ensuring it's compiled with `/MT` instead of `/MD`.

cc rust-lang#37406
@bors
Copy link
Collaborator

bors commented Feb 17, 2017

⌛ Testing commit c02c44d with merge 309a294...

@bors
Copy link
Collaborator

bors commented Feb 17, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member Author

alexcrichton commented Feb 17, 2017 via email

@bors
Copy link
Collaborator

bors commented Feb 17, 2017

⌛ Testing commit c02c44d with merge 01dc103...

@bors
Copy link
Collaborator

bors commented Feb 17, 2017

💔 Test failed - status-appveyor

@alexcrichton
Copy link
Member Author

@bors: retry

  • network error

@bors
Copy link
Collaborator

bors commented Feb 17, 2017

⌛ Testing commit c02c44d with merge 6429d91...

@bors
Copy link
Collaborator

bors commented Feb 17, 2017

💔 Test failed - status-appveyor

@alexcrichton
Copy link
Member Author

alexcrichton commented Feb 17, 2017 via email

@bors
Copy link
Collaborator

bors commented Feb 17, 2017

⌛ Testing commit c02c44d with merge 2b11f48...

@nikomatsakis nikomatsakis assigned brson and unassigned nikomatsakis Feb 17, 2017
@bors
Copy link
Collaborator

bors commented Feb 17, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member Author

alexcrichton commented Feb 17, 2017 via email

@bors
Copy link
Collaborator

bors commented Feb 17, 2017

⌛ Testing commit c02c44d with merge 23a0c26...

bors added a commit that referenced this pull request Feb 17, 2017
rustc: Link statically to the MSVCRT

This commit changes all MSVC rustc binaries to be compiled with
`-C target-feature=+crt-static` to link statically against the MSVCRT instead of
dynamically (as it does today). This also necessitates compiling LLVM in a
different fashion, ensuring it's compiled with `/MT` instead of `/MD`.

cc #37406
@bors
Copy link
Collaborator

bors commented Feb 18, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: brson
Pushing 23a0c26 to master...

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
relnotes Marks issues that should be documented in the release notes of the next release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants