Skip to content

Conversation

@ViktorHofer
Copy link
Member

@ViktorHofer ViktorHofer commented Sep 5, 2022

Unblocks #75058

The listed PR brings in a change in roslyn that errors for duplicate analyzer references. There were only two duplicates in the libraries source projects which are being removed. The actual reference is already brought in via generators.targets.

Long term for inbox projects, we should agree on either explicitly referencing analyzers (what is currently being done for the JSImportGenerator) or automatically referencing them based on other inputs (currently, LibraryImportGenerator). Mixing both modes caused this issue but also makes it hard to reason about which analyzers run.

Unblocks #75058

The listed PR brings in a change in roslyn that errors for duplicate analyzer references. There were only two duplicates in the libraries source projects which are being removed. The actual reference is already brought in via generators.targets.
Copy link
Member

@stephentoub stephentoub left a comment

Choose a reason for hiding this comment

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

Thanks!

@ghost
Copy link

ghost commented Sep 5, 2022

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

Issue Details

Unblocks #75058

The listed PR brings in a change in roslyn that errors for duplicate analyzer references. There were only two duplicates in the libraries source projects which are being removed. The actual reference is already brought in via generators.targets.

Author: ViktorHofer
Assignees: ViktorHofer
Labels:

area-Infrastructure-libraries

Milestone: -

@ViktorHofer
Copy link
Member Author

Failures are Linux arm64 process terminations, unrelated.

@ViktorHofer ViktorHofer merged commit 60be1de into main Sep 5, 2022
@ViktorHofer ViktorHofer deleted the ViktorHofer-patch-1 branch September 5, 2022 17:49
@cston
Copy link
Contributor

cston commented Sep 6, 2022

Thanks @ViktorHofer.

It looks like there may be another duplicate reference to Microsoft.Interop.SourceGeneration.csproj in System.Net.WebSockets.Client.csproj.

Could we port this change to release/7.0 as well, to unblock updating the Roslyn build in that branch? Thanks.

@cston
Copy link
Contributor

cston commented Sep 6, 2022

/backport to release/7.0

@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2022

Started backporting to release/7.0: https://github.com/dotnet/runtime/actions/runs/3001098463

@cston
Copy link
Contributor

cston commented Sep 6, 2022

I've started a backport to release/7.0 for this change, and removed the extra duplicate reference in #75058.

@ghost ghost locked as resolved and limited conversation to collaborators Oct 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants