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

[release/7.0.1xx] repos/sdk.proj: set NativeAotSupported=false on arm platform #15039

Merged
merged 1 commit into from
Jan 13, 2023

Conversation

ayakael
Copy link

@ayakael ayakael commented Nov 28, 2022

Just like on mono-flavored runtime, arm does not support NativeAot. Thus, when building on that platform, the build fails when building on itself as sdk looks for Microsoft.DotNet.ILCompiler when it is never produced by runtime.

Piggy-backing off of dotnet/sdk#28674, fix by setting NativeAotSupported=false when building source-build on arm.

@ayakael ayakael requested a review from a team as a code owner November 28, 2022 14:51
@ayakael ayakael changed the title repos/sdk.proj: set NativeAotSupported=false on arm platform [release/7.0.1xx] repos/sdk.proj: set NativeAotSupported=false on arm platform Nov 28, 2022
@ayakael
Copy link
Author

ayakael commented Dec 12, 2022

dotnet/sdk#28674 is slated for inclusion on 7.0.3. As this PR requires the latter to work, we should aim for merge after 7.0.2 is released.

@crummel
Copy link

crummel commented Jan 12, 2023

@marcpopMSFT This is related to dotnet/sdk#28674, should it go through tactics separately or can we just merge it after that's in?

@marcpopMSFT
Copy link
Member

@crummel are these essentially both required and do you recall if the tactics approval covered both? Could be worth an email to tactics just in case as I don't remember.

@crummel
Copy link

crummel commented Jan 13, 2023

Sent the tactics email.

@crummel
Copy link

crummel commented Jan 13, 2023

Approved by tactics, merging.

@crummel crummel merged commit 4a2d55e into dotnet:release/7.0.1xx Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants