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

Provide mechanism to mark packages as PackageReference style only #8566

Open
dasMulli opened this issue Sep 10, 2019 · 2 comments
Open

Provide mechanism to mark packages as PackageReference style only #8566

dasMulli opened this issue Sep 10, 2019 · 2 comments

Comments

@dasMulli
Copy link

Similar to "minimum client version", a nuspec should have a feature to indicate that the package was authored for consumption only for packages using PackageReference.

Currently, package authors have to add targets to create a build error when the package was installed using packages.config.

There should be a path forward for allowing maintainers to drop packages.config support, especially when the supported platforms will only use SDK-style projects (.NET Core, upcoming Xamarin, possibly UWP projects) or PackageReference by default.
This migration path should create an error on package install rather than build to be visible to the user and guide the user to PackageReference migration documentation.

@rrelyea
Copy link
Contributor

rrelyea commented Sep 17, 2019

@nkolev92 - can you noodle on this with your other PR related work this sprint? This would have a downside of not erroring in way old versions. But we should think this through.

@dasMulli
Copy link
Author

This would have a downside of not erroring in way old versions.

I guess this what's minimum client version in the nuspec was for. So it would error on old versions and error for packages.config in new versions.

There are just scenarios that are way complicated to make work in both worlds. contentFiles comes to my mind first. Then ref/runtimes splits etc.

At some point it would be like "built with .NET Core in mind but should also work on UWP/Mono and even .NET if you care upgrading the projects to PackageReference".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants