search

sourcegraph / go-langserver

Go language server to add Go support to editors and other tools that use the Language Server Protocol (LSP)
https://sourcegraph.com
MIT License
1.17k stars 89 forks source link

TestIntegration_FileSystem_Lint fails on windows #328

Open keegancsmith opened 6 years ago

keegancsmith commented 6 years ago

https://ci.appveyor.com/project/sourcegraph/go-langserver/builds/19433013#L40

2018/10/11 13:44:23 warning: lint command stderr: "warning: \"/C:/Users/appveyor/AppData/Local/Temp/1/langserver-go-integration705672984/gopath/src/test/p/...\" matched no packages\n"
--- FAIL: TestIntegration_FileSystem_Lint (10.07s)
    integration_test.go:518: Timeout while waiting for notification of type *lsp.PublishDiagnosticsParams
2018/10/11 13:45:37 refs.DefInfo: not a package-level definition (ident: x, object: var x map[string]test/pkg/a.A) and unable to follow type (type: map[string]test/pkg/a.A)
ianlopshire commented 6 years ago

I wonder if this is a result of changing the pattern passed to golint to include rootFSPath (see #321).

I'm also not familiar with windows pathing, but it looks like the path is prefixed with an extra /.

keegancsmith commented 6 years ago

Yeah windows is mysterious to me and I don’t have an env for it to iterate. I did quite a few attempts via appveyor to work it out, but gave up.

ianlopshire commented 6 years ago

I think I can borrow a machine to look into this with. If I can find a solution I will submit a PR. I’ll provide an updated either way.

keegancsmith commented 6 years ago

Thank you!!