-
Notifications
You must be signed in to change notification settings - Fork 18k
proposal: x/build/cmd/release: remove godoc for Go 1.13 #30029
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
Comments
I am skeptical about removing batteries like godoc from the go distribution. If anything, I think we should be having discussions about what tools are missing from the go distribution, like goimports. |
I suspect few users use godoc's HTML mode, though. People are always shocked to hear that it's possible or that I use it. As a user of it myself, I'd be fine with it being go get-able if it means smaller downloads. |
We discussed this some more. A lot of us use godoc to view package docs and the spec locally but we suspect very few users do. Let's try deleting godoc from cmd/release now, it will be dropped from Go 1.13 beta, and we'll see if we get pushback then. |
Change https://golang.org/cl/162417 mentions this issue: |
Updates #30029 Change-Id: I88e09035d675e7a6855ada0262eb42636c9822cc Reviewed-on: https://go-review.googlesource.com/c/162417 Reviewed-by: Andrew Bonventre <andybons@golang.org>
Change https://golang.org/cl/162557 mentions this issue: |
…ease to include godoc Updates #30029 Change-Id: I88e09035d675e7a6855ada0262eb42636c9822cc Reviewed-on: https://go-review.googlesource.com/c/162417 Reviewed-by: Andrew Bonventre <andybons@golang.org> (cherry picked from commit 7cf31d8) Reviewed-on: https://go-review.googlesource.com/c/162557
Updates golang#30029 Change-Id: I88e09035d675e7a6855ada0262eb42636c9822cc Reviewed-on: https://go-review.googlesource.com/c/162417 Reviewed-by: Andrew Bonventre <andybons@golang.org>
Updates golang#30029 Change-Id: I88e09035d675e7a6855ada0262eb42636c9822cc Reviewed-on: https://go-review.googlesource.com/c/162417 Reviewed-by: Andrew Bonventre <andybons@golang.org>
Maybe it's only me but I start the web server before starting my editor. I preach about Go when ever I get a chance and one of the selling points is that all you need to do to get started is to install the binary, start the web server and all you need is there on your machine to start learning Go. When having to use Java I download the API docs to always have it available. With Go it's so great to have everything in one package, while a I'm all for removing clutter but I think easy access to the documentation should come with the binary. To me it's not clutter. If it’s underused maybe it should be more prominent in the getting started section? Or maybe have |
Just to see how long it takes I just downloaded the latest MacOS binary and it took me 8 seconds. I don't know how much I would save by loosing the web server but even if I had a slower connection I wouldn't consider a 4-5 minute download a long time. But then again maybe I'm just old when considering a 4 minute download as "fast"... :) |
@spagettikod Thanks for feedback. One of the bigger motivations for making it available primarily via When I talked about this issue with @ianthehat, he made a good point that we've only considered two extremes in this proposal so far: to include vs not include the It would be a bigger change to design and implement (and should be a separate proposal), but perhaps a similar solution can provide a better experience to Go users (e.g., they can conveniently choose whether to have the |
Thanks for considering my feedback @dmitshur! I didn't realize that
I looked at the Google Cloud link but I don't see the difference between that and what you already get using Anyway I don't mind the possibility of updating the docs provided in the release using |
@spagettikod You should be aware of #29206 (comment). The current plan is to make |
Change https://golang.org/cl/174322 mentions this issue: |
Change https://golang.org/cl/182619 mentions this issue: |
Updates #30029 Updates #31457 Change-Id: I08414a544615e74afb47f7a10a00f1e22adfd40c Reviewed-on: https://go-review.googlesource.com/c/go/+/182619 Reviewed-by: Katie Hockman <katie@golang.org>
Split out of #27151 (reduce download sizes), this proposal is to remove
godoc
(the webserver) from Go 1.13 releases and document that users cango get
it instead.As of Go 1.12,
godoc
is only a webserver andgo doc
is the CLI tool./cc @dmitshur
The text was updated successfully, but these errors were encountered: