Skip to content

[release/5.0.2xx] Backport x64 on ARM64 installer changes #12494

Merged
ericstj merged 3 commits intodotnet:release/5.0.2xxfrom
ericstj:5.0.2-x64onARM64
Oct 25, 2021
Merged

[release/5.0.2xx] Backport x64 on ARM64 installer changes #12494
ericstj merged 3 commits intodotnet:release/5.0.2xxfrom
ericstj:5.0.2-x64onARM64

Conversation

@ericstj
Copy link
Member

@ericstj ericstj commented Oct 22, 2021

Customer Impact

Customers cannot install the 3.1 SDK nor 5.0 SDK on ARM64 machines. This was an intentional decision during our first wave of changes. We would like a call on adjusting that.
We would like to consider a change that would permit installation of the 3.1 x86 SDK, the 5.0 x86 and Arm64 SDK on Arm64 machines.

Testing

Manual build of bundle to validate conditions.

Risk

Low, but schedule risk to 3.1/5.0 if we take the change. Changes are only to installer so build impact/time is low but it will cause a delay.

* Remove writing InstallLocation from SDK, it will be handled by host

* Update WiX to signed build (dotnet#12078)

# Conflicts:
#	eng/Versions.props
#	src/finalizer_shim/finalizer_shim.csproj

* Update WiX to 3.14.0.5722 (dotnet#12252)

* Add installation checks to prevent install on ARM64

* Update wording of blockers

* Ensure we allow specifying DOTNETALLOWINSTALLONARM64

Co-authored-by: Jacques Eloff <joeloff@users.noreply.github.com>
@ericstj ericstj changed the title [release/5.02xx] Backport x64 on ARM64 installer changes [release/5.0.2xx] Backport x64 on ARM64 installer changes Oct 22, 2021
Copy link
Member

@joeloff joeloff left a comment

Choose a reason for hiding this comment

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

Do we need to fix up the final screen here to remove the reporting of DOTNETHOME or is that already part of 5.0.2?

@ericstj
Copy link
Member Author

ericstj commented Oct 22, 2021

Do we need to fix up the final screen here to remove the reporting of DOTNETHOME

We don't support installing the SDK ARM64 downlevel so that fix was never ported.

@ericstj ericstj merged commit 63a5652 into dotnet:release/5.0.2xx Oct 25, 2021
@ericstj
Copy link
Member Author

ericstj commented Oct 25, 2021

Approved over mail

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants