Skip to content

[main] Update dependencies from dotnet/winforms #9260

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 7 commits into from
Jun 19, 2024

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Jun 18, 2024

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:
    • System.Reflection.MetadataLoadContext: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Windows.Extensions: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • Microsoft.NETCore.Platforms: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Resources.Extensions: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • Microsoft.NETCore.App.Ref: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • Microsoft.NETCore.App.Runtime.win-x64: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • VS.Redist.Common.NetCore.SharedFramework.x64.9.0: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • Microsoft.Win32.Registry.AccessControl: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • Microsoft.Win32.SystemEvents: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.CodeDom: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.ComponentModel.Composition: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Configuration.ConfigurationManager: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Data.Odbc: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Data.OleDb: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Diagnostics.EventLog: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Diagnostics.PerformanceCounter: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.DirectoryServices.AccountManagement: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.DirectoryServices.Protocols: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.DirectoryServices: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.IO.Packaging: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.IO.Ports: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Management: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Reflection.Context: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Runtime.Caching: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Security.Cryptography.Pkcs: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Security.Cryptography.ProtectedData: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Security.Cryptography.Xml: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Security.Permissions: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.ServiceModel.Syndication: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.ServiceProcess.ServiceController: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Speech: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Text.Encoding.CodePages: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.Threading.AccessControl: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)
    • System.ComponentModel.Composition.Registration: from 9.0.0-preview.6.24316.2 to 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms)

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

  • Subscription: dd483b3f-65b0-44ab-f62d-08d8e93a0c51
  • Build: 20240618.4
  • Date Produced: June 19, 2024 1:07:30 AM UTC
  • Commit: d768c8b927117995f9cfa2423a8d2d4c6253e080
  • Branch: refs/heads/main
Microsoft Reviewers: Open in CodeFlow

…40618.3

Microsoft.Dotnet.WinForms.ProjectTemplates , Microsoft.Private.Winforms , System.Drawing.Common
 From Version 9.0.0-preview.6.24317.4 -> To Version 9.0.0-preview.6.24318.3

Dependency coherency updates

System.Reflection.MetadataLoadContext,System.Windows.Extensions,Microsoft.NETCore.Platforms,System.Resources.Extensions,Microsoft.NETCore.App.Ref,Microsoft.NETCore.App.Runtime.win-x64,VS.Redist.Common.NetCore.SharedFramework.x64.9.0,Microsoft.Win32.Registry.AccessControl,Microsoft.Win32.SystemEvents,System.CodeDom,System.ComponentModel.Composition,System.Configuration.ConfigurationManager,System.Data.Odbc,System.Data.OleDb,System.Diagnostics.EventLog,System.Diagnostics.PerformanceCounter,System.DirectoryServices.AccountManagement,System.DirectoryServices.Protocols,System.DirectoryServices,System.IO.Packaging,System.IO.Ports,System.Management,System.Reflection.Context,System.Runtime.Caching,System.Security.Cryptography.Pkcs,System.Security.Cryptography.ProtectedData,System.Security.Cryptography.Xml,System.Security.Permissions,System.ServiceModel.Syndication,System.ServiceProcess.ServiceController,System.Speech,System.Text.Encoding.CodePages,System.Threading.AccessControl,System.ComponentModel.Composition.Registration
 From Version 9.0.0-preview.6.24316.2 -> To Version 9.0.0-preview.6.24318.4 (parent: Microsoft.Private.Winforms
@dotnet-maestro dotnet-maestro bot requested a review from a team as a code owner June 18, 2024 22:59
Copy link
Contributor

@dotnet-policy-service dotnet-policy-service bot left a comment

Choose a reason for hiding this comment

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

Go, you big red fire engine!

@dotnet-policy-service dotnet-policy-service bot added the PR metadata: Label to tag PRs, to facilitate with triage label Jun 18, 2024
Copy link
Contributor Author

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

@dotnet/dotnet-winforms-admin

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/winforms) 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

…40618.4

Microsoft.Dotnet.WinForms.ProjectTemplates , Microsoft.Private.Winforms , System.Drawing.Common
 From Version 9.0.0-preview.6.24317.4 -> To Version 9.0.0-preview.6.24318.4

Dependency coherency updates

System.Reflection.MetadataLoadContext,System.Windows.Extensions,Microsoft.NETCore.Platforms,System.Resources.Extensions,Microsoft.NETCore.App.Ref,Microsoft.NETCore.App.Runtime.win-x64,VS.Redist.Common.NetCore.SharedFramework.x64.9.0,Microsoft.Win32.Registry.AccessControl,Microsoft.Win32.SystemEvents,System.CodeDom,System.ComponentModel.Composition,System.Configuration.ConfigurationManager,System.Data.Odbc,System.Data.OleDb,System.Diagnostics.EventLog,System.Diagnostics.PerformanceCounter,System.DirectoryServices.AccountManagement,System.DirectoryServices.Protocols,System.DirectoryServices,System.IO.Packaging,System.IO.Ports,System.Management,System.Reflection.Context,System.Runtime.Caching,System.Security.Cryptography.Pkcs,System.Security.Cryptography.ProtectedData,System.Security.Cryptography.Xml,System.Security.Permissions,System.ServiceModel.Syndication,System.ServiceProcess.ServiceController,System.Speech,System.Text.Encoding.CodePages,System.Threading.AccessControl,System.ComponentModel.Composition.Registration
 From Version 9.0.0-preview.6.24316.2 -> To Version 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms
