code review 5477056: doc/go1: the simpler package changes

doc/go1: the simpler package changes

[r, 2011-12-09 23:30:27]

Hello golang-dev@googlegroups.com,

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

[albert.strasheim, 2011-12-10 05:47:21]

http://codereview.appspot.com/5477056/diff/1/doc/go1.html
File doc/go1.html (right):

http://codereview.appspot.com/5477056/diff/1/doc/go1.html#newcode637
doc/go1.html:637: <a href="/pkg/syscall"><code>strconv</code></a>
/pkg/strconv?

[r, 2011-12-10 06:01:33]

Hello golang-dev@googlegroups.com, fullung@gmail.com (cc:
golang-dev@googlegroups.com),

Please take another look.

[dsymonds, 2011-12-10 06:02:35]

Just a random drive-by thought.

http://codereview.appspot.com/5477056/diff/1/doc/go1.tmpl
File doc/go1.tmpl (right):

http://codereview.appspot.com/5477056/diff/1/doc/go1.tmpl#newcode515
doc/go1.tmpl:515: <a href="/pkg/syscall"><code>syscall</code></a>
Can you make all the package link paths end in a slash? These ones end up with a
301 redirect which, while functional, adds an unnecessary HTTP round trip.

[r, 2011-12-10 06:41:02]

Hello golang-dev@googlegroups.com, fullung@gmail.com, dsymonds@golang.org (cc:
golang-dev@googlegroups.com),

Please take another look.

[r2, 2011-12-10 06:41:18]

On Dec 9, 2011, at 10:02 PM, dsymonds@golang.org wrote:

> Can you make all the package link paths end in a slash? These ones end
> up with a 301 redirect which, while functional, adds an unnecessary HTTP
> round trip.

i learn something new and broken about the web every day.

done.

-rob

[r, 2011-12-12 04:11:24]

Hello golang-dev@googlegroups.com, fullung@gmail.com, dsymonds@golang.org,
r@google.com (cc: golang-dev@googlegroups.com),

Please take another look.

[r, 2011-12-12 04:48:46]

Hello golang-dev@googlegroups.com, fullung@gmail.com, dsymonds@golang.org,
r@google.com (cc: golang-dev@googlegroups.com),

Please take another look.

[adg, 2011-12-12 05:14:11]

LGTM

http://codereview.appspot.com/5477056/diff/2002/doc/go1.html
File doc/go1.html (right):

http://codereview.appspot.com/5477056/diff/2002/doc/go1.html#newcode731
doc/go1.html:731: standard Go 1 release distributions, although they will be
available in source code form for
How will we make them available? By checking them out from the trunk? Should
this be explained?

[r, 2011-12-12 17:58:05]

Hello golang-dev@googlegroups.com, fullung@gmail.com, dsymonds@golang.org,
r@google.com, adg@golang.org (cc: golang-dev@googlegroups.com),

Please take another look.

[r, 2011-12-12 20:27:01]

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

doc/go1: the simpler package changes

R=golang-dev, fullung, dsymonds, r, adg
CC=golang-dev
http://codereview.appspot.com/5477056