Skip to content

Conversation

wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Sep 18, 2024

Test build: https://dev.azure.com/dnceng/internal/_build/results?buildId=2541983&view=results

Aspnetcore version of dotnet/arcade#15048

Description

Add registry search operations to Aspnetcore installer bundles. The search operation will check for a global and version specific registry key. The version specific key takes precedence when present.

This is related to a customer request and part of a larger change.

Unlike the runtime and desktop runtime, Aspnetcore needs to explicitly pull in the additional source file.

Risk

Medium/Low - this change modifies how bundles behave when upgrading. It is based off a modified copy of wixstdba (bootstrapper and UI layer) with changes to Burn (setup engine) and a copy of WiX v3 built from source.

Testing

We've been testing the change for a while now. Verified against a private build of Aspnetcore. After installing preview7, followed by a new RC2 build, both runtimes remain present, and the RunOnce key for the removal is present for preview7.

@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Sep 18, 2024
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.

LGTM - let's see what the test results look like

@wtgodbe wtgodbe marked this pull request as ready for review September 19, 2024 17:49
@wtgodbe wtgodbe requested a review from a team as a code owner September 19, 2024 17:49
@wtgodbe wtgodbe added Servicing-consider Shiproom approval is required for the issue Servicing-approved Shiproom has approved the issue and removed Servicing-consider Shiproom approval is required for the issue labels Sep 19, 2024
@wtgodbe wtgodbe merged commit 74bdb8b into release/9.0-rc2 Sep 19, 2024
25 checks passed
@wtgodbe wtgodbe deleted the wtgodbe/Wixin branch September 19, 2024 22:18
@dotnet-policy-service dotnet-policy-service bot added this to the 9.0-rc2 milestone Sep 19, 2024
wtgodbe added a commit that referenced this pull request Sep 26, 2024
wtgodbe added a commit that referenced this pull request Sep 26, 2024
* Add registry search for upgrade policy keys

* SharedFx bundle too

* Add util extension

* Another fix
wtgodbe added a commit that referenced this pull request Oct 7, 2024
* Add registry search for upgrade policy keys

* SharedFx bundle too

* Add util extension

* Another fix
dotnet-maestro bot added a commit that referenced this pull request Oct 7, 2024
… Arcade (#58278)

Add registry search for upgrade policy keys, update dependencies from Arcade


 - Add registry search for upgrade policy keys (#57952)

* Add registry search for upgrade policy keys

* SharedFx bundle too

* Add util extension

* Another fix
wtgodbe added a commit that referenced this pull request Oct 30, 2024
* Add registry search for upgrade policy keys

* SharedFx bundle too

* Add util extension

* Another fix
captainsafia pushed a commit that referenced this pull request Dec 31, 2024
* Add registry search for upgrade policy keys

* SharedFx bundle too

* Add util extension

* Another fix
captainsafia pushed a commit that referenced this pull request Dec 31, 2024
captainsafia pushed a commit that referenced this pull request Feb 11, 2025
* Add registry search for upgrade policy keys

* SharedFx bundle too

* Add util extension

* Another fix
captainsafia pushed a commit that referenced this pull request Feb 11, 2025
captainsafia pushed a commit that referenced this pull request Feb 11, 2025
* Add registry search for upgrade policy keys

* SharedFx bundle too

* Add util extension

* Another fix
This was referenced Aug 15, 2025
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 Servicing-approved Shiproom has approved the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants