-
Notifications
You must be signed in to change notification settings - Fork 17.7k
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
x/tools/cmd/gopls: gopls doesn't work #30027
Comments
This is part of stacktrace.
pkg might be nil. |
/cc @stamblerre I think this is the same bug I reported on slack. Here are the salient details reproduced below.
So it looks like an issue with path escaping. My surprise comes that surely this must be an issue for anyone using modules, since modules result in source files which have |
Change https://golang.org/cl/161077 mentions this issue: |
This change makes sure to handle escaped characters, for instance, files in the module cache contain "@" in the path. Also, return errors when a package is not found for a file. Updates golang/go#30027 Change-Id: I6c4f6d0f30dde55701308b89785f02b671e81cc6 Reviewed-on: https://go-review.googlesource.com/c/161077 Reviewed-by: Ian Cottrell <iancottrell@google.com> Reviewed-by: Yasuhiro MATSUMOTO <mattn.jp@gmail.com>
This case should be fixed now - @skanehira, can you confirm? |
@stamblerre gopls log Output[Trace - 9:45:55 AM] Sending request 'initialize - (1)'. Params: {"rootUri":"file:///Users/skanehira/dev/go/src/github.com/skanehira/docui","initializationOptions":{},"capabilities":{"workspace":{"applyEdit ":true}},"rootPath":"/Users/skanehira/dev/go/src/github.com/skanehira/docui","trace":"off"} |
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
What did you expect to see?
What did you see instead?
gopls log Output
vim-lsp log Output
The text was updated successfully, but these errors were encountered: