Skip to content

Commit 3ee7d29

Browse files
JanProvaznikvseanreesermsft
authored andcommitted
Merged PR 53411: [internal/release/8.0.4xx] update msbuild
update msbuild ---- #### AI description (iteration 4) #### PR Classification This PR performs a dependency update by upgrading msbuild and aligning related package configurations. #### PR Summary The PR upgrades msbuild from version 17.11.45 to 17.11.48, ensuring version consistency across the build by updating package sources and overlaying newer SDK assemblies. - **`NuGet.config`**: Updates package source keys for msbuild and removes deprecated entries. - **`src/Layout/redist/targets/GenerateLayout.targets`**: Adds steps to copy newer Microsoft.Build assemblies from the SDK for consistency. - **`eng/Version.Details.xml`**: Updates the msbuild dependency details and source URIs. - **`eng/Versions.props`**: Aligns msbuild version properties to the new version. <!-- GitOpsUserAgent=GitOps.Apps.Server.pullrequestcopilot -->
2 parents 9a7ab2a + 1458a83 commit 3ee7d29

File tree

4 files changed

+30
-12
lines changed

4 files changed

+30
-12
lines changed

NuGet.config

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
<add key="darc-int-dotnet-aspnetcore-22b4865" value="https://pkgs.dev.azure.com/dnceng/internal/_packaging/darc-int-dotnet-aspnetcore-22b4865a/nuget/v3/index.json" />
1818
<!-- End: Package sources from dotnet-aspnetcore -->
1919
<!-- Begin: Package sources from DotNet-msbuild-Trusted -->
20-
<add key="darc-pub-DotNet-msbuild-Trusted-70f0ec6" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/darc-pub-DotNet-msbuild-Trusted-70f0ec66/nuget/v3/index.json" />
20+
<add key="darc-int-DotNet-msbuild-Trusted-02bf662" value="https://pkgs.dev.azure.com/dnceng/internal/_packaging/darc-int-DotNet-msbuild-Trusted-02bf6629/nuget/v3/index.json" />
2121
<!-- End: Package sources from DotNet-msbuild-Trusted -->
2222
<!-- Begin: Package sources from dotnet-roslyn-analyzers -->
2323
<!-- End: Package sources from dotnet-roslyn-analyzers -->
@@ -57,6 +57,7 @@
5757
<add key="darc-int-dotnet-aspnetcore-22b4865" value="true" />
5858
<!-- End: Package sources from dotnet-aspnetcore -->
5959
<!-- Begin: Package sources from DotNet-msbuild-Trusted -->
60+
<add key="darc-int-DotNet-msbuild-Trusted-02bf662" value="true" />
6061
<!-- End: Package sources from DotNet-msbuild-Trusted -->
6162
<!-- Begin: Package sources from dotnet-format -->
6263
<!-- End: Package sources from dotnet-format -->

eng/Version.Details.xml

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -55,17 +55,17 @@
5555
<Uri>https://github.com/dotnet/emsdk</Uri>
5656
<Sha>ea366a7a40bc588c70ef59376040130baf864c85</Sha>
5757
</Dependency>
58-
<Dependency Name="Microsoft.Build" Version="17.11.45">
59-
<Uri>https://github.com/dotnet/msbuild</Uri>
60-
<Sha>70f0ec66d526ee9e8c658bc6c6c2cf6498286fb6</Sha>
58+
<Dependency Name="Microsoft.Build" Version="17.11.48">
59+
<Uri>https://dev.azure.com/devdiv/DevDiv/_git/DotNet-msbuild-Trusted</Uri>
60+
<Sha>02bf66295b64ab368d12933041f7281aad186a2d</Sha>
6161
</Dependency>
62-
<Dependency Name="Microsoft.Build.Localization" Version="17.11.45-servicing-25452-07">
63-
<Uri>https://github.com/dotnet/msbuild</Uri>
64-
<Sha>70f0ec66d526ee9e8c658bc6c6c2cf6498286fb6</Sha>
62+
<Dependency Name="Microsoft.Build.Localization" Version="17.11.48-servicing-25466-05">
63+
<Uri>https://dev.azure.com/devdiv/DevDiv/_git/DotNet-msbuild-Trusted</Uri>
64+
<Sha>02bf66295b64ab368d12933041f7281aad186a2d</Sha>
6565
</Dependency>
66-
<Dependency Name="Microsoft.SourceBuild.Intermediate.msbuild" Version="17.11.45-servicing-25452-07">
67-
<Uri>https://github.com/dotnet/msbuild</Uri>
68-
<Sha>70f0ec66d526ee9e8c658bc6c6c2cf6498286fb6</Sha>
66+
<Dependency Name="Microsoft.SourceBuild.Intermediate.msbuild" Version="17.11.48-servicing-25466-05">
67+
<Uri>https://dev.azure.com/devdiv/DevDiv/_git/DotNet-msbuild-Trusted</Uri>
68+
<Sha>02bf66295b64ab368d12933041f7281aad186a2d</Sha>
6969
<SourceBuild RepoName="msbuild" ManagedOnly="true" />
7070
</Dependency>
7171
<Dependency Name="Microsoft.FSharp.Compiler" Version="12.8.403-beta.24526.2">

eng/Versions.props

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@
126126
</PropertyGroup>
127127
<PropertyGroup>
128128
<!-- Dependencies from https://github.com/Microsoft/msbuild -->
129-
<MicrosoftBuildPackageVersion>17.11.45</MicrosoftBuildPackageVersion>
129+
<MicrosoftBuildPackageVersion>17.11.48</MicrosoftBuildPackageVersion>
130130
<MicrosoftBuildCurrentPackageVersion>$(MicrosoftBuildPackageVersion)</MicrosoftBuildCurrentPackageVersion>
131131
<!-- Some tasks and the resolver will need to run in an MSBuild that is older than the very latest,
132132
so target one that matches the version in minimumMSBuildVersion.
@@ -135,7 +135,7 @@
135135
<MicrosoftBuildMinimumVersion Condition="'$(DotNetBuildFromSource)' != 'true'">17.8.27</MicrosoftBuildMinimumVersion>
136136
<MicrosoftBuildFrameworkPackageVersion>$(MicrosoftBuildMinimumVersion)</MicrosoftBuildFrameworkPackageVersion>
137137
<MicrosoftBuildRuntimePackageVersion>$(MicrosoftBuildMinimumVersion)</MicrosoftBuildRuntimePackageVersion>
138-
<MicrosoftBuildLocalizationPackageVersion>17.11.45-servicing-25452-07</MicrosoftBuildLocalizationPackageVersion>
138+
<MicrosoftBuildLocalizationPackageVersion>17.11.48-servicing-25466-05</MicrosoftBuildLocalizationPackageVersion>
139139
<MicrosoftBuildUtilitiesCorePackageVersion>$(MicrosoftBuildMinimumVersion)</MicrosoftBuildUtilitiesCorePackageVersion>
140140
<MicrosoftBuildTasksCorePackageVersion>$(MicrosoftBuildMinimumVersion)</MicrosoftBuildTasksCorePackageVersion>
141141
<MicrosoftBuildTasksCoreVersion>$(MicrosoftBuildTasksCorePackageVersion)</MicrosoftBuildTasksCoreVersion>

src/Layout/redist/targets/GenerateLayout.targets

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,10 +117,27 @@
117117
<DotnetFormatConfigFiles Include="$(Pkgdotnet-format)/tools/net8.0/any/**/*.config" />
118118
</ItemGroup>
119119

120+
<!-- SDK Microsoft.Build assemblies to overlay for version consistency -->
121+
<ItemGroup>
122+
<SdkMicrosoftBuildFiles Include="$(OutputPath)/Microsoft.Build.dll" />
123+
<SdkMicrosoftBuildFiles Include="$(OutputPath)/Microsoft.Build.Framework.dll" />
124+
<SdkMicrosoftBuildFiles Include="$(OutputPath)/Microsoft.Build.Tasks.Core.dll" />
125+
<SdkMicrosoftBuildFiles Include="$(OutputPath)/Microsoft.Build.Utilities.Core.dll" />
126+
</ItemGroup>
127+
120128
<Error Condition="'@(DotnetFormatDllFiles)' == ''" Text="Something moved around in dotnet-format package, adjust code here accordingly. TFM change?" />
121129
<Error Condition="'@(DotnetFormatConfigFiles)' == ''" Text="Something moved around in dotnet-format package, adjust code here accordingly. TFM change?" />
130+
131+
<!-- Copy dotnet-format files first -->
122132
<Copy SourceFiles="@(DotnetFormatDllFiles)" DestinationFiles="@(DotnetFormatDllFiles->'$(DotnetFormatDestinationDirectory)/%(RecursiveDir)%(Filename)%(Extension)')" />
123133
<Copy SourceFiles="@(DotnetFormatConfigFiles)" DestinationFiles="@(DotnetFormatConfigFiles->'$(DotnetFormatDestinationDirectory)/%(RecursiveDir)%(Filename)%(Extension)')" />
134+
135+
<!-- Overlay newer Microsoft.Build.* assemblies from SDK for version consistency
136+
this does not fix deps.json inconsistencies but ensures we don't ship vulnerable packages -->
137+
<Copy SourceFiles="@(SdkMicrosoftBuildFiles)"
138+
DestinationFolder="$(DotnetFormatDestinationDirectory)"
139+
Condition="Exists('%(FullPath)')"
140+
OverwriteReadOnlyFiles="true" />
124141
</Target>
125142

126143
<Target Name="PublishMSBuildExtensions"

0 commit comments

Comments
 (0)