-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add support for multiple $GOPATH:s #33
Comments
Ah yes, I totally forgot you could even have multiple GOPATHs without switching. This is definitely something Goggles should support. PS thanks for the detailed issue, awesome! |
@KyleBanks Thanks for the very quick response time! I tried it out, and it seems to be partially working. Now I can search my list of packages, but eventually, something seems to be searching in only one of the GOPATH:s and goggles exists with the following error:
Note. github.com/decomp/decomp/cmd/ll2dot is indeed present in my GOPATH, however not in /home/u/goget, but rather in /home/u/Desktop/go. |
I'll try to see if the issue is resolved in rev 1600139. |
Now it's working! Thanks for the brilliant response time. |
Awesome! |
I tried to use goggles today, and it does look great. However, upon doing so, it seems it encountered an error when trying to parse my $GOPATH, which is a list of GOPATH:s.
When GOPATH is set to a list, goggles fails with the following error:
Here are a couple of references that may be helpful for dealing with multiple $GOPATH:s.
https://golang.org/pkg/path/filepath/#SplitList
https://golang.org/pkg/go/build/#Context.SrcDirs
From mewkiz/pkg/goutil.SrcDir:
The text was updated successfully, but these errors were encountered: