Skip to content

[main] Update dependencies from dotnet/windowsdesktop #47992

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

Merged
merged 5 commits into from
Mar 31, 2025

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Mar 28, 2025

This pull request updates the following dependencies

Coherency Updates

The following updates ensure that dependencies with a CoherentParentDependency
attribute were produced in a build used as input to the parent dependency's build.
See Dependency Description Format

  • Coherency Updates:
    • Microsoft.NET.Sdk.WindowsDesktop: from 10.0.0-preview.4.25177.1 to 10.0.0-preview.4.25178.3 (parent: Microsoft.WindowsDesktop.App.Ref)
    • Microsoft.Dotnet.WinForms.ProjectTemplates: from 10.0.0-preview.4.25174.6 to 10.0.0-preview.4.25177.2 (parent: Microsoft.WindowsDesktop.App.Runtime.win-x64)
    • Microsoft.DotNet.Wpf.ProjectTemplates: from 10.0.0-preview.4.25177.1 to 10.0.0-preview.4.25178.3 (parent: Microsoft.WindowsDesktop.App.Runtime.win-x64)

From https://github.com/dotnet/windowsdesktop

…ld 20250328.1

Microsoft.WindowsDesktop.App.Ref , Microsoft.WindowsDesktop.App.Runtime.win-x64 , VS.Redist.Common.WindowsDesktop.SharedFramework.x64.10.0 , VS.Redist.Common.WindowsDesktop.TargetingPack.x64.10.0
 From Version 10.0.0-preview.4.25177.2 -> To Version 10.0.0-preview.4.25178.1

Dependency coherency updates