dotnet-maestro bot and others added 3 commits June 19, 2024 02:00
…40618.4

Microsoft.Dotnet.WinForms.ProjectTemplates , Microsoft.Private.Winforms , System.Drawing.Common
 From Version 9.0.0-preview.6.24317.4 -> To Version 9.0.0-preview.6.24318.4

Dependency coherency updates

System.Reflection.MetadataLoadContext,System.Windows.Extensions,Microsoft.NETCore.Platforms,System.Resources.Extensions,Microsoft.NETCore.App.Ref,Microsoft.NETCore.App.Runtime.win-x64,VS.Redist.Common.NetCore.SharedFramework.x64.9.0,Microsoft.Win32.Registry.AccessControl,Microsoft.Win32.SystemEvents,System.CodeDom,System.ComponentModel.Composition,System.Configuration.ConfigurationManager,System.Data.Odbc,System.Data.OleDb,System.Diagnostics.EventLog,System.Diagnostics.PerformanceCounter,System.DirectoryServices.AccountManagement,System.DirectoryServices.Protocols,System.DirectoryServices,System.IO.Packaging,System.IO.Ports,System.Management,System.Reflection.Context,System.Runtime.Caching,System.Security.Cryptography.Pkcs,System.Security.Cryptography.ProtectedData,System.Security.Cryptography.Xml,System.Security.Permissions,System.ServiceModel.Syndication,System.ServiceProcess.ServiceController,System.Speech,System.Text.Encoding.CodePages,System.Threading.AccessControl,System.ComponentModel.Composition.Registration
 From Version 9.0.0-preview.6.24316.2 -> To Version 9.0.0-preview.6.24318.5 (parent: Microsoft.Private.Winforms
anjali-wpf
anjali-wpf previously approved these changes Jun 19, 2024
@ViktorHofer
Copy link
Member

C:\Program Files\Microsoft Visual Studio\2022\Preview\MSBuild\Current\Bin\Microsoft.Common.CurrentVersion.targets(2401,5): error MSB3277: (NETCORE_ENGINEERING_TELEMETRY=Build) Found conflicts between different versions of "System.Runtime.Serialization.Formatters" that could not be resolved.

@bartonjs

@adamsitnik
Copy link
Member

@MichalStrehovsky attempted to fix it in a9c9b7d

I wonder why the fix did not help (WinForms did sth similar: dotnet/winforms@cf00f74)

What we have right now in the log:

    References which depend on or have been unified to "System.Runtime.Serialization.Formatters, Version=9.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a" [].
        D:\a\_work\1\s\artifacts\bin\DirectWriteForwarder\ARM64\Release\net9.0\DirectWriteForwarder.dll
          Project file item includes which caused reference "D:\a\_work\1\s\artifacts\bin\DirectWriteForwarder\ARM64\Release\net9.0\DirectWriteForwarder.dll".
            D:\a\_work\1\s\artifacts\bin\DirectWriteForwarder\ARM64\Release\net9.0\DirectWriteForwarder.dll
        D:\a\_work\1\s\artifacts\bin\System.Xaml\ARM64\Release\net9.0\System.Xaml.dll
          Project file item includes which caused reference "D:\a\_work\1\s\artifacts\bin\System.Xaml\ARM64\Release\net9.0\System.Xaml.dll".
            D:\a\_work\1\s\artifacts\bin\System.Xaml\ARM64\Release\net9.0\System.Xaml.dll
            D:\a\_work\1\s\artifacts\bin\WindowsBase\ARM64\Release\net9.0\WindowsBase.dll
            D:\a\_work\1\s\artifacts\bin\DirectWriteForwarder\ARM64\Release\net9.0\DirectWriteForwarder.dll
            D:\a\_work\1\s\artifacts\bin\UIAutomationTypes\ARM64\Release\net9.0\UIAutomationTypes.dll
            D:\a\_work\1\s\artifacts\bin\UIAutomationProvider\ARM64\Release\net9.0\UIAutomationProvider.dll

Perhaps this fix should be extended to all the mentioned dlls?

FWIW I've tried to build it locally and failed and DirectWriteForwarder is a managed C++ and I don't know how to fix it.

@ViktorHofer
Copy link
Member

I submitted a fix. WPF's infra makes it impossible for RAR (ResolveAssemblyReference) to decide which assembly is right. Filed #9261

@am11
Copy link
Member

am11 commented Jun 19, 2024

Major kudos to @ViktorHofer for cracking yet another tricky MSBuild related snafu with those magic properties! 🌟 Your expertise saved the day as always!

@dotnet-maestro dotnet-maestro bot merged commit 7478bb1 into main Jun 19, 2024
12 checks passed
@dotnet-maestro dotnet-maestro bot deleted the darc-main-35238ce3-138f-4676-929c-c05f140ca2c6 branch June 19, 2024 09:05
@adamsitnik
Copy link
Member

I submitted a fix. WPF's infra makes it impossible for RAR (ResolveAssemblyReference) to decide which assembly is right.

Thanks a lot, you have most likely saved us from a lot of pain (either a major delay or a revert)

@ViktorHofer
Copy link
Member

Thanks for the kind words @am11 and @adamsitnik. I left a few more details in the tracking issue and explained why wpf's PrivateAssets global setting is problematic. There's a reason why PackageReferences flow transitively by default.

@github-actions github-actions bot locked and limited conversation to collaborators Jul 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR metadata: Label to tag PRs, to facilitate with triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants