:GoInstall error navigation seems broken

[daaku]

Haven't figure out what exactly is happening, but this was working fine until I upgraded yesterday. Steps to reproduce:

1. Start vim in a say $HOME
2. Open in go file say $HOME/go/src/foo/foo.go
3. :GoInstall foo/..
4. If there is a error from the GoInstall in $HOME/go/src/bar/bar.go you'll see it listed but won't be opened as it used to be.

[fatih]

Hi @daaku. Weird thing indeed. The thing is there is no code that changed :GoInstall since the previous release. If there is any error, :GoInstall calls cwindow which opens the quick fix list and shows all errors. It doesn't jump to it though, you need to call :cc. In your case, you want to jump to it, just like :GoBuild right ?

[daaku]

It's getting the absolute paths wrong, :cc doesn't work either. :GoBuild does in fact get them right.

[fatih]

It's so hard to test this without a proper test files. I've tried in my current GOPATH with several other packages, and in all of them it was working. Is there a way you could paste me a self contained go code so I could directly reproduce it ?

[fatih]

I also wonder if this is due: 0d1e8c9 But again there is no way I can test it. Pinging @mattn may be he has some idea. I couldn't reproduce it (waiting for your example files)

[mattn]

I'll look into it later

[fatih]

@daaku can you test with #405 please ?

[daaku]

I pulled but it still fails for me :/ Here's a repro case:

# setup a test case
export GOPATH=$PWD/tmp
mkdir -p $GOPATH/src/foo
mkdir -p $GOPATH/src/bar
cat << 'EOF' > $GOPATH/src/foo/foo.go
package foo

import "fmt"

func Do() {
  fmt.Println("hello from foo")
}
EOF
cat << 'EOF' > $GOPATH/src/bar/bar.go
package main

import (
  "fmt"
  "foo"
)

func main() {
  foo.Do()
  fmt.Println("hello from bar")
}
EOF
go install -v ...
$GOPATH/bin/bar

# now break the code so go install fails
cat << 'EOF' > $GOPATH/src/foo/foo.go
package foo

func Do() {
  fmt.Println("hello from foo")
}
EOF
go install -v ...

# open this and run :GoInstall
vim $GOPATH/src/bar/bar.go

[fatih]

Thanks @daaku. Will look at it in the morning.

[fatih]

Thanks @daaku I can now reproduce it. I've fixed it with #410. Can you please try it? Now it successfully parses the error location and put's into the quickfix list. Note that :GoInstall doesn't have an autojump to error like the others (just noticed it). So you need to call manually :cc. That is also fixed with the PR: #411. Please let me know how it works for you, I've really tried to cover all cases this time.

[daaku]

Really appreciate the effort @fatih! The path logic still isn't working -- it's getting the correct filename, but not the directory.

[fatih]

It will show the file name but if you call :cc you should go the correct definition. Do you mean it should show the full path rather the file name? Or do I miss something?