-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
cmd/go: improve go help goproxy
#29009
Comments
Is this still in go1.12 milestone? Is it a possibility to get rid of the > go get -v -d golang.org/x/tools/cmd/guru/serial Fetching http://localhost:8080/golang.org/x/tools/cmd/guru/serial/@v/list go: finding golang.org/x/tools/cmd/guru/serial latest Fetching http://localhost:8080/golang.org/x/tools/cmd/guru/serial/@latest Fetching http://localhost:8080/golang.org/x/tools/cmd/guru/serial/@v/list Fetching http://localhost:8080/golang.org/x/tools/cmd/guru/@v/list go: finding golang.org/x/tools/cmd/guru latest Fetching http://localhost:8080/golang.org/x/tools/cmd/guru/@latest Fetching http://localhost:8080/golang.org/x/tools/cmd/guru/@v/list Fetching http://localhost:8080/golang.org/x/tools/cmd/@v/list go: finding golang.org/x/tools/cmd latest Fetching http://localhost:8080/golang.org/x/tools/cmd/@latest Fetching http://localhost:8080/golang.org/x/tools/cmd/@v/list Fetching http://localhost:8080/golang.org/x/tools/@v/list go: finding golang.org/x/tools v0.0.0-20181212200058-49db546f375e Fetching http://localhost:8080/golang.org/x/tools/@v/v0.0.0-20181212200058-49db546f375e.info go: downloading golang.org/x/tools v0.0.0-20181212200058-49db546f375e Fetching http://localhost:8080/golang.org/x/tools/@v/v0.0.0-20181212200058-49db546f375e.zip go: extracting golang.org/x/tools v0.0.0-20181212200058-49db546f375e Fetching http://localhost:8080/golang.org/x/tools/@v/v0.0.0-20181212200058-49db546f375e.mod |
Ops, I've missed the cc notification on this one. So I opened a CL a while ago (https://go-review.googlesource.com/c/go/+/126615) that updated the documentation and @bcmills suggested to look at potentially removing From what I know, having
GOPROXY implementers therefore, can easily miss the fact that when the implement their own BTW, |
Thanks @marwan-at-work for the explanation. Can you update your cl to mention this bug? One minor thing - If the > go list -m -versions -json golang.org/x/textgo: finding golang.org/x/text v0.3.0 { "Path": "golang.org/x/text", "Version": "v0.3.0", "Versions": [ "v0.1.0", "v0.2.0", "v0.3.0" ], "Time": "2017-12-14T13:08:43Z" } vs. go list -m -versions -json golang.org/x/text@master go: finding golang.org/x/text master { "Path": "golang.org/x/text", "Version": "v0.0.0-20181211190257-17bcc049122f", "Versions": [ "v0.1.0", "v0.2.0", "v0.3.0" ], "Time": "2018-12-11T19:02:57Z" } I don't know why the pseudo version picks up v0.0.0 instead of v0.3.0 (maybe related to #29262). |
@hyangah this is because you specified But good call out, |
Change https://golang.org/cl/126615 mentions this issue: |
Looking again into the So, I am still not sure what's the purpose of this latest query. Regarding the v0.0.0 comment, I meant the prefix used to generate the pseudo version. |
Change https://golang.org/cl/157800 mentions this issue: |
Let's close this since now there is a comprehensive set of documentation about modules. |
'go help goproxy' describes the module proxy protocol, and it needs to document more details to guide correct proxy implementation.
'go get; issues a /latest query to go proxy, (https://go.googlesource.com/go/+/master/src/cmd/go/internal/modfetch/proxy.go#185) but this type of query is not documented.
document what's the expected results for for non modules.
For example, what's the expected results or error code for the following module or package?
@bcmills @rsc
The text was updated successfully, but these errors were encountered: