-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[release/9.0.3xx] Update dependencies from dotnet/msbuild #48005
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
[release/9.0.3xx] Update dependencies from dotnet/msbuild #48005
Conversation
…0328.17 Microsoft.SourceBuild.Intermediate.msbuild , Microsoft.Build , Microsoft.Build.Localization From Version 17.14.0-preview-25177-05 -> To Version 17.14.0-preview-25178-17
The failures look related. cc @carlossanlop @rainersigwald |
…darc-release/9.0.3xx-3f4a1dd1-ab01-4a37-86b4-c2c5a7bc3491
…1-4a37-86b4-c2c5a7bc3491
…0401.10 Microsoft.SourceBuild.Intermediate.msbuild , Microsoft.Build , Microsoft.Build.Localization From Version 17.14.0-preview-25177-05 -> To Version 17.14.1-preview-25201-10
I'm seeing:
I saw ViktorHofer assigned you, @carlossanlop, are you handling this? |
The System.Text.Json error does not look familiar or related to the changes we've been making in maintenance-packages. @rainersigwald do you know if the changes from maintenance-packages merged in main in msbuild are being flowed into this 9.0.3xx branch? I also see a warning stating that netcoreapp2.1 is out of support but I don't think we're treating warnings as errors.
|
both main and 9.0.3xx builds should have the maintenance-packages changes reverted at the moment. |
…1-4a37-86b4-c2c5a7bc3491
…0402.7 Microsoft.SourceBuild.Intermediate.msbuild , Microsoft.Build , Microsoft.Build.Localization From Version 17.14.0-preview-25177-05 -> To Version 17.14.2-preview-25202-07
…0404.3 Microsoft.SourceBuild.Intermediate.msbuild , Microsoft.Build , Microsoft.Build.Localization From Version 17.14.0-preview-25177-05 -> To Version 17.14.3-preview-25204-03
The latest commit contains the maintenance-packages changes again. We're seeing the same issue in #48212 |
Can you link to the pr(s) to revert the updates? I see dotnet/msbuild#11684 as making them, and I don't see anything reverting that |
…0407.8 Microsoft.SourceBuild.Intermediate.msbuild , Microsoft.Build , Microsoft.Build.Localization From Version 17.14.0-preview-25177-05 -> To Version 17.14.4-preview-25207-08
@carlossanlop now that the work in #48034 is completed, can you please take a look at this dependency flow for 9.0.3xx? I assume we need to backport a subset of the changes done in the runtime PR. |
The latest maestro update included @rainersigwald 's change in this flow: dotnet/msbuild#11677 There are test failures. I'm investigating. |
I see failures only in the FullFramework tests job: https://dev.azure.com/dnceng-public/public/_build/results?buildId=1008785&view=logs&j=b69c0128-beae-536a-0344-1c918351dbc7&t=60696cef-0589-543b-3dbb-b76b5b65d586&l=334 One failure is can't find Microsoft.CodeAnalysis.Razor.Compiler Version 9.0.0.0:
Another failure is can't find System.Text.Json methods, both are methods that use span as arguments, so it could be related to System.Memory. Found two variants:
and
|
…es of System.Text.Json.
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.
I pushed a commit for these two tasks so they are forced to consume the toolset version of System.Text.Json like they do in main:
- src/RazorSdk/Tasks/Microsoft.NET.Sdk.Razor.Tasks.csproj
- src/StaticWebAssetsSdk/Tasks/Microsoft.NET.Sdk.StaticWebAssets.Tasks.csproj
Are we sure these are running on a VS that has the updated MSBuild and bindingRedirects? What ensures that? |
This reverts commit 728a9c6.
The tests are on an MSBuild/VS that lags, and that's fairly intentional. |
Reference specific old versions of these packages at compile time, which will be moved forward to the VS- delivered versions at runtime. This avoids pinning them to the old versions for the whole repo, allowing MSBuild to move forward without breaking SDK. Partial backport of 9cf989b.
@dsplaisted I (manually) backported your 9cf989b to unstick this. |
…1-4a37-86b4-c2c5a7bc3491
<PackageReference Include="Microsoft.Build.Utilities.Core" ExcludeAssets="Runtime" /> | ||
<PackageReference Include="System.Reflection.Metadata" VersionOverride="$(SystemReflectionMetadataVersion)" Condition="'$(TargetFramework)'=='net472'" /> | ||
<PackageReference Include="System.Text.Json" Condition="'$(TargetFramework)'=='net472'" /> | ||
<PackageReference Include="System.Text.Json" VersionOverride="$(SystemTextJsonToolsetPackageVersion)" /> |
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.
In main, this reference is in a netframework conditioned itemgroup. Should it be here as well?
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.
I don't think it matters much since this should get dropped on core but might as well make the merge as easy as possible.
…91' of https://github.com/dotnet/sdk into darc-release/9.0.3xx-3f4a1dd1-ab01-4a37-86b4-c2c5a7bc3491
This pull request updates the following dependencies
From https://github.com/dotnet/msbuild