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

Drop netstandard for $(NetCoreTFM) #749

Merged
merged 1 commit into from
May 14, 2024
Merged

Conversation

jimmylewis
Copy link
Contributor

Netstandard gives compatibility benefits for libraries to be used against netfx or netcore frameworks. However, for libman, all of our projects that run on netcore already target that framework directly.

This change modifies the existing multitargeted projects to target netcore directly instead of netstandard. This will allow future changes to use conditional compilation to leverage newer APIs in netcore that do not exist in the netstandard API surface.

The only breaking change here is that the Build project now uses .NET 8 for the build task. This means consumers of that package who are not using .NET Framework will need to have a compatible version of the SDK; however, this aligns with the prior change in the CLI tool to require .NET 8 as well.

@jimmylewis
Copy link
Contributor Author

jimmylewis commented May 2, 2024

Note that this change led to a bunch of runtime package dependency changes (which will likely run afoul of #728) and several dependency conflicts (which are addressed via #747). Hence I've created this PR as a draft until the other can be merged first, and it will also need someone to populate the package feed for the new runtime packages.

Netstandard gives compatibility benefits for libraries to be used
against netfx or netcore frameworks.  However, for libman, all of our
projects that run on netcore already target that framework directly.

This change modifies the existing multitargeted projects to target
netcore directly instead of netstandard.  This will allow future changes
to use conditional compilation to leverage newer APIs in netcore that do
not exist in the netstandard API surface.

The only breaking change here is that the Build project now uses .NET 8
for the build task.  This means consumers of that package who are not
using .NET Framework will need to have a compatible version of the SDK;
however, this aligns with the prior change in the CLI tool to require
.NET 8 as well.
@jimmylewis jimmylewis marked this pull request as ready for review May 11, 2024 07:12
@phil-allen-msft phil-allen-msft merged commit 5f16885 into aspnet:main May 14, 2024
2 checks passed
@jimmylewis jimmylewis deleted the netcore branch May 21, 2024 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants