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

Add registry search for upgrade policy keys #20106

Merged
merged 3 commits into from
Oct 7, 2024

Conversation

joeloff
Copy link
Member

@joeloff joeloff commented Sep 5, 2024

Description

Add registry search operations to SDK 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. This PR only includes infrastructure changes to support changes in the Windows installer bundles. The change depends on changes in Arcade (dotnet/arcade#15047).

NB: This PR will not build until the Arcade change is merged and flowed to the SDK.

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

Risk

Low, this change only adds detection for the key, nothing will currently act on its value.

Testing

Testing can be done once all the relevant Arcade/Wix changes are merged and have flowed through.

@joeloff joeloff requested a review from a team September 5, 2024 17:17
Copy link
Member

@nagilson nagilson left a comment

Choose a reason for hiding this comment

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

LGTM, bar a few questions/concerns.

@@ -393,7 +396,7 @@
InstallerFile="$(CombinedFrameworkSdkHostMSIInstallerFile)"
WixExtensions="WixBalExtension;WixUtilExtension;WixTagExtension"
WixSrcFiles="@(BundleMsiWixSrcFiles)"
AdditionalBasePaths="$(MSBuildThisFileDirectory)packaging/windows/clisdk">
AdditionalBasePaths="$(MSBuildThisFileDirectory)packaging/windows/clisdk;$(PkgMicrosoft_DotNet_Build_Tasks_Installers)\build\wix\bundle">
Copy link
Member

Choose a reason for hiding this comment

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

What is this property used for? There are 2 other references to this property in other LightCommandPackageDrop creations. Should those also provide $(PkgMicrosoft_DotNet_Build_Tasks_Installers)\build\wix\bundle?

Copy link
Member Author

Choose a reason for hiding this comment

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

It modifies when/how we'd perform upgrades.

Copy link
Member

Choose a reason for hiding this comment

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

Thank you for the explanations.

src/redist/targets/GenerateMSIs.targets Show resolved Hide resolved
@joeloff
Copy link
Member Author

joeloff commented Oct 4, 2024

Depends on merging dotnet/arcade#15047 and having that flow into the installer repo

@joeloff
Copy link
Member Author

joeloff commented Oct 4, 2024

/backport to release/8.0.3xx

@joeloff
Copy link
Member Author

joeloff commented Oct 4, 2024

/backport to release/8.0.4xx

Copy link

github-actions bot commented Oct 4, 2024

Started backporting to release/8.0.3xx: https://github.com/dotnet/installer/actions/runs/11184350561

Copy link

github-actions bot commented Oct 4, 2024

Started backporting to release/8.0.4xx: https://github.com/dotnet/installer/actions/runs/11184352998

@joeloff
Copy link
Member Author

joeloff commented Oct 7, 2024

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@joeloff joeloff merged commit 8504b3d into dotnet:release/8.0.1xx Oct 7, 2024
18 checks passed
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.

4 participants