code review 5858043: doc: general update of gccgo_install

doc: general update of gccgo_install

Fixes issue 3325.

[iant, 2012-03-20 18:24:09]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg/

[bradfitz, 2012-03-20 18:36:10]

LGTM

that's a lot of docs.

http://codereview.appspot.com/5858043/diff/1/doc/gccgo_install.html
File doc/gccgo_install.html (right):

http://codereview.appspot.com/5858043/diff/1/doc/gccgo_install.html#newcode50
doc/gccgo_install.html:50: convenience, a known stable version of the Go support
is available in
known-stable?

http://codereview.appspot.com/5858043/diff/1/doc/gccgo_install.html#newcode86
doc/gccgo_install.html:86: On x86 GNU/Linux systems the gccgo compiler is able
to
gccgo was previously in <code>.

likewise with gold?

[minux1, 2012-03-20 18:36:40]

http://codereview.appspot.com/5858043/diff/1/doc/gccgo_install.html
File doc/gccgo_install.html (right):

http://codereview.appspot.com/5858043/diff/1/doc/gccgo_install.html#newcode18
doc/gccgo_install.html:18: Note that gccgo is not the 6g compiler; see
s/6g/gc/
to be consistent with the other documents.

http://codereview.appspot.com/5858043/diff/1/doc/gccgo_install.html#newcode117
doc/gccgo_install.html:117: option <code>--with-ld=GOLD_BINARY</code>.
slant the word GOLD_BINARY or use something like /path/to/your/gold/binary
if the user should replace the word with the actual gold binary?

[iant, 2012-03-20 18:53:29]

Thanks for the comments.  Updated.

http://codereview.appspot.com/5858043/diff/1/doc/gccgo_install.html
File doc/gccgo_install.html (right):

http://codereview.appspot.com/5858043/diff/1/doc/gccgo_install.html#newcode18
doc/gccgo_install.html:18: Note that gccgo is not the 6g compiler; see
On 2012/03/20 18:36:41, minux wrote:
> s/6g/gc/
> to be consistent with the other documents.

Done.

http://codereview.appspot.com/5858043/diff/1/doc/gccgo_install.html#newcode50
doc/gccgo_install.html:50: convenience, a known stable version of the Go support
is available in
On 2012/03/20 18:36:10, bradfitz wrote:
> known-stable?

I just changed it to stable.

http://codereview.appspot.com/5858043/diff/1/doc/gccgo_install.html#newcode86
doc/gccgo_install.html:86: On x86 GNU/Linux systems the gccgo compiler is able
to
On 2012/03/20 18:36:10, bradfitz wrote:
> gccgo was previously in <code>.
> 
> likewise with gold?

I dropped all the <code> annotations for gccgo and GCC and gold, I don't think
they helped.

http://codereview.appspot.com/5858043/diff/1/doc/gccgo_install.html#newcode117
doc/gccgo_install.html:117: option <code>--with-ld=GOLD_BINARY</code>.
On 2012/03/20 18:36:41, minux wrote:
> slant the word GOLD_BINARY or use something like /path/to/your/gold/binary
> if the user should replace the word with the actual gold binary?

Done.

[r, 2012-03-20 20:51:28]

LGTM
this is great
please run tidy before submitting

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html
File doc/gccgo_install.html (right):

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcode10
doc/gccgo_install.html:10: frontend itself is under a BSD-style license, gccgo
is
sigh. i guess the Teutonic "frontend" is a word now.

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcode44
doc/gccgo_install.html:44: If you can not use a release, or simply prefer to
build gccgo for
s/can not/cannot/
s/simply //

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:236: Use the <code>-static-libgo</code> option to link
statically against
not --static-libgo? (two minus signs?)

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:244: default for the <code>gc</code> compiler).
ditto

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:330: and interfaces will show up as two element
structures.  Go
s/two element/two-element/

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:331: maps and channels are always represented as C
pointers.
C pointers to run-time structures

(for clarity)

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:484: <code>-fdump-go-spec=<var>FILENAME</var></code>. 
This will create the
again, only one minus?

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:485: file <code><var>FILENAME</var></code> as a
side-effect of the
s/-/ /

multiword adjectives have hyphens, adjective plus noun don't need them.

[iant, 2012-03-20 21:16:13]

Thanks.  Submitting.

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html
File doc/gccgo_install.html (right):

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcode10
doc/gccgo_install.html:10: frontend itself is under a BSD-style license, gccgo
is
On 2012/03/20 20:51:28, r wrote:
> sigh. i guess the Teutonic "frontend" is a word now.

Yep.

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcode44
doc/gccgo_install.html:44: If you can not use a release, or simply prefer to
build gccgo for
On 2012/03/20 20:51:28, r wrote:
> s/can not/cannot/
> s/simply //

Done.

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:236: Use the <code>-static-libgo</code> option to link
statically against
On 2012/03/20 20:51:28, r wrote:
> not --static-libgo? (two minus signs?)

Surprisingly, no (although --static-libgo does work also).  gcc mostly does not
follow the GNU long argument conventions, except for --help, --version, etc.

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:244: default for the <code>gc</code> compiler).
On 2012/03/20 20:51:28, r wrote:
> ditto

Yes, correct here too.

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:330: and interfaces will show up as two element
structures.  Go
On 2012/03/20 20:51:28, r wrote:
> s/two element/two-element/

Done.

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:331: maps and channels are always represented as C
pointers.
On 2012/03/20 20:51:28, r wrote:
> C pointers to run-time structures
> 
> (for clarity)

Done.

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:484: <code>-fdump-go-spec=<var>FILENAME</var></code>. 
This will create the
On 2012/03/20 20:51:28, r wrote:
> again, only one minus?

Yep.

http://codereview.appspot.com/5858043/diff/3003/doc/gccgo_install.html#newcod...
doc/gccgo_install.html:485: file <code><var>FILENAME</var></code> as a
side-effect of the
On 2012/03/20 20:51:28, r wrote:
> s/-/ /
> 
> multiword adjectives have hyphens, adjective plus noun don't need them.

Done.

[iant, 2012-03-20 21:16:39]

*** Submitted as http://code.google.com/p/go/source/detail?r=ec3f6fe48043 ***

doc: general update of gccgo_install

Fixes issue 3325.

R=golang-dev, bradfitz, minux.ma, r
CC=golang-dev
http://codereview.appspot.com/5858043