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

Can't use go get -u on this module with its current latest pseudo-version #151

Closed
dmitshur opened this Issue Mar 14, 2019 · 2 comments

Comments

Projects
None yet
2 participants
@dmitshur
Copy link
Member

dmitshur commented Mar 14, 2019

This module is currently affected by an issue similar to openzipkin/zipkin-go#114. It can be reproduced with Go 1.12 as follows:

$ export GO111MODULE=on
$ cd $(mktemp -d)
$ go mod init m
go: creating new go.mod: module m
$ go get -u google.golang.org/genproto
...
go: finding github.com/golang/lint latest
go: github.com/golang/lint@v0.0.0-20190313153728-d0100b6bd8b3: parsing go.mod: unexpected module path "golang.org/x/lint"
...
go get: error loading module requirements
$ echo $?
1

To resolve this for this repository, its go.mod file needs to be updated to include newer versions of dependencies, in order to avoid pulling in older module versions that cause this conflict.

I haven't looked into the exact cause for this repo yet. It might be just a matter of waiting for golang/go#30833 to be resolved. I'll investigate more later.

/cc @broady @bcmills

@jadekler

This comment has been minimized.

Copy link
Member

jadekler commented Mar 14, 2019

Seems to be an issue with how go modules interprets modules whose names have changed (github.com/x/lint and golang.org/x/lint). Tracking discussion in golang/go#30831.

@jadekler

This comment has been minimized.

Copy link
Member

jadekler commented Mar 14, 2019

Since this is a core issue and affects anyone depending on x/lint, I'm closing this in favour of golang/go#30831.

@jadekler jadekler closed this Mar 14, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.