-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[release/6.0] Use full suffixed version for DepProviderKey in installers #39035
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
Conversation
|
Hi @wtgodbe. If this is not a tell-mode PR, please make sure to follow the instructions laid out in the servicing process document. |
| <TargetingPackInstallerBaseName>aspnetcore-targeting-pack</TargetingPackInstallerBaseName> | ||
|
|
||
| <!-- This is used to produce targeting pack installers/packages once per major.minor. --> | ||
| <!-- This is used to produce targeting pack installers/packages once per major.minor, or when we need to service it (e.g. 6.0.2) --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is to address the package code issue with the repair scenario in the short term, correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
|
@TanayParikh this is still failing on macOS as if you did not change the |
This source nor the target branches have the macos timeout related fixes from yesterday. Backport discussion here: #39032 |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |

6.0 port of #39022.
Fixes a bug where preview, RC, and RTM targeting pack installers all have the same Dependency Provider Key due to using only Major.Minor.Patch as their version string. Also turns on build of the targeting pack for 6.0.2.