Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

cmd/go: unknown revision converting vendor, but revision exists #27191

Closed
mitchellh opened this issue Aug 24, 2018 · 7 comments
Closed

cmd/go: unknown revision converting vendor, but revision exists #27191

mitchellh opened this issue Aug 24, 2018 · 7 comments
Labels
FrozenDueToAge modules NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone
Go1.12

Comments

@mitchellh
Copy link

What version of Go are you using (go version)?

go version go1.11rc2 darwin/amd64

Does this issue reproduce with the latest release?

Latest go1.11 release - yes
Latest go1.10 - no because no modules

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/mitchellh/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/mitchellh/go"
GOPROXY=""
GORACE=""
GOROOT="/Users/mitchellh/code/3rdparty/go"
GOTMPDIR=""
GOTOOLDIR="/Users/mitchellh/code/3rdparty/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/mitchellh/code/go/src/github.com/hashicorp/consul/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/gh/ts3cxwpd7l9cwb2n5z8j51980000gn/T/go-build206170924=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

Checkout 542cace9a from https://github.com/hashicorp/consul

$ git clone https://github.com/hashicorp/consul.git 
$ cd consul/
$ git checkout 542cace9a
$ go mod init
go: creating new go.mod: module github.com/hashicorp/consul
go: copying requirements from vendor/vendor.json
go: converting vendor/vendor.json: stat github.com/hashicorp/go-discover@1b9cec3de1d963ee4354e0151cbcc0a976791be6: unknown revision 1b9cec3de1d963ee4354e0151cbcc0a976791be6

What did you expect to see?

Probably no error.

What did you see instead?

Reference to revision 1b9... but if I go to the repo, that exists:
hashicorp/go-discover@1b9cec3
@oiooj oiooj added the modules label Aug 24, 2018
@ianlancetaylor ianlancetaylor added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Aug 24, 2018
@ianlancetaylor ianlancetaylor added this to the Go1.12 milestone Aug 24, 2018
@ianlancetaylor
Copy link
Contributor

CC @bcmills @rsc

@thepudds
Copy link
Contributor

@mitchellh I wonder if that 1b9... commit hash in go-discover is not associated with a branch or tag. The git-based lookup in the go command was tighten up a bit recently.

For example, in #27043 the problem was that the commit hash of interest referenced an unmerged pull request, which the go command now disallows (though it was previously more liberal).

See for example the comment in #27043 (comment) or https://go-review.googlesource.com/c/go/+/129800.

Perhaps you are encountering something in that neighborhood, or perhaps you are seeing something else entirely.

@oiooj
Copy link
Member

oiooj commented Aug 29, 2018

I think @thepudds is right. The commit hash hashicorp/go-discover@1b9cec3 is not associated with a branch or tag.

@bcmills
Copy link
Contributor

bcmills commented Sep 6, 2018

Yep, it's on an unmerged fork in github.com/MagnumOpus21/go-discover.

screenshot 2018-09-06 at 14 28 35

This seems to be working as intended: we want individual maintainers to be in charge of which commits they publish. (Rejected pull requests and forks by other users should not be available for version resolution.)

@bcmills bcmills closed this as completed Sep 6, 2018
@bcmills
Copy link
Contributor

bcmills commented Sep 6, 2018

Hmm, and @banks even specifically questioned the vendoring changes that added that commit, in hashicorp/consul#4412 (comment).

(I assume that @MagnumOpus21 pulled in the fork by accident.)

That just goes to show the value of module paths and version tags, I suppose. 🙂

@MagnumOpus21
Copy link

I don't know what I did to cause this. Every time I would open a PR on the hashicorp repo, I would push commits to my own repo and then open a PR to merge them. I just followed govendor's FAQ for modifying the vendor file. Hope this throws some light on this issue.

@mitchellh
Copy link
Author

No problem. Thanks for clarifying @bcmills we probably just got a weird commit into the vendor.

@backkem backkem mentioned this issue Jan 13, 2019
Merged
@gopherbot gopherbot mentioned this issue Apr 25, 2019
Closed
@golang golang locked and limited conversation to collaborators Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge modules NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Milestone
Go1.12
Development

No branches or pull requests
7 participants
@mitchellh @oiooj @ianlancetaylor @bcmills @MagnumOpus21 @gopherbot @thepudds