Support Go 1.5 `vendor/` experiment. about godef HOT 15 CLOSED

It works when an absolute path name is used for the go file.
Perhaps godef should make the file name absolute before considering it.

from godef.

@pwaller yes, master branch has fixed this issue.

from godef.

Fixed by #33. Sorry for the long wait, guys :)

from godef.

@douglarek fork is always the last option...

@rogpeppe , maybe it'd be better if you could elect some Collaborator to help you manage this repo (e.g. maintain, review the PRs, etc.) if you're too busy.
thanks

from godef.

@rogpeppe the example in #13 (comment) still doesn't seem to work.

from godef.

As far as I can tell, godef does not support vendored files:

// ~/go/src/test/test.go
package main

import "a"

func main() {
    println(a.HelloFromA)
}

// ~/go/src/test/vendor/a/a.go
package a

var HelloFromA = "Hello!"

$ export GO15VENDOREXPERIMENT=1
$ go run test.go
Hello!
$ godef -f test.go "a.HelloFromA"
parseLocalPackage error: no more package files found
godef: no declaration found for a.HelloFromA

from godef.

Yeah, this needs to be fixed.

from godef.

I have a PR for that, don't I ? My fork has that implemented.

Le jeu. 14 janv. 2016 06:47, Roger Peppe [email protected] a
écrit :

Yeah, this needs to be fixed.

—
Reply to this email directly or view it on GitHub
#13 (comment).

from godef.

@rogpeppe 1.6 has released, any plan for this issue ?

from godef.

Also interested :-)

from godef.

@abourget would you like to make a PR based the latest codes ?

from godef.

link PR #25

from godef.

+1

from godef.

Maybe this project is dead, It's time to have a fork !!!

from godef.

Has #33 finally fixed this?

from godef.