cmd/go: misleading "invalid version" error message for gopkg.in modules with mismatched module path #43809

jayconrod · 2021-01-20T18:27:20Z

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

$ go version
go version go1.16beta1 darwin/amd64

Does this issue reproduce with the latest release?

yes

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

go env Output

$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/jayconrod/Library/Caches/go-build"
GOENV="/Users/jayconrod/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/jayconrod/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/jayconrod/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/opt/go/installed"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/opt/go/installed/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.16beta1"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/jayconrod/Code/test/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/rq/x0692kqj6ml8cvrhcqh5bswc008xj1/T/go-build892501000=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

Run go mod download on a module with a gopkg.in module path, when the actual go.mod file has a different module path.

go mod download gopkg.in/golang/mock.v1@v1.4.4

What did you expect to see?

go mod download should succeed. go get and anything else that actually loads the module graph should fail with a mismatched module path error message.

In general, go mod download should succeed when the module path on the command line doesn't match the actual module path in go.mod. If it didn't, module version replacements wouldn't work. For example, go mod download github.com/google/go-cloud@v0.21.0 works even though it declares itself as gocloud.dev.

It's fine that the go command checks that the version matches the major version suffix, but it should check that against the module path named on the command line, not the module path in go.mod.

What did you see instead?

go mod download: gopkg.in/golang/mock.v1@v1.4.4: invalid version: go.mod has non-....v1 module path "github.com/golang/mock" at revision v1.4.4

gopkg.in module paths must have a major version suffix, in this case, .v1. The go command is reporting that the actual module path in go.mod, github.com/golang/mock does not end with .v1.

cc @bcmills @matloob @julieqiu

The text was updated successfully, but these errors were encountered:

bcmills · 2021-01-20T20:06:26Z

That error is coming from here:

go/src/cmd/go/internal/modfetch/coderepo.go

Lines 769 to 771 in f1980ef

    
           if r.pathMajor != "" { // ".v1", ".v2" for gopkg.in 
        
           	return "", "", nil, fmt.Errorf("%s has non-...%s module path %q%s at revision %s", file1, r.pathMajor, mpath1, suffix, rev) 
        
           }

See previously #34254.

bcmills · 2021-01-20T20:11:42Z

I think the relevant check is here:

go/src/cmd/go/internal/modfetch/coderepo.go

Lines 815 to 822 in f1980ef

    
           if mpathMajor == "" { 
        
           	// Even if pathMajor is ".v0" or ".v1", we can't be sure that a module 
        
           	// without a suffix is tagged appropriately. Besides, we don't expect clones 
        
           	// of non-gopkg.in modules to have gopkg.in paths, so a non-empty, 
        
           	// non-gopkg.in mpath is probably the wrong module for any such pathMajor 
        
           	// anyway. 
        
           	return false 
        
           }

bcmills · 2021-01-20T20:14:55Z

Prompted by that comment, I'm not sure what the use of the go mod download command in this example would be.

gopkg.in/golang/mock.v1@v1.4.4 cannot be used for module gopkg.in/golang/mock.v1 because its go.mod file declares a non-matching path. The only direction it could go would be:

replace github.com/golang/mock => gopkg.in/golang/mock.v1 v1.4.4

but in that case why not use the canonical path instead?

replace github.com/golang/mock => github.com/golang/mock v1.4.4

So I agree that the error message we're emitting is confusing, but I'm not sure that it should be changed to a non-error.

jayconrod · 2021-01-20T21:17:20Z

Context for this was a 404 error on pkgsite. They're trying to understand why they show errors for some pages where godoc.org succeeds. gopkg.in/golang/mock.v1 worked in GOPATH mode, so https://godoc.org/gopkg.in/golang/mock.v1 works.

Since go mod download fails here (proxy returns 410 with same message), they can't read the go.mod to find the correct module path and redirect to the right page.

bcmills · 2021-01-20T21:20:39Z

We could perhaps weaken the mpathMajor == "" case in isMajor to return pathMajor == "v0" || pathMajor == "v1".

That would change the semantics of isMajor from “whether the versions allowed for mpath are compatible” to “whether the versions allowed for mpath may be compatible”. I'm not sure what other implications that would carry, but I suspect they're relatively benign.

bcmills · 2021-01-26T19:23:35Z

See previously #34254.

jayconrod · 2021-01-26T19:40:19Z

Just to add a note: this is a bug with the way cmd/go handles gopkg.in modules that are being used as replacements for non-gopkg.in paths. So the fix here should be very narrow.

In general, a replacement module should have the original module path in its go.mod file. The go command doesn't check that in all cases. We should be careful not to loosen that restriction though.

gopherbot · 2021-06-03T06:38:30Z

Change https://golang.org/cl/324470 mentions this issue: cmd/go: improve "invalid version" error messages

gopherbot · 2021-07-28T14:37:14Z

Change https://golang.org/cl/338150 mentions this issue: cmd/go: add error message when a gopkg.in module is in a replacement

ianlancetaylor · 2021-11-17T00:05:30Z

This is in the Go1.18 milestone. Is it likely to happen for 1.18? Thanks.

ianlancetaylor · 2022-01-28T22:24:37Z

@bcmills This is in the 1.18 milestone; time to move to 1.19? Thanks.

ianlancetaylor · 2022-06-24T19:00:45Z

@bcmills @matloob This issue is marked for 1.19. It's just been rolling forward in milestones. Should it move to Backlog?

jayconrod added the NeedsFix The path to resolution is known, but the work has not been done. label Jan 20, 2021

jayconrod added this to the Go1.17 milestone Jan 20, 2021

This comment has been minimized.

Sign in to view

jayconrod added the okay-after-beta1 Used by release team to mark a release-blocker issue as okay to resolve either before or after beta1 label May 17, 2021

jayconrod modified the milestones: Go1.17, Go1.18 Jul 30, 2021

bcmills modified the milestones: Go1.18, Go1.19 Feb 1, 2022

bcmills self-assigned this Feb 1, 2022

gopherbot removed the okay-after-beta1 Used by release team to mark a release-blocker issue as okay to resolve either before or after beta1 label Jun 10, 2022

bcmills modified the milestones: Go1.19, Backlog Jun 28, 2022

bcmills removed their assignment Jun 28, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

cmd/go: misleading "invalid version" error message for gopkg.in modules with mismatched module path #43809

cmd/go: misleading "invalid version" error message for gopkg.in modules with mismatched module path #43809

jayconrod commented Jan 20, 2021

This comment has been minimized.

This comment has been minimized.

bcmills commented Jan 20, 2021

bcmills commented Jan 20, 2021

bcmills commented Jan 20, 2021

jayconrod commented Jan 20, 2021

bcmills commented Jan 20, 2021

bcmills commented Jan 26, 2021

jayconrod commented Jan 26, 2021 •

edited by bcmills

gopherbot commented Jun 3, 2021

gopherbot commented Jul 28, 2021

ianlancetaylor commented Nov 17, 2021

ianlancetaylor commented Jan 28, 2022

ianlancetaylor commented Jun 24, 2022

cmd/go: misleading "invalid version" error message for gopkg.in modules with mismatched module path #43809

cmd/go: misleading "invalid version" error message for gopkg.in modules with mismatched module path #43809

Comments

jayconrod commented Jan 20, 2021

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

Does this issue reproduce with the latest release?

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

What did you do?

What did you expect to see?

What did you see instead?

This comment has been minimized.

This comment has been minimized.

bcmills commented Jan 20, 2021

bcmills commented Jan 20, 2021

bcmills commented Jan 20, 2021

jayconrod commented Jan 20, 2021

bcmills commented Jan 20, 2021

bcmills commented Jan 26, 2021

jayconrod commented Jan 26, 2021 • edited by bcmills

gopherbot commented Jun 3, 2021

gopherbot commented Jul 28, 2021

ianlancetaylor commented Nov 17, 2021

ianlancetaylor commented Jan 28, 2022

ianlancetaylor commented Jun 24, 2022

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

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

jayconrod commented Jan 26, 2021 •

edited by bcmills