Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

dep ensure panics #1945

Closed
mvpmvh opened this issue Jul 16, 2018 · 4 comments
Closed

dep ensure panics #1945

mvpmvh opened this issue Jul 16, 2018 · 4 comments

Comments

@mvpmvh
Copy link

mvpmvh commented Jul 16, 2018

What version of dep are you using (dep version)?

dep:
 version     : devel
 build date  : 
 git hash    : 
 go version  : go1.10
 go compiler : gc
 platform    : darwin/amd64
 features    : ImportDuringSolve=false

What dep command did you run?

dep ensure -v

What did you expect to see?

success response

What did you see instead?

# Bringing vendor into sync
(1/39) Wrote github.com/pkg/[email protected]: prune options changed ( -> UT)
panic: interface conversion: gps.LockedProject is nil, not verify.VerifiableProject

goroutine 1 [running]:
github.com/golang/dep.(*DeltaWriter).Write(0xc420510e40, 0xc42002a0a4, 0x44, 0x15fb5e0, 0xc420408060, 0xc42002bd01, 0xc4200b0ff0, 0x0, 0x15f3660)
        /Users/michael.hatch/go/src/github.com/golang/dep/txn_writer.go:539 +0x1f31
main.(*ensureCommand).runDefault(0xc420183990, 0xc4200c4980, 0xc42019c7d0, 0x0, 0x0, 0xc4200e0090, 0x15fb5e0, 0xc420408060, 0xc42002a0a4, 0x44, ...)
        /Users/michael.hatch/go/src/github.com/golang/dep/cmd/dep/ensure.go:318 +0x94a
main.(*ensureCommand).Run(0xc420183990, 0xc4200c4980, 0xc42019c7d0, 0x0, 0x0, 0x0, 0x0)
        /Users/michael.hatch/go/src/github.com/golang/dep/cmd/dep/ensure.go:218 +0x865
main.(*Config).Run(0xc420084fc0, 0x0)
        /Users/michael.hatch/go/src/github.com/golang/dep/cmd/dep/main.go:199 +0xf4b
main.main()
        /Users/michael.hatch/go/src/github.com/golang/dep/cmd/dep/main.go:76 +0x54d
@mvpmvh
Copy link
Author

mvpmvh commented Jul 16, 2018

i was on master branch when this occurred. removing dep from my $GOBIN and checkout out tag v0.4.1 and installing that version seemed to work as expected.

@sdboyer
Copy link
Member

sdboyer commented Jul 18, 2018

Hmm, that case shouldn't be reachable (but obviously it is). i'll get it figured out and fixed up. Thanks!

@sdboyer
Copy link
Member

sdboyer commented Jul 18, 2018

Actually, if you could provide repro steps, it'd be helpful!

sdboyer added a commit that referenced this issue Jul 20, 2018
Relates to #1945.

While this doesn't fix the issue, it at least prevents a panic from
occurring that blows up a `dep ensure` run, and should give us enough
information to debug it if/when someone runs into it.
@sdboyer
Copy link
Member

sdboyer commented Aug 4, 2018

Figured out the problem! All taken care of now.

@sdboyer sdboyer closed this as completed in 6b79ccc Aug 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants