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

[Reopen] Fix ANCM installer on ARM64 #59481

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

Conversation

halter73
Copy link
Member

This is effectively reopening @lextm's PR (#47290) from over a year ago. I would have actually reopened that PR if GitHub let me, but it did not.

Screenshot showing the "Reopen pull request" button is disabled

We have gotten more reports of how broken the ANCM ARM64 installer is when you enable x86 and x64 emulation. Enabling emulation with ANCM installed will cause all app pools to crash on startup, even those that aren't using ANCM. This is all well described in the issue @lextm filed at #47115, but we missed the severity of the impact until now.

@lextm, I found your blogpost describing your valiant attempt to get this merged the first time. I'm very sorry for the "The Not-So-Happy Ending". I'm hoping to correct it now. If you want to reopen the PR yourself, please feel welcome. I don't expect that though.

Otherwise, I know you've had a lot of experience since then helping people out with https://github.com/lextm/ancm-arm64. If there's anything you want us to change about this PR, please let us know. Thanks for all your hard work filing the issue, submitting a PR, and then helping people manually patch the installer.

Fixes #47115

@halter73 halter73 requested review from wtgodbe and a team as code owners December 14, 2024 01:10
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Dec 14, 2024
Copy link
Contributor

Hey @dotnet/aspnet-build, looks like this PR is something you want to take a look at.

@lextm
Copy link

lextm commented Dec 14, 2024

@halter73 Thanks for reopening the discussion.

The patches for #47290 have been out-of-date by now. I have improved and tested the required changes elsewhere, so I opened a new pull request #59483 to share those improvements. Compared to the original pull request, it also fixed the IIS Express ARM64 installer.

I suggest that the reviewers focus on #59483 instead of this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ASP.NET Core module installer should be improved further for Windows ARM64
2 participants