tools / godep

dependency tool for go
http://godoc.org/github.com/tools/godep
BSD 3-Clause "New" or "Revised" License
5.54k stars 455 forks source link

godep save missing some dependencies #486

Open setekhid opened 8 years ago

setekhid commented 8 years ago

See here https://github.com/setekhid/godeps-bug

I expected godep save all dependencies recursively in Godeps folder when I run godep save ./....

But I got some packages missing, suck like github.com/setekhid/godeps-debug in Godeps.json file, but the folder is empty in _workspace.

godep version output

godep v74 (darwin/amd64/go1.4.2)

go version output

go version go1.4.2 darwin/amd64

freeformz commented 8 years ago

This works for me.

$ go get github.com/setekhid/godeps-bug
$ cd $GOPATH/go/src/github.com/setekhid/godeps-bug
$ rm -rf Godeps              # remove any old Godep data
$ godep save
$ find Godeps
Godeps
Godeps/_workspace
Godeps/_workspace/.gitignore
Godeps/_workspace/src
Godeps/_workspace/src/github.com
Godeps/_workspace/src/github.com/setekhid
Godeps/_workspace/src/github.com/setekhid/godeps-debug
Godeps/_workspace/src/github.com/setekhid/godeps-debug/ignored.go
Godeps/_workspace/src/github.com/setekhid/godeps-debug/subpkt
Godeps/_workspace/src/github.com/setekhid/godeps-debug/subpkt/struct.go
Godeps/Godeps.json
Godeps/Readme

Note that ignored.go is included.

PS: Workspace support is deprecated.

I tried this with my godep compiled with 1.7beta1 and 1.4.2 and the results are the same.

If you can reproduce please run save with -d -v, capture all of the output and upload it somewhere so that I can inspect it.

Thanks!

setekhid commented 8 years ago

Thanks for the guild above, I've figured out the reason.

Well, I'd like to collect every my own projects into a special path, and link these repo's root directories into $GOPATH. That cause some trouble while godep iterating the dependencies. kr/fs.Walk will not trace symbolic links.