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

Correctly handle assembly qualified names in generic parameters #1546

Merged

Conversation

vitek-karas
Copy link
Member

We always assumed that the generic parameters would come from the same assembly as the generic type itself.
Adds a new test to validate the scenario.

Fixes the primary issue in #1536
The other parts to make that scenario work fully are tracked by different issues.

@vitek-karas vitek-karas added this to the .NET 6.0 milestone Oct 7, 2020
@vitek-karas vitek-karas requested a review from mateoatr October 7, 2020 15:18
@vitek-karas vitek-karas self-assigned this Oct 7, 2020
@akoeplinger
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@mateoatr mateoatr left a comment

Choose a reason for hiding this comment

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

LGTM. We should add the test for invalid assemblies.

We always assumed that the generic parameters would come from the same assembly as the generic type itself.
Adds a new test to validate the scenario.
@vitek-karas vitek-karas force-pushed the HandleAssemblyQualifiedNamesInGenerics branch from 927c0c1 to c7a62ed Compare October 7, 2020 18:01
@vitek-karas vitek-karas merged commit 82ae21e into dotnet:master Oct 7, 2020
@vitek-karas vitek-karas deleted the HandleAssemblyQualifiedNamesInGenerics branch October 9, 2020 17:35
mateoatr pushed a commit to mateoatr/linker that referenced this pull request Nov 13, 2020
…et#1546)

We always assumed that the generic parameters would come from the same assembly as the generic type itself.
Adds a new test to validate the scenario.
agocke pushed a commit to dotnet/runtime that referenced this pull request Nov 16, 2022
…et/linker#1546)

We always assumed that the generic parameters would come from the same assembly as the generic type itself.
Adds a new test to validate the scenario.


Commit migrated from dotnet/linker@82ae21e
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.

4 participants