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

Add osx-arm64 to official RIDs in ILCompiler NuGet #82558

Merged
merged 1 commit into from
Feb 23, 2023

Conversation

filipnavara
Copy link
Member

@filipnavara filipnavara commented Feb 23, 2023

Contributes to #82542

When cross-compiling from osx-x64 to osx-arm64 the target runtime package was not restored correctly. This can be explicitly fixed in the SDK. However, the reason why the opposite (osx-arm64 -> osx-x64) works is because the Microsoft.DotNet.ILCompiler dependency pulls the correct runtime package. The osx-arm64 dependency was missing though, and thus the runtime package was never restored.

More details

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Feb 23, 2023
@ghost
Copy link

ghost commented Feb 23, 2023

Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas
See info in area-owners.md if you want to be subscribed.

Issue Details

Contributes to #82542

When cross-compiling from osx-x64 to osx-arm64 the target runtime package was not restored correctly. This can be explicitly fixed in the SDK. However, the reason why the opposite (osx-arm64 -> osx-x64) works is because the Microsoft.DotNet.ILCompiler dependency pulls the correct runtime package. The osx-arm64 dependency was missing though, and thus the runtime package was never restored.

More details

Author: filipnavara
Assignees: -
Labels:

community-contribution, area-NativeAOT-coreclr

Milestone: -

@am11
Copy link
Member

am11 commented Feb 23, 2023

Neat fix! 👍

Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

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

LGTM!

@filipnavara filipnavara marked this pull request as ready for review February 23, 2023 23:44
@agocke agocke merged commit 4ab53e1 into dotnet:main Feb 23, 2023
@filipnavara filipnavara deleted the patch-7 branch February 23, 2023 23:46
@ghost ghost locked as resolved and limited conversation to collaborators Mar 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-NativeAOT-coreclr community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants