code review 13891043: doc: document installation of go.tools commands

doc: document installation of go.tools commands

Fixes issue 5663.

[adg, 2013-09-25 03:49:08]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/go

[r, 2013-09-25 03:55:27]

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html
File doc/install-source.html (right):

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html#newcod...
doc/install-source.html:226: Various Go tools (including <a
href="/cmd/godoc/">godoc</a>)
The source code for several Go tools .... is kept in ....

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html#newcod...
doc/install-source.html:228: You can install them with the <code>go get</code>
command:
<code>go</code> <code>get</code>
the fixed-width space looks crappy.

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html#newcod...
doc/install-source.html:228: You can install them with the <code>go get</code>
command:
To install them, run the ....

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html#newcod...
doc/install-source.html:232: $ go get code.google.com/p/go.tools/cmd/...
this installs all of them. maybe say how to install one?

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html#newcod...
doc/install-source.html:236: Note that <code>go get</code> requires various
<code>go</code> <code>get</code>
the fixed-width space looks crappy.

[adg, 2013-09-25 04:01:48]

PTAL

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html
File doc/install-source.html (right):

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html#newcod...
doc/install-source.html:226: Various Go tools (including <a
href="/cmd/godoc/">godoc</a>)
On 2013/09/25 03:55:27, r wrote:
> The source code for several Go tools .... is kept in ....

Done.

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html#newcod...
doc/install-source.html:228: You can install them with the <code>go get</code>
command:
On 2013/09/25 03:55:27, r wrote:
> <code>go</code> <code>get</code>
> the fixed-width space looks crappy.

Done.

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html#newcod...
doc/install-source.html:228: You can install them with the <code>go get</code>
command:
On 2013/09/25 03:55:27, r wrote:
> To install them, run the ....

Done.

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html#newcod...
doc/install-source.html:232: $ go get code.google.com/p/go.tools/cmd/...
On 2013/09/25 03:55:27, r wrote:
> this installs all of them. maybe say how to install one?

Done.

https://codereview.appspot.com/13891043/diff/1/doc/install-source.html#newcod...
doc/install-source.html:236: Note that <code>go get</code> requires various
On 2013/09/25 03:55:27, r wrote:
> <code>go</code> <code>get</code>
> the fixed-width space looks crappy.
> 

Done.

[r, 2013-09-25 04:04:53]

LGTM

https://codereview.appspot.com/13891043/diff/6001/doc/install-source.html
File doc/install-source.html (right):

https://codereview.appspot.com/13891043/diff/6001/doc/install-source.html#new...
doc/install-source.html:244: present on your system to check out code (just
Mercurial is required in this case).
To install these tools, <code>go</code> <code>get</code> command requires that
Mercurial be installed locally.

[adg, 2013-09-25 04:10:58]

*** Submitted as https://code.google.com/p/go/source/detail?r=78fe414819a1 ***

doc: document installation of go.tools commands

Fixes issue 5663.

R=golang-dev, r
CC=golang-dev
https://codereview.appspot.com/13891043

https://codereview.appspot.com/13891043/diff/6001/doc/install-source.html
File doc/install-source.html (right):

https://codereview.appspot.com/13891043/diff/6001/doc/install-source.html#new...
doc/install-source.html:244: present on your system to check out code (just
Mercurial is required in this case).
On 2013/09/25 04:04:54, r wrote:
> To install these tools, <code>go</code> <code>get</code> command requires that
> Mercurial be installed locally.

Done.

[minux1, 2013-09-25 04:11:22]

do we also need to note that godoc will be installed to $GOROOT/bin and
vet to $GOROOT/pkg/tool/$GOOS_$GOARCH/?
(this fact is also contained in doc/go1.2.html)

[satish.puranam, 2013-09-25 04:12:01]

there a small typo
s/seeral/several

https://codereview.appspot.com/13891043/diff/6001/doc/install-source.html
File doc/install-source.html (right):

https://codereview.appspot.com/13891043/diff/6001/doc/install-source.html#new...
doc/install-source.html:226: The source code for seeral Go tools (including <a
href="/cmd/godoc/">godoc</a>)
s/seeral/several

[adg, 2013-09-25 04:14:24]

On 25 September 2013 14:11, <minux.ma@gmail.com> wrote:

> do we also need to note that godoc will be installed to $GOROOT/bin and
> vet to $GOROOT/pkg/tool/$GOOS_$**GOARCH/?
> (this fact is also contained in doc/go1.2.html)
>

You're right, that will be confusing, particularly for vet and cover which
will only be accessible as "go tool vet" etc.

I'll send another CL>