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 #97152

Merged
merged 9 commits into from
Jan 23, 2024

Conversation

dotnet-maestro[bot]
Copy link
Contributor

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

This pull request updates the following dependencies

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

  • Subscription: 037eb78c-9902-4cb9-856d-08db8ed7cafd
  • Build: 20240119.13
  • Date Produced: January 20, 2024 5:45:12 AM UTC
  • Commit: 2fe96bca1092f880e91eea6eb17ea3487d89309a
  • Branch: refs/heads/main

…117.18

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.9.0-3.24067.18
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-codeflow for labeling automated codeflow label Jan 18, 2024
@ViktorHofer
Copy link
Member

@dotnet/source-build-internal can you please take a look at the prebuilts coming in with this new Roslyn update?

@mthalman
Copy link
Member

Prebuilts output:

.packages/microsoft.dotnet.arcade.sdk/9.0.0-beta.23607.2/tools/SourceBuild/AfterSourceBuild.proj(68,5): error : (NETCORE_ENGINEERING_TELEMETRY=AfterSourceBuild) 9 new pre-builts discovered! Detailed usage report can be found at /__w/1/s/artifacts/source-build/self/prebuilt-report/baseline-comparison.xml.
See https://aka.ms/dotnet/prebuilts for guidance on what pre-builts are and how to eliminate them.
Package IDs are:
System.Composition.8.0.0
System.Composition.AttributedModel.8.0.0
System.Composition.Convention.8.0.0
System.Composition.Hosting.8.0.0
System.Composition.Runtime.8.0.0
System.Composition.TypedParts.8.0.0
System.IO.Pipelines.8.0.0
System.Text.Encoding.CodePages.8.0.0
System.Threading.Channels.8.0.0

Taking a look.

@mthalman
Copy link
Member

This is just caused by Roslyn upgrading to net8.0 packages.

Example from project.assets.json file:

Previously:

      "Microsoft.CodeAnalysis.Workspaces.Common/4.9.0-3.24054.13": {
        "type": "package",
        "dependencies": {
          "Humanizer.Core": "2.14.1",
          "Microsoft.Bcl.AsyncInterfaces": "7.0.0",
          "Microsoft.CodeAnalysis.Common": "[4.9.0-3.24054.13]",
          "System.Composition": "7.0.0",
          "System.IO.Pipelines": "7.0.0",
          "System.Threading.Channels": "7.0.0"
        },

Now:

"Microsoft.CodeAnalysis.Workspaces.Common/4.9.0-3.24067.18": {
        "type": "package",
        "dependencies": {
          "Humanizer.Core": "2.14.1",
          "Microsoft.Bcl.AsyncInterfaces": "8.0.0",
          "Microsoft.CodeAnalysis.Common": "[4.9.0-3.24067.18]",
          "System.Composition": "8.0.0",
          "System.IO.Pipelines": "8.0.0",
          "System.Threading.Channels": "8.0.0"
        },

It can be resolved by adding the reported prebuilts to SBRP.

…117.18

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.9.0-3.24067.18
…119.10

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.9.0-3.24069.10
…119.10

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.9.0-3.24069.10
ViktorHofer added a commit to dotnet/source-build-reference-packages that referenced this pull request Jan 22, 2024
... necessary to consume new Roslyn dependencies. Roslyn recently updated
to 8.0.0 dependencies which makes this change necessary.

Unblocks dotnet/runtime#97152
@ViktorHofer
Copy link
Member

…119.10

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.9.0-3.24069.10
ViktorHofer added a commit to dotnet/source-build-reference-packages that referenced this pull request Jan 22, 2024
* Add Microsoft.CodeAnalysis/4.10 dependencies

... necessary to consume new Roslyn dependencies. Roslyn recently updated
to 8.0.0 dependencies which makes this change necessary.

Unblocks dotnet/runtime#97152

* Undo incorrect GenAPI changes

* Fix order in Build.props
…119.13

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.10.0-1.24069.13
@ViktorHofer ViktorHofer merged commit 3f7ffb5 into main Jan 23, 2024
186 of 191 checks passed
@ViktorHofer ViktorHofer deleted the darc-main-99eefebb-2987-437f-b7d6-aa5b3a7a4c8c branch January 23, 2024 07:05
tmds pushed a commit to tmds/runtime that referenced this pull request Jan 23, 2024
* Update dependencies from https://github.com/dotnet/roslyn build 20240117.18

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.9.0-3.24067.18

* Update dependencies from https://github.com/dotnet/roslyn build 20240117.18

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.9.0-3.24067.18

* Update dependencies from https://github.com/dotnet/roslyn build 20240119.10

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.9.0-3.24069.10

* Update dependencies from https://github.com/dotnet/roslyn build 20240119.10

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.9.0-3.24069.10

* Update dependencies from https://github.com/dotnet/roslyn build 20240119.10

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.9.0-3.24069.10

* Update SBRP

* Update SBRP again

* Update SDK as well

* Update dependencies from https://github.com/dotnet/roslyn build 20240119.13

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.9.0-3.24054.13 -> To Version 4.10.0-1.24069.13

---------

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
@github-actions github-actions bot locked and limited conversation to collaborators Mar 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-codeflow for labeling automated codeflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants