Skip to content

Update Microsoft.Build package references#11518

Closed
Redth wants to merge 1 commit into
mainfrom
deps/update-msbuild-package
Closed

Update Microsoft.Build package references#11518
Redth wants to merge 1 commit into
mainfrom
deps/update-msbuild-package

Conversation

@Redth
Copy link
Copy Markdown
Member

@Redth Redth commented May 27, 2026

Summary

  • Update Microsoft.Build package references from 17.14.28 to 18.6.3
  • Add a direct System.Reflection.Metadata package reference for the shared base tasks project

Validation

  • dotnet restore tests/MSBuildDeviceIntegration/MSBuildDeviceIntegration.csproj --configfile nuget.config --disable-parallel --verbosity minimal
  • dotnet restore src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/Xamarin.Android.Build.Tests.csproj --configfile nuget.config --disable-parallel --verbosity minimal
  • dotnet restore build-tools/Xamarin.Android.Tools.BootstrapTasks/Xamarin.Android.Tools.BootstrapTasks.csproj --configfile nuget.config --disable-parallel --verbosity minimal
  • dotnet build external/xamarin-android-tools/src/Microsoft.Android.Build.BaseTasks/Microsoft.Android.Build.BaseTasks.csproj --no-restore --verbosity minimal
  • dotnet build build-tools/xa-prep-tasks/xa-prep-tasks.csproj --no-restore --verbosity minimal
  • dotnet build build-tools/Xamarin.Android.Tools.BootstrapTasks/Xamarin.Android.Tools.BootstrapTasks.csproj --no-restore --verbosity minimal

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 27, 2026 16:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to centralize/upgrade MSBuild-related package versions and ensure Microsoft.Android.Build.BaseTasks has an explicit System.Reflection.Metadata dependency during restore/build.

Changes:

  • Added a new MSBuildPackageReferenceVersion property (intended for MSBuild package versioning).
  • Added a conditional PackageReference to System.Reflection.Metadata for the Microsoft.Android.Build.BaseTasks project.

Comment thread Directory.Build.props
Comment on lines 51 to 56
<NuGetApiPackageVersion>5.4.0</NuGetApiPackageVersion>
<LZ4PackageVersion>1.3.6</LZ4PackageVersion>
<MicrosoftTestingPlatformVersion>2.2.1</MicrosoftTestingPlatformVersion>
<MSBuildPackageReferenceVersion>18.6.3</MSBuildPackageReferenceVersion>
<MonoOptionsVersion>6.12.0.148</MonoOptionsVersion>
<SystemCollectionsImmutableVersion>8.0.0</SystemCollectionsImmutableVersion>
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is used in the xamarin-android-tools submodule.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it has a point, that we needed to update the other repo:

But there is something weird going on with System.Reflection.Metadata, like they removed a transitive dependency.

@jonathanpeppers
Copy link
Copy Markdown
Member

Closing in favor of:

Reading the changes here, copilot was making up stuff because it was in the wrong repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants