Skip to content

Commit 7f0aeb7

Browse files
authored
Clean-up ExtraPackageVersionPropsPackageInfo items (#46340)
1 parent 63f75ac commit 7f0aeb7

File tree

2 files changed

+18
-35
lines changed

2 files changed

+18
-35
lines changed

src/SourceBuild/content/repo-projects/Directory.Build.props

Lines changed: 1 addition & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -195,47 +195,13 @@
195195
<WindowsDesktopSdkOverride Include="Microsoft.Net.Sdk.WindowsDesktop" Group="WINDOWS_DESKTOP" Location="$(ToolsDir)EmptySdk" Condition="'$(DotNetBuildSourceOnly)' == 'true'" />
196196
</ItemGroup>
197197

198-
<!-- CLI internal version is statically set by us to a version that will never show up in the wild.
199-
This ensures we will never restore a public version instead of our source-built version. We
200-
invlude the version number because it is used both by CLI.proj and the core-sdk build and they
201-
have to be synced up. ExtraPackageVersionPropsPackageInfo doesn't work in cli.proj because
202-
toolset is between CLI and core-sdk, and the extra package version info is lost. -->
203-
<PropertyGroup>
204-
<CliInternalReleaseTag>source</CliInternalReleaseTag>
205-
<CliInternalBuildVersion>30000001-1</CliInternalBuildVersion>
206-
</PropertyGroup>
207-
208-
<ItemGroup>
209-
<ExtraPackageVersionPropsPackageInfo Include="DotnetCliInternalVersion" Version="3.0.100-$(CliInternalReleaseTag)-$(CliInternalBuildVersion)" />
210-
</ItemGroup>
211-
212-
<!-- Additional pseudo-versions that some repos depend on -->
198+
<!-- Additional pseudo-versions that **multiple** repos depend on. -->
213199
<ItemGroup>
214-
<!-- we don't produce the Windows version of this package but that's the one sdk keys off of for the ASP.NET version -->
215-
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftAspNetCoreAppRuntimewinx64PackageVersion" Version="%24(MicrosoftAspNetCoreAppRefPackageVersion)" />
216-
<!-- same thing here for CLI -->
217200
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftNETCoreAppRuntimewinx64Version" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
218-
<!-- same thing here for toolset -->
219201
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftNETCoreAppRuntimewinx64PackageVersion" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
220-
<!-- same thing here for sdk -->
221-
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftNETCoreAppRuntimePackageVersion" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
222202
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftNETCoreAppRuntimeVersion" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
223-
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftNETCoreAppHostPackageVersion" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
224-
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftNETCoreAppHostwinx64PackageVersion" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
225203

226-
<!-- Used by windowsdesktop to determine msi version -->
227204
<ExtraPackageVersionPropsPackageInfo Include="VSRedistCommonNetCoreSharedFrameworkx64100PackageVersion" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
228-
229-
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftWindowsDesktopAppRuntimewinx64PackageVersion" Version="%24(MicrosoftWindowsDesktopAppRefPackageVersion)" />
230-
231-
<!-- Used by installer partition in sdk repo to determine sdk version -->
232-
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftDotnetToolsetInternalPackageVersion" Version="%24(MicrosoftNETSdkPackageVersion)" />
233-
234-
<!-- Used by installer partition to determine the non-shipping runtime version -->
235-
<ExtraPackageVersionPropsPackageInfo Include="VSRedistCommonNetCoreTargetingPackx64100PackageVersion" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
236-
237-
<!-- Used by sdk to determine msbuild version for fsharp -->
238-
<ExtraPackageVersionPropsPackageInfo Include="FSharpBuildVersion" Version="%24(MicrosoftBuildPackageVersion)" />
239205
</ItemGroup>
240206

241207
</Project>

src/SourceBuild/content/repo-projects/sdk.proj

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,4 +89,21 @@
8989
<EnvironmentVariables Include="PublishWindowsPdb=false" />
9090
</ItemGroup>
9191

92+
<ItemGroup>
93+
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftWindowsDesktopAppRuntimewinx64PackageVersion" Version="%24(MicrosoftWindowsDesktopAppRefPackageVersion)" />
94+
95+
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftNETCoreAppRuntimePackageVersion" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
96+
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftNETCoreAppHostPackageVersion" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
97+
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftNETCoreAppHostwinx64PackageVersion" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
98+
99+
<!-- we don't produce the Windows version of this package but that's the one sdk keys off of for the ASP.NET version -->
100+
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftAspNetCoreAppRuntimewinx64PackageVersion" Version="%24(MicrosoftAspNetCoreAppRefPackageVersion)" />
101+
102+
<!-- Used by installer partition to determine the non-shipping runtime version -->
103+
<ExtraPackageVersionPropsPackageInfo Include="VSRedistCommonNetCoreTargetingPackx64100PackageVersion" Version="%24(MicrosoftNETCoreAppRefPackageVersion)" />
104+
105+
<!-- Used by installer partition in sdk repo to determine sdk version -->
106+
<ExtraPackageVersionPropsPackageInfo Include="MicrosoftDotnetToolsetInternalPackageVersion" Version="%24(MicrosoftNETSdkPackageVersion)" />
107+
</ItemGroup>
108+
92109
</Project>

0 commit comments

Comments
 (0)