Skip to content

[main] Update dependencies from dotnet/winforms #9264

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

Conversation

dotnet-maestro[bot]
Copy link
Contributor

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

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

  • Subscription: dd483b3f-65b0-44ab-f62d-08d8e93a0c51
  • Build: 20240619.5
  • Date Produced: June 19, 2024 9:51:18 PM UTC
  • Commit: 77be8127d323f322a9c168acaf5920d84271096c
  • Branch: refs/heads/main
Microsoft Reviewers: Open in CodeFlow

…40619.5

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

Dependency coherency updates

System.Reflection.MetadataLoadContext,System.Windows.Extensions,Microsoft.NETCore.Platforms,System.Resources.Extensions,System.Runtime.Serialization.Formatters,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.24318.5 -> To Version 9.0.0-preview.6.24319.3 (parent: Microsoft.Private.Winforms
@dotnet-maestro dotnet-maestro bot requested a review from a team as a code owner June 19, 2024 22:45
@dotnet-policy-service dotnet-policy-service bot added the PR metadata: Label to tag PRs, to facilitate with triage label Jun 19, 2024
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!

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

…40619.5

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

Dependency coherency updates

System.Reflection.MetadataLoadContext,System.Windows.Extensions,Microsoft.NETCore.Platforms,System.Resources.Extensions,System.Runtime.Serialization.Formatters,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.24318.5 -> To Version 9.0.0-preview.6.24319.3 (parent: Microsoft.Private.Winforms
@dipeshmsft
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ViktorHofer
Copy link
Member

ViktorHofer commented Jun 20, 2024

@dipeshmsft /azp run just reruns the build but doesn't merge latest main into the branch. I just updated the branch.

By the way, please approve the PR.

ViktorHofer
ViktorHofer previously approved these changes Jun 20, 2024
@ViktorHofer
Copy link
Member

@MichalStrehovsky looks like that didn't help. No idea why this didn't fail in the other PR.

@MichalStrehovsky
Copy link
Member

The mismatch still persist. Are the CI machines at https://dev.azure.com/dnceng/internal/_git/dotnet-wpf-int synchronized with the CI machines used in the public CI?

LINK(0,0): error C1047: (NETCORE_ENGINEERING_TELEMETRY=Build) The object or library file 'D:\a\_work\1\s\.packages\runtime.win-arm64.microsoft.dotnet.wpf.dnceng\9.0.0-preview.7.24319.2\runtimes\win-arm64\native\bilinearspan.lib' was created by a different version of the compiler than other objects like 'D:\a\_work\1\s\artifacts\obj\wpfgfx\ARM64\Release\precomp.obj'; rebuild all objects and libraries with the same compiler

@ViktorHofer
Copy link
Member

ViktorHofer commented Jun 20, 2024

wpf-int uses: windows.vs2022preview.amd64 -> Preview 17.11.0-pre.1.0
wpf uses: windows.vs2022preview.amd64.open -> Preview 17.11.0-pre.1.0

according to https://helix.dot.net

@MichalStrehovsky
Copy link
Member

@MichalStrehovsky looks like that didn't help. No idea why this didn't fail in the other PR.

My theory is that it was the last CI run before the CI machines were updated:

image

If the dotnet-wpf-int build machines get updated at the same cadence, we probably just need to trigger a new build. If they're not we're stuck until someone updates them.

@ViktorHofer
Copy link
Member

That's a good guess because the Eng Infra Rollout happened yesterday.

Queued https://dev.azure.com/dnceng/internal/_build/results?buildId=2478108&view=results

dipeshmsft
dipeshmsft previously approved these changes Jun 20, 2024
@ViktorHofer
Copy link
Member

ViktorHofer commented Jun 20, 2024

@MichalStrehovsky while we are waiting on that, is there a way to avoid that tight coupling between VS versions being used in wpf-int and wpf? We hit that issue in the VMR as well a few months ago.

@MichalStrehovsky
Copy link
Member

@MichalStrehovsky while we are waiting on that, is there a way to avoid that tight coupling between VS versions being used in wpf-int and wpf? We hit that issue in the VMR as well a few months ago.

Disabling LTCG (link-time code generation) would probably solve that. It's a bit of a deoptimization, so one would have to measure if it's acceptable. It might.

With LTCG, the object files/libraries don't have machine code, but whatever intermediate bytecode Visual C++ compiler uses. This bytecode is not stable and requires a matching compiler.

@dipeshmsft
Copy link
Member

I think we have disabled LTCG once and later enabled it, once the systems were up to date.

@ViktorHofer
Copy link
Member

Given that this breaks CI whenever VS and the MSVC compiler gets updated I would really want to question the value of that. Especially when it comes to servicing and/or tight schedules. I will file an issue.

@ViktorHofer
Copy link
Member

@dipeshmsft your review got dismissed. Can you please approve once more? Thanks

@dipeshmsft
Copy link
Member

Once the build completes, we can merge this PR and close the newer one right, as its changes are now included here ?

@dotnet-maestro dotnet-maestro bot merged commit 4093a16 into main Jun 20, 2024
12 checks passed
@dotnet-maestro dotnet-maestro bot deleted the darc-main-0e30143d-9131-4a11-af1f-862326597df0 branch June 20, 2024 10:17
@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.

4 participants