-
Notifications
You must be signed in to change notification settings - Fork 525
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
Suggestion: Skip download of dependencies #1381
Comments
I think this should all go into real nuget meta data. That said: maybe we can still be smart here and special case fake inside of
|
Yeah that would probably work as well for most users. Thinking about the dependencies of FAKE they probably need to be "==" because even if paket could solve a ">=" or "<=" constraint, downloading it wouldn't really help (because the version is locked/bundled in FAKE). |
I think netcore and FAKE 5 will solve this |
Follow up of fsprojects/FSharp.Formatting#337.
I think the underlaying problem is that paket doesn't know that FAKE depends on FSharp.Compiler.Service. If it would, it could resolve compatible versions and the problem would no longer appear in the future.
On the other hand I can see that we don't want to download all of FAKEs dependencies, because they are already bundled (which is good)....
Would it make sense to either have a paket feature to specify 'light' dependencies on a nuget package (ie dependencies which are not downloaded on restore), possibly even outside of the regular nuget dependencies, for example as an additional file bundled in the nuget package?
Or maybe even easier: Specify the dependencies as usual in the nuspec and add a feature for paket to not download dependencies (if they are not required by other packages in the group). Something like
Another idea would be to have this flag but packages can enable it by themself (for example by adding a special file to the nupkg or via nuspec metadata?), this would enable backwards compat with exisiting
paket.dependencies
.What do you think?
The text was updated successfully, but these errors were encountered: