Skip to content

Set the MSVC RuntimeLibrary properties for ASM_MASM when we use MASM.#44155

Merged
jkoritzinsky merged 1 commit into
dotnet:masterfrom
jkoritzinsky:masm-runtimelib-patch
Nov 2, 2020
Merged

Set the MSVC RuntimeLibrary properties for ASM_MASM when we use MASM.#44155
jkoritzinsky merged 1 commit into
dotnet:masterfrom
jkoritzinsky:masm-runtimelib-patch

Conversation

@jkoritzinsky

Copy link
Copy Markdown
Member

Set the MSVC RuntimeLibrary properties for ASM_MASM since they aren't populated until CMake 3.16.4 due to a bug in CMake.

This is currently blocking local builds for anyone using a CMake version prior to 3.16.4.

… populated until CMake 3.16.4 due to a bug in CMake.
@jkoritzinsky jkoritzinsky added this to the 6.0.0 milestone Nov 2, 2020
@jkoritzinsky
jkoritzinsky requested review from a team and stephentoub November 2, 2020 17:24
@ghost

ghost commented Nov 2, 2020

Copy link
Copy Markdown

Tagging subscribers to this area: @ViktorHofer
See info in area-owners.md if you want to be subscribed.

@ghost

ghost commented Nov 2, 2020

Copy link
Copy Markdown

Hello @jkoritzinsky!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@jkoritzinsky

Copy link
Copy Markdown
Member Author

Merging in now to unblock people since the fix has been validated.

@jkoritzinsky
jkoritzinsky merged commit 598426c into dotnet:master Nov 2, 2020
@jkoritzinsky
jkoritzinsky deleted the masm-runtimelib-patch branch November 2, 2020 17:45

@safern safern left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ghost ghost locked as resolved and limited conversation to collaborators Dec 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to build dotnet/runtime due to cmake error about MultiThreadedDebug

3 participants