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

Update Desktop NUSPEC to use ARM64EC #13335

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

JunielKatarn
Copy link
Contributor

@JunielKatarn JunielKatarn commented Jun 11, 2024

Description

Add ARM64EC platform to Desktop NuGet.

Type of Change

  • New feature (non-breaking change which adds functionality)

Why

Currently the Desktop NUSPEC OfficeReact.Win32.nuspec expects source and target paths for ARM64.

What

Change the NUSPEC entries from ARM64 to ARM64EC.

Microsoft Reviewers: Open in CodeFlow

@JunielKatarn JunielKatarn requested review from a team as code owners June 11, 2024 18:59
Copy link

@jeffstall jeffstall left a comment

Choose a reason for hiding this comment

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

I think we need to have both ARM64 and ARM64EC builds. We can't replace one with the other. There's usage cases for both. Can you please update?

@@ -492,13 +492,13 @@ extends:
configuration: Release
- platform: x86
configuration: Release
- platform: ARM64
- platform: ARM64EC

Choose a reason for hiding this comment

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

I think we need to have both ARM64 and ARM64EC builds. We can't replace one with the other. There's usage cases for both. Can you please update?

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Author Feedback The issue/PR needs activity from its author (label drives bot activity) label Jun 21, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the no-recent-activity Issue/PR has gone stale and may be closed (label applied by bot) label Jun 28, 2024
Copy link
Contributor

This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 7 days. It will be closed if no further activity occurs within 7 days of this comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs: Author Feedback The issue/PR needs activity from its author (label drives bot activity) no-recent-activity Issue/PR has gone stale and may be closed (label applied by bot)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants