Skip to content
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

[main] Update dependencies from dotnet/roslyn #46546

Merged
merged 14 commits into from
Feb 13, 2025

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Feb 5, 2025

This pull request updates the following dependencies

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

  • Subscription: d5dc2a0a-0871-47fc-022d-08dc4502ccaa
  • Build: 20250212.6
  • Date Produced: February 12, 2025 11:36:33 PM UTC
  • Commit: 36036f8f7458b5da58207b579e326608efc86bdf
  • Branch: refs/heads/main

…204.11

Microsoft.SourceBuild.Intermediate.roslyn , Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.CodeAnalysis.CSharp.CodeStyle , Microsoft.CodeAnalysis.CSharp.Features , Microsoft.CodeAnalysis.CSharp.Workspaces , Microsoft.CodeAnalysis.Workspaces.Common , Microsoft.CodeAnalysis.Workspaces.MSBuild , Microsoft.Net.Compilers.Toolset , Microsoft.Net.Compilers.Toolset.Framework
 From Version 4.14.0-1.25070.2 -> To Version 4.14.0-2.25104.11
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-CodeFlow untriaged Request triage from a team member labels Feb 5, 2025
Copy link
Contributor Author

dotnet-maestro bot commented Feb 5, 2025

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

@dotnet/roslyn-infrastructure-current-swat

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

@akoeplinger
Copy link
Member

darc lost track of old PRs so it opened this new one, this is currently blocked as explained in #46195

…205.10

Microsoft.SourceBuild.Intermediate.roslyn , Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.CodeAnalysis.CSharp.CodeStyle , Microsoft.CodeAnalysis.CSharp.Features , Microsoft.CodeAnalysis.CSharp.Workspaces , Microsoft.CodeAnalysis.Workspaces.Common , Microsoft.CodeAnalysis.Workspaces.MSBuild , Microsoft.Net.Compilers.Toolset , Microsoft.Net.Compilers.Toolset.Framework
 From Version 4.14.0-1.25070.2 -> To Version 4.14.0-2.25105.10
…206.12

Microsoft.SourceBuild.Intermediate.roslyn , Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.CodeAnalysis.CSharp.CodeStyle , Microsoft.CodeAnalysis.CSharp.Features , Microsoft.CodeAnalysis.CSharp.Workspaces , Microsoft.CodeAnalysis.Workspaces.Common , Microsoft.CodeAnalysis.Workspaces.MSBuild , Microsoft.Net.Compilers.Toolset , Microsoft.Net.Compilers.Toolset.Framework
 From Version 4.14.0-1.25070.2 -> To Version 4.14.0-2.25106.12
…207.8

Microsoft.SourceBuild.Intermediate.roslyn , Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.CodeAnalysis.CSharp.CodeStyle , Microsoft.CodeAnalysis.CSharp.Features , Microsoft.CodeAnalysis.CSharp.Workspaces , Microsoft.CodeAnalysis.Workspaces.Common , Microsoft.CodeAnalysis.Workspaces.MSBuild , Microsoft.Net.Compilers.Toolset , Microsoft.Net.Compilers.Toolset.Framework
 From Version 4.14.0-1.25070.2 -> To Version 4.14.0-2.25107.8
dotnet-maestro bot and others added 2 commits February 9, 2025 05:01
…208.3

Microsoft.SourceBuild.Intermediate.roslyn , Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.CodeAnalysis.CSharp.CodeStyle , Microsoft.CodeAnalysis.CSharp.Features , Microsoft.CodeAnalysis.CSharp.Workspaces , Microsoft.CodeAnalysis.Workspaces.Common , Microsoft.CodeAnalysis.Workspaces.MSBuild , Microsoft.Net.Compilers.Toolset , Microsoft.Net.Compilers.Toolset.Framework
 From Version 4.14.0-1.25070.2 -> To Version 4.14.0-2.25108.3
@ViktorHofer
Copy link
Member

@jjonescz @jaredpar what is the right team to tag here? The dotnet/roslyn GH team has 64 members so I didn't cc that. See build failures:

src/roslyn/src/Features/ExternalAccess/Copilot/Analyzer/CopilotUtilities.cs(25,43): error CS0433: (NETCORE_ENGINEERING_TELEMETRY=Build) The type 'NotNullWhenAttribute' exists in both 'Microsoft.CodeAnalysis.Workspaces, Version=4.14.0.0, Culture=neutral, PublicKeyToken=31bf3856ad364e35' and 'System.Runtime, Version=8.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'

@jjonescz
Copy link
Member

jjonescz commented Feb 10, 2025

what is the right team to tag here?

if you mean in general instead of @dotnet/roslyn-infrastructure-current-swat which was removed, that should be probably replaced with @dotnet/roslyn-infrastructure

if you mean roslyn/src/Features specifically, that's owned by @dotnet/roslyn-ide

@333fred
Copy link
Member

333fred commented Feb 10, 2025

@dotnet/product-construction @jaredpar @genlu, this was broken by dotnet/roslyn@f3ef599. Previously, the Copilot EA was in the EditorFeatures layer, which meant that it was excluded from source-build. Now, it's in the Features layer, which is not excluded from source build, which has the following impact:

  • NetRoslynSourceBuild is set to net10.0. This means that Workspaces is built for both net10.0 and netstandard2.0.
  • The Copilot EA has a TFM of $(NetVSShared);net472, which evaluates to net8.0;net472. So it picks up the netstandard2.0 version of Workspaces, which defines an internal copy of NotNullWhenAttribute.
  • We build Copilot for net8.0, and since it has IVT to Workspaces, it sees both the copy from the framework and from Workspaces, resulting in this error.

dotnet/roslyn#77136 will turn off building this EA assembly in source-build, since we shouldn't be doing that in the first place.

nagilson and others added 4 commits February 10, 2025 14:03
…210.8

Microsoft.SourceBuild.Intermediate.roslyn , Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.CodeAnalysis.CSharp.CodeStyle , Microsoft.CodeAnalysis.CSharp.Features , Microsoft.CodeAnalysis.CSharp.Workspaces , Microsoft.CodeAnalysis.Workspaces.Common , Microsoft.CodeAnalysis.Workspaces.MSBuild , Microsoft.Net.Compilers.Toolset , Microsoft.Net.Compilers.Toolset.Framework
 From Version 4.14.0-1.25070.2 -> To Version 4.14.0-2.25110.8
@ViktorHofer
Copy link
Member

So, the remaining failure now is IDE0060 again as in #46353 (comment)

@sharwell please reply to @jaredpar's question. Do we have a tracking issue? @DustinCampbell shared repro instructions which should help. Please prioritize this as roslyn is already more than 500 commits old in main.

@nagilson nagilson added Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch and removed untriaged Request triage from a team member labels Feb 11, 2025
@nagilson nagilson added Area-Roslyn and removed Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch labels Feb 11, 2025
@dotnet dotnet deleted a comment from dotnet-policy-service bot Feb 11, 2025
nagilson and others added 2 commits February 12, 2025 15:20
…212.6

Microsoft.SourceBuild.Intermediate.roslyn , Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.CodeAnalysis.CSharp.CodeStyle , Microsoft.CodeAnalysis.CSharp.Features , Microsoft.CodeAnalysis.CSharp.Workspaces , Microsoft.CodeAnalysis.Workspaces.Common , Microsoft.CodeAnalysis.Workspaces.MSBuild , Microsoft.Net.Compilers.Toolset , Microsoft.Net.Compilers.Toolset.Framework
 From Version 4.14.0-1.25070.2 -> To Version 4.14.0-2.25112.6
@carlossanlop
Copy link
Member

There are unknown failures in VMR with the message Failed to synchronize the VMR. Who's the best person to look into them?

@ViktorHofer
Copy link
Member

ViktorHofer commented Feb 13, 2025

FYI @sharwell and @CyrusNajmabadi are investigating the remaining IDE0060 issue offline. We have a Teams thread about this.

@MichaelSimons
Copy link
Member

There are unknown failures in VMR with the message Failed to synchronize the VMR. Who's the best person to look into them?

There was a VMR patch that was backported and flowing in. I removed the patch to address the problem.

@dotnet/product-construction is the appropriate team to ping on issues like this.

@nagilson nagilson enabled auto-merge February 13, 2025 18:19
@nagilson nagilson merged commit 2e6698a into main Feb 13, 2025
43 checks passed
@nagilson nagilson deleted the darc-main-12be1bc0-f5c1-4bfb-b9a7-dad09a5f97f8 branch February 13, 2025 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants