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

Use ridless PNSE configurations where applicable #44230

Merged
merged 2 commits into from
Dec 10, 2020

Conversation

ViktorHofer
Copy link
Member

For projects that require platform support we should use a rid-less
configuration, i.e. $(NetCoreAppCurrent). That guarantees that PNSE
assemblies are available for new configurations that aren't represented
in the defined project's TargetFrameworks. This is especially important
for inbox assemblies.

I left the following assemblies as is as they are partial PNSEs:

  • System.Security.Cryptography.Algorithms
  • System.Net.NameResolution

System.Net.Requests and System.Net.WebClient are IL only but
intentionally marked as throwing on Browser.

Fixes #44229

For projects that require platform support we should use a rid-less
configuration, i.e. $(NetCoreAppCurrent). That guarantees that PNSE
assemblies are available for new configurations that aren't represented
in the defined project's TargetFrameworks. This is especially important
for inbox assemblies.

I left the following assemblies as is as they are partial PNSEs:
- System.Security.Cryptography.Algorithms
- System.Net.NameResolution

System.Net.Requests and System.Net.WebClient are IL only but
intentionally marked as throwing on Browser.
@ghost
Copy link

ghost commented Nov 3, 2020

Tagging subscribers to this area: @maryamariyan
See info in area-owners.md if you want to be subscribed.

Copy link
Member

@safern safern left a comment

Choose a reason for hiding this comment

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

LGTM

@safern
Copy link
Member

safern commented Nov 4, 2020

Should we add validation that when the PNSE is set, we check that the tfm is ridless?

@ViktorHofer
Copy link
Member Author

Should we add validation that when the PNSE is set, we check that the tfm is ridless?

This is a bit more complex as we have partial PNSE assemblies now as well with Browser. Also we have PNSE configurations that use a specific implementation ie -Browser but the IL only configuration has the rid-less configuration, i.e. System.Net.WebClient.

@safern
Copy link
Member

safern commented Nov 4, 2020

This is a bit more complex as we have partial PNSE assemblies now as well with Browser. Also we have PNSE configurations that use a specific implementation ie -Browser but the IL only configuration has the rid-less configuration, i.e. System.Net.WebClient.

That is true... disregard.

@ghost
Copy link

ghost commented Nov 12, 2020

Tagging subscribers to this area: @safern, @ViktorHofer
See info in area-owners.md if you want to be subscribed.


Issue meta data
Issue content:
For projects that require platform support we should use a rid-less

configuration, i.e. $(NetCoreAppCurrent). That guarantees that PNSE
assemblies are available for new configurations that aren't represented
in the defined project's TargetFrameworks. This is especially important
for inbox assemblies.

I left the following assemblies as is as they are partial PNSEs:

  • System.Security.Cryptography.Algorithms
  • System.Net.NameResolution

System.Net.Requests and System.Net.WebClient are IL only but
intentionally marked as throwing on Browser.

Fixes #44229

</td>
Issue author: ViktorHofer
Assignees: ViktorHofer
Labels:
`area-Infrastructure-libraries`

</td>
Milestone: -

@ViktorHofer
Copy link
Member Author

/azp run runtime

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@marek-safar
Copy link
Contributor

Is there anything blocking the merge?

@dotnet dotnet deleted a comment from azure-pipelines bot Dec 10, 2020
@ViktorHofer ViktorHofer merged commit a03ab43 into dotnet:master Dec 10, 2020
@ViktorHofer ViktorHofer deleted the PNSEconfigurations branch December 10, 2020 20:05
@ghost ghost locked as resolved and limited conversation to collaborators Jan 9, 2021
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.

Use rid-less configurations to produce PNSE assemblies in projects that require platform support
7 participants