-
Notifications
You must be signed in to change notification settings - Fork 663
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
WinUIGallery Submodule Update in WinUI3: Fixes #1802
Conversation
/azp run |
/azp run |
<Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003"> | ||
<PropertyGroup> | ||
<PublishProtocol>FileSystem</PublishProtocol> | ||
<Platform>x64</Platform> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why should ARM64-ec point to x64 and not ARM?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
arm64ec and x64 are interchangeable. Arm64ec is basically x64 under the hood but with arm64
We were also running into this error that setting the pubxml to point to x64 runtime identifier solves.
C:\ProgramFiles\dotnet\sdk\9.0.200\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.FrameworkReferenceResolution.targets(107,5): error NETSDK1083: The specified RuntimeIdentifier 'win-arm64ec' is not recognized. See https://aka.ms/netsdk1083 for more information. [C:__w\1\s\Samples\WinUIGallery\WinUIGallery.UnitTests\WinUIGallery.UnitTests.csproj]
Description
Various fixes to be able to update the WinUIGallery's Submodule in WinUI3.
Motivation and Context
We should update WinUI3 to use the main branch of WinUIGallery. Currently it is on a stale topic branch.
How Has This Been Tested?
Ran in the PR pipelines with the submodule updated
Screenshots (if appropriate):
Types of changes