zastai / nuget-server Goto Github PK

Some servers, like Nexus, don't seem to support enough of the NuGet protocol for the -SkipDuplicates to work.
Instead, pushes of an already-pushed package result in 400 Bad Request.

Perhaps -SkipDuplicates could be extended to allow an argument, Yes, No or Check, with Yes being the default if no argument is given.
If Check is specified, the import utility will do a does-package-exist check itself before pushing.

With that in place, an import could simply be re-run to resume an interrupted import, without getting spurious errors for the portion that was already completed.

Currently, this option determines which target frameworks used by dependencies will be taken into account.

However, these should be treated as "restore context" targets instead.

Specifically, if net462 is specified as dependency framework:

• for direct dependencies of an imported package:
  • if it has dependencies for net40 and net462, net462 should be used
  • if it has dependencies for net40 only, net40 should be used
• for transitive dependencies: only the target for the direct dependency should apply
  • if A has a dependency B which has a net40 dependency group containing C, then the dependency processing for C
    should use net40
    • however, if C also appears elsewhere in the dependencies for A, then perhaps the "best" common match is needed

It seems likely that this dependency graph handling is already present in the NuGet assemblies (if possibly forcibly oriented around projects); if possible, that should be reused.