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

[mono][workload] Add workloads for win-arm64 using emulation #65089

Merged
merged 2 commits into from
Mar 25, 2022

Conversation

lewing
Copy link
Member

@lewing lewing commented Feb 9, 2022

This pairs with dotnet/emsdk#109 and would be a workaround until we have native builds of the tooling. I don't have a way to test this so we'll need some manual checks to verify.

@steveisok
Copy link
Member

We should do the same for android and iOS

@lewing
Copy link
Member Author

lewing commented Feb 9, 2022

We should do the same for android and iOS

done, I added a win-arm64 alias everywhere we had a win-x64 alias

@lewing lewing changed the title [wasm][workload] Add wasm workload for win-arm64 using emulation [mono][workload] Add workloads for win-arm64 using emulation Feb 9, 2022
@@ -166,6 +166,7 @@
"version": "${PackageVersion}",
"alias-to": {
"win-x64": "Microsoft.NETCore.App.Runtime.AOT.win-x64.Cross.android-x86",
"win-arm64": "Microsoft.NETCore.App.Runtime.AOT.win-x64.Cross.android-x86",
Copy link
Member

Choose a reason for hiding this comment

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

@Redth when you're testing win-arm64, can you validate if this .exe runs:

> "C:\Program Files\dotnet\packs\Microsoft.NETCore.App.Runtime.AOT.win-x64.Cross.android-x86\6.0.2-mauipre.1.22074.5\tools\mono-aot-cross.exe"

If it prints help output, I think that means it would work.

@ghost
Copy link

ghost commented Mar 12, 2022

Draft Pull Request was automatically closed for inactivity. Please let us know if you'd like to reopen it.

@ghost ghost closed this Mar 12, 2022
@lewing lewing reopened this Mar 24, 2022
@lewing lewing marked this pull request as ready for review March 24, 2022 16:23
@lewing lewing requested a review from marek-safar as a code owner March 24, 2022 16:23
@lewing lewing merged commit 4e7ccfa into dotnet:main Mar 25, 2022
@lewing lewing deleted the win-wasm-arm64-emu branch March 25, 2022 18:22
lewing added a commit that referenced this pull request Mar 25, 2022
steveisok pushed a commit that referenced this pull request Mar 25, 2022
lewing added a commit that referenced this pull request Mar 28, 2022
radekdoulik pushed a commit to radekdoulik/runtime that referenced this pull request Mar 30, 2022
…65089)

* Add wasm workload for win-arm64 using emulation

* Add aliases for win-arm64 to all the win-x64 cases as well (android, ios)
radekdoulik pushed a commit to radekdoulik/runtime that referenced this pull request Mar 30, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Apr 24, 2022
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.

3 participants