Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Commit a4296bf

Browse files
authored
[release/2.1] Always build VS insertion packages as prerelease (#8989)
* Always build VS insertion packages as prerelease * Fix VSInsertionVersionSuffix => VSInsertionVersion Improper naming, it's the full version, not a suffix.
1 parent a4fa713 commit a4296bf

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

src/pkg/packaging/dir.proj

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -210,25 +210,31 @@
210210
<NuSpecFile>$(MSBuildThisFileDirectory)windows/vscomponents/VS.Redist.Common.Component.nuspec</NuSpecFile>
211211
<GenerateNupkgPowershellScript>$(MSBuildThisFileDirectory)windows/vscomponents/generatenupkg.ps1</GenerateNupkgPowershellScript>
212212
<ProjectUrl>https://github.com/dotnet/core-setup</ProjectUrl>
213+
<!--
214+
Always build VS insertion packages with non-stable version. Ensures uniqueness for each
215+
build, even if building stable multiple times in a row. Necessary because these are
216+
published to an immutable feed before release day.
217+
-->
218+
<VSInsertionVersion>$(ProductionVersion)-$(PreReleaseLabel)-$(BuildNumberMajor)-$(BuildNumberMinor)</VSInsertionVersion>
213219
</PropertyGroup>
214220

215221
<ItemGroup>
216222
<SdkMsiComponent Include="HostFxrMsiNupkg">
217223
<MsiInstallerFile>$(HostFxrInstallerFile)</MsiInstallerFile>
218224
<ComponentId>VS.Redist.Common.NetCore.HostFXR.$(TargetArchitecture).$(NETCoreAppFrameworkVersion)</ComponentId>
219-
<ComponentVersion>$(SharedFrameworkNugetVersion)</ComponentVersion>
225+
<ComponentVersion>$(VSInsertionVersion)</ComponentVersion>
220226
<ComponentFriendlyName>$(NETCoreAppFrameworkVersion) .NET Core HostFX Resolver</ComponentFriendlyName>
221227
</SdkMsiComponent>
222228
<SdkMsiComponent Include="SharedFrameworkMsiNupkg">
223229
<MsiInstallerFile>$(SharedFrameworkInstallerFile)</MsiInstallerFile>
224230
<ComponentId>VS.Redist.Common.NetCore.SharedFramework.$(TargetArchitecture).$(NETCoreAppFrameworkVersion)</ComponentId>
225-
<ComponentVersion>$(SharedFrameworkNugetVersion)</ComponentVersion>
231+
<ComponentVersion>$(VSInsertionVersion)</ComponentVersion>
226232
<ComponentFriendlyName>$(NETCoreAppFrameworkVersion) .NET Core SharedFramework</ComponentFriendlyName>
227233
</SdkMsiComponent>
228234
<SdkMsiComponent Include="SharedHostMsiNupkg">
229235
<MsiInstallerFile>$(SharedHostInstallerFile)</MsiInstallerFile>
230236
<ComponentId>VS.Redist.Common.NetCore.SharedHost.$(TargetArchitecture).$(NETCoreAppFrameworkVersion)</ComponentId>
231-
<ComponentVersion>$(SharedFrameworkNugetVersion)</ComponentVersion>
237+
<ComponentVersion>$(VSInsertionVersion)</ComponentVersion>
232238
<ComponentFriendlyName>$(NETCoreAppFrameworkVersion) .NET Core SharedHost</ComponentFriendlyName>
233239
</SdkMsiComponent>
234240
</ItemGroup>

0 commit comments

Comments
 (0)