-
Notifications
You must be signed in to change notification settings - Fork 258
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
netcore projects don't support setting as developmentDependency #4694
Comments
Set See also: #4125 |
Not what I'm talking about -- I'm talking about creating a library/package that has developmentDependency set to true in its metadata. That other issue is about consuming PackageRefs and turning those into dependencies or not. |
Oh I see, to write this into the nuspec for a new package 😄 That doesn't exist currently since the developmentDependency flag is not supported for PackageReference yet, this would be good to fix along with #4125 //cc @rohit21agrawal |
This is only useful if we do #4125 (which we should) |
@davidfowl I agree we should fix #4125, but fixing this is absolutely useful even without that. I can't build packages that are consumed downlevel without this feature. I'm trying to migrate my nerdbank.gitversioning package over to your new pack targets and I am blocked because if I start releasing packages that are missing this metadata, everyone using packages.config will regress when they use my latest package. |
In trying to workaround the problem by using the |
+1 to @AArnott's comments. We lose a lot of MSBuild magic by "dropping down" to nuspec. Using |
This needs to be reopened... this doesn't actually work in either 4.3 or the 2.0 preview 1 tooling... the MSBuild.Build.Tasks.Pack.Targets don't have any mention of |
your nuget.build.tasks.pack.dll version please? |
In the tools preview 1 SDK, it's 4.3.0.2418 |
The current Pack targets don't seem to have a way to specify
developmentDependency
for the generated NuSpec. This leaves a gap when creating packages intended to be used at build-time only.The text was updated successfully, but these errors were encountered: