Closed dmitshur closed 8 years ago
I'm not sure about the performance implications of removing this optimization. Removing it altogether would require more thorough investigation.
For now, I'll fix this by adding the grpc special case handling. It should still be an issue to resolve this more generally in the long term.
See https://github.com/sourcegraph/srclib-go/blob/4c327dd481924fe3bdf3e9d71604d9b59eab28cb/depresolve/depresolve.go#L36-L40:
That is not valid for the following import path:
google.golang.org/grpc
In reality, it maps to:
https://github.com/grpc/grpc-go
But
depresolve.ResolveImportPath
incorrectly maps it to https://github.com/golang/grpc.