Skip to content

[release/5.0] Change bundle branding from '.Net Core' to '.Net' #28682

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

Merged
merged 1 commit into from
Jan 13, 2021

Conversation

wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Dec 15, 2020

Fixes #28658

@joeloff the only instances of the string BundleRegFamily I found in the dotnet org are in this repo - will changing it from .Net Core to .Net affect any downstream repos?

@wtgodbe wtgodbe added the feature-installers Includes: Installers label Dec 15, 2020
@wtgodbe wtgodbe added this to the 5.0.3 milestone Dec 15, 2020
@wtgodbe wtgodbe requested review from joeloff and a team December 15, 2020 23:43
@joeloff
Copy link
Member

joeloff commented Dec 15, 2020

@wtgodbe It shouldn't. This information is only used in the registry when the bundle writes its registration data. Typically under HKLM\SOFTWARE\Microsoft\Updates (IIRC)

It is a simple way to detect an installed bundle, but not a reliable key to use because OS updates can reset it.

@wtgodbe wtgodbe added the tell-mode Indicates a PR which is being merged during tell-mode label Dec 16, 2020
@wtgodbe wtgodbe merged commit c63c4b1 into release/5.0 Jan 13, 2021
@wtgodbe wtgodbe deleted the wtgodbe/50HostingNet branch January 13, 2021 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-installers Includes: Installers tell-mode Indicates a PR which is being merged during tell-mode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants