https://go-review.googlesource.com/#/c/17728/

excited about this feature landing :) (trying to move a bunch of code to subpackages)

just mirroring here in case people don't look at the gerrit review, here's a patch on top of that review that gets goimports and vim-go working together again https://gist.github.com/termie/34326bd29e89565c95fa

Thanks for the patch @termie it works fine for me.

This may be a duplicate of #10339.

@pwaller not quite. #10339 predates the vendoring support in the Go tool; it is about choosing "github.com/a/b/vendor/github.com/c/d" over "github.com/c/d". I would say that #10339 as written is obviated by GO15VENDOREXPERIMENT and this issue.

Once CL 17728 is merged both this issue and #10339 should probably be closed.

I think this is fixed by https://go-review.googlesource.com/#/c/17728/

Regarding this issue I think I found another code path that wasn't considering packages inside /vendor and created a new CL fixing it: https://go-review.googlesource.com/#/c/22020/

The problem happens when the vendored package name is different from the base import path. goimports calls build.Import() to figure out the real package name, however, it was being called with an empty srcDir parameter this way it couldn't find packages inside /vendor.

@cezarsa what else need to be done to have your patch merged?

Please, any news on this ?

This was fixed some time ago.