dep silently fails if no .go files are present

[lavalamp]

In an empty dir in an empty GOPATH, I ran dep ensure, and it made a lock and a manifest file, but did not actually download any dependencies. I added a .go file and it started behaving as expected.

[sdboyer]

Yeah, it's probably worth adding a check to make sure there's at least one valid dir/pkg of go files, and error out if not.

[lavalamp]

Having it work anyway would be fine by me, too ;)

[schmaluk]

Maybe Im doing sthg wrong. But I dont see that dep downloads anything to the vendor-folder at all although there is sthg in the manifest. Can you please explain the meaning of lock.json? Thanks

[sdboyer]

@lavalamp i think probably better to error, as demonstrated by @mlukasch's confusion here.

@mlukasch unlike tools in other languages, dependencies are only required if they actually appear in the import graph (or if they appear in the requires section of the manifest). The dependencies section in the manifest merely specifies constraints on projects IF they are required.

[sdboyer]

This should be a pretty straightforward fix, if anyone wants to tackle it :)

I would like to tackle this! :)

[sdboyer]

@mylk13 awesome! 🎉🎉

The most important direction I can give here is that you'll be looking for the results of a call to gps.ListPackages() - a gps.PackageTree. If the PackageTree.Packages map is empty, then it means no packages were found.

Of course, the map also might have some elements, but gps.PackageOrErr is basically an enum, so if all of the members just have errs, then it's basically equivalent to having no packages. Probably. You can start poking and get a feel for how it works, then we can discuss in the PR the best handling for these cases.