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

Set CMAKE_OSX_DEPLOYMENT_TARGET for all Mono AOT compilers on macOS #57507

Merged
merged 2 commits into from
Aug 17, 2021

Conversation

akoeplinger
Copy link
Member

Not just if we're targetting iOS/tvOS/macOS since the AOT compiler for WASM or Android can run on macOS too.

Fixes #57431

Not just if we're targetting iOS/tvOS/macOS since the AOT compiler for WASM or Android can run on macOS too.

Fixes dotnet#57431
Copy link
Member

@lambdageek lambdageek left a comment

Choose a reason for hiding this comment

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

I'm not sure how much support we have for build != host && host != target kind of builds, but it seems like this is checking the machine that msbuild is running on, not the machine for which it is going to produce binaries.

src/mono/mono.proj Outdated Show resolved Hide resolved
src/mono/mono.proj Outdated Show resolved Hide resolved
@akoeplinger
Copy link
Member Author

Failures are unrelated.

@lewing
Copy link
Member

lewing commented Aug 17, 2021

browser failures are #57501

@lewing lewing merged commit 24e9212 into dotnet:main Aug 17, 2021
@akoeplinger akoeplinger deleted the fix-macos-min-version branch August 17, 2021 01:28
@ghost ghost locked as resolved and limited conversation to collaborators Sep 16, 2021
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.

Blazor WASM AOT not working on MacOS 10.13.6
4 participants