Microsoft.NET.Sdk.WindowsDesktop,Microsoft.Dotnet.WinForms.ProjectTemplates,Microsoft.DotNet.Wpf.ProjectTemplates
 From Version 10.0.0-preview.4.25177.1 -> To Version 10.0.0-preview.4.25178.3 (parent: Microsoft.WindowsDesktop.App.Ref
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-CodeFlow untriaged Request triage from a team member labels Mar 28, 2025
Copy link
Contributor Author

Notification for subscribed users from https://github.com/dotnet/windowsdesktop:

@dotnet/wpf-developers

Action requested: Please take a look at this failing automated dependency-flow pull request's checks; failures may be related to changes which originated in your repo.

  • This pull request contains changes from your source repo (https://github.com/dotnet/windowsdesktop) and seems to have failed checks in this PR. Please take a peek at the failures and comment if they seem relevant to your changes.
  • If you're being tagged in this comment it is due to an entry in the related Maestro Subscription of the Build Asset Registry. If you feel this entry has added your GitHub login or your GitHub team in error, please update the subscription to reflect this.
  • For more details, please read the Arcade Darc documentation

…ld 20250328.2

Microsoft.WindowsDesktop.App.Ref , Microsoft.WindowsDesktop.App.Runtime.win-x64 , VS.Redist.Common.WindowsDesktop.SharedFramework.x64.10.0 , VS.Redist.Common.WindowsDesktop.TargetingPack.x64.10.0
 From Version 10.0.0-preview.4.25177.2 -> To Version 10.0.0-preview.4.25178.2
…ld 20250329.1

Microsoft.WindowsDesktop.App.Ref , Microsoft.WindowsDesktop.App.Runtime.win-x64 , VS.Redist.Common.WindowsDesktop.SharedFramework.x64.10.0 , VS.Redist.Common.WindowsDesktop.TargetingPack.x64.10.0
 From Version 10.0.0-preview.4.25177.2 -> To Version 10.0.0-preview.4.25179.1
@v-wuzhai
Copy link
Member

@dotnet/source-build @dotnet/product-construction PTAL

@ViktorHofer ViktorHofer requested review from a team as code owners March 31, 2025 07:49
@ViktorHofer ViktorHofer enabled auto-merge (squash) March 31, 2025 07:51
@ViktorHofer
Copy link
Member

ViktorHofer commented Mar 31, 2025

C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018: The "GenerateDepsFile" task failed unexpectedly. [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018: System.TypeInitializationException: The type initializer for 'System.Text.Encodings.Web.DefaultJavaScriptEncoder' threw an exception. ---> System.IO.FileNotFoundException: Could not load file or assembly 'System.Runtime.CompilerServices.Unsafe, Version=6.0.1.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified. [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    at System.ReadOnlySpan`1..ctor(T[] array) [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    at System.Text.Encodings.Web.DefaultJavaScriptEncoder..ctor(TextEncoderSettings settings, Boolean allowMinimalJsonEscaping) [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    at System.Text.Encodings.Web.DefaultJavaScriptEncoder..ctor(TextEncoderSettings settings) [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    at System.Text.Encodings.Web.DefaultJavaScriptEncoder..cctor() [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    --- End of inner exception stack trace --- [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    at System.Text.Encodings.Web.JavaScriptEncoder.get_UnsafeRelaxedJsonEscaping() [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    at Microsoft.Extensions.DependencyModel.DependencyContextWriter.Write(DependencyContext context, Stream stream) [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    at Microsoft.NET.Build.Tasks.GenerateDepsFile.WriteDepsFile(String depsFilePath) in //src/Tasks/Microsoft.NET.Build.Tasks/GenerateDepsFile.cs:line 254 [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    at Microsoft.NET.Build.Tasks.GenerateDepsFile.ExecuteCore() in //src/Tasks/Microsoft.NET.Build.Tasks/GenerateDepsFile.cs:line 297 [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    at Microsoft.NET.Build.Tasks.TaskBase.Execute() in /_/src/Tasks/Common/TaskBase.cs:line 45 [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskExecutionHost.Execute() [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]
C:\h\w\AEF40999\p\d\sdk\10.0.100-ci\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.targets(305,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskBuilder.<ExecuteInstantiatedTask>d__26.MoveNext() [C:\h\w\AEF40999\t\dotnetSdkTests\5cn1nhcq.l20\It_checks_for---BC89084B\Dependency\Dependency.csproj::TargetFramework=netstandard1.5]

@carlossanlop some .NET Framework msbuild tests are failing because of the System.Runtime.CompilerServices.Unsafe/6.1.0 package update. In this case I bet that something is loading the newer CU assembly into a desktop msbuild task host. Can you please take a look?

Looks like this affects all PRs in dotnet/sdk. Presumably a PR with the failures got force merged.

@ViktorHofer ViktorHofer merged commit 3fb7dd4 into main Mar 31, 2025
40 of 43 checks passed
@ViktorHofer ViktorHofer deleted the darc-main-e3b7da1d-a1cb-414b-b610-a49c478868dd branch March 31, 2025 10:07
@ViktorHofer
Copy link
Member

Why are those PRs getting auto-merged with a failing job? That should be fixed asap. In this case it's fine because the regression already happened in #47898. cc @marcpopMSFT - probably BuildAnalysis related

@premun
Copy link
Member

premun commented Mar 31, 2025

@ViktorHofer the dotnet-sdk-public-ci leg is not marked as Required in this repo's settings

@ViktorHofer
Copy link
Member

ViktorHofer commented Mar 31, 2025

Right. But IIRC the "Build Analysis" leg should be marked as required and check whether the dotnet-sdk-public-ci pipeline is green.

@premun
Copy link
Member

premun commented Mar 31, 2025

Seems like BA is neutral until other checks fail:

image

So maybe UB + SB finished green and then it got merged without waiting for SDK CI as BA could not have gone red yet?

@ViktorHofer
Copy link
Member

dotnet/dnceng#3327 is probably the tracking issue for this. cc @missymessa

@marcpopMSFT
Copy link
Member

It was suggested we use BA is the primary check but that doesn't work because of 3327 and we were told not to allow people to override the check on sourcebuild/unified build so we landed with this compromise. I guess in the case where the public PR build is still running but the other builds have completed, we hit 3327 and things get merged.

Should I add the CI leg back and hold off using BA until that issue is resolved?

@ViktorHofer
Copy link
Member

Based on PRs with failing tests getting merged, I would say yes. We need resolution on dotnet/dnceng#3327 and then decide if we can again gate auto-merge on the Build Analysis job. We had to disable auto-merge in runtime for that exact reason.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-CodeFlow untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants