Skip to content

Foundation: build CoreFoundation as /MD #1919

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

Merged
merged 1 commit into from
Feb 16, 2019

Conversation

compnerd
Copy link
Member

Use /MD and /MDd mode compilation rather than /MT or /MTd as per
Microsoft's recommendation. This allows us to use a shared C runtime.
The static MSVCRT/ucrt cannot be intermingled with a dynamic one.

@compnerd
Copy link
Member Author

CC: @millenomi @phausler @parkera

@compnerd
Copy link
Member Author

@swift-ci please test and merge

@parkera
Copy link
Contributor

parkera commented Feb 15, 2019

Seems reasonable

@compnerd
Copy link
Member Author

@swift-ci please test and merge

1 similar comment
@compnerd
Copy link
Member Author

@swift-ci please test and merge

Use /MD and /MDd mode compilation rather than /MT or /MTd as per
Microsoft's recommendation.  This allows us to use a shared C runtime.
The static MSVCRT/ucrt cannot be intermingled with a dynamic one.
@compnerd
Copy link
Member Author

@swift-ci please test

@gwynne
Copy link
Contributor

gwynne commented Feb 16, 2019

@swift-ci please test

@compnerd compnerd merged commit 4888a0e into swiftlang:master Feb 16, 2019
@compnerd compnerd deleted the trust-me-i-am-a-MD branch February 16, 2019 21:21
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.

3 participants