code review 5919048: cmd/go: update for go1 tag format

cmd/go: update for go1 tag format

[rsc, 2012-03-27 03:52:02]

Hello golang-dev@googlegroups.com,

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

[r, 2012-03-27 03:56:53]

http://codereview.appspot.com/5919048/diff/4001/src/cmd/go/get.go
File src/cmd/go/get.go (right):

http://codereview.appspot.com/5919048/diff/4001/src/cmd/go/get.go#newcode327
src/cmd/go/get.go:327: // goTag matches go release tags such as go.1 and
go.1.2.3.
this comment doesn't match the regexp. there's no period after "go" in the
regexp

[rsc, 2012-03-27 03:59:04]

PTAL

Grepped for 'go\.' and found a comment in vcs.go that
needed fixing too.  Obviously I used to think it was go.1,
but since we committed to go1 earlier today, I've been
sticking to that.

[r, 2012-03-27 04:04:49]

http://codereview.appspot.com/5919048/diff/5003/src/cmd/go/get.go
File src/cmd/go/get.go (right):

http://codereview.appspot.com/5919048/diff/5003/src/cmd/go/get.go#newcode337
src/cmd/go/get.go:337: // Version "go.N[.N[.N]]" matches tags of the form
"go.N[.N[.N]]".
goN[.N] etc.

http://codereview.appspot.com/5919048/diff/5003/src/cmd/go/get.go#newcode338
src/cmd/go/get.go:338: // Version "release.rN" matches tags of the form "go.rN"
(N being a decimal).
ditto. i think. (actually i'm totally confused now)

[rsc, 2012-03-27 04:08:59]

how about

// Version "goX" (or "goX.Y" or "goX.Y.Z") matches tags of the same form.
// Version "release.rN" matches tags of the form "go.rN" (N being a
floating-point number).

[r2, 2012-03-27 04:10:29]

LGTM

code looks given given the regexp, it's just the comments that seemed incorrect

[rsc, 2012-03-27 04:11:04]

After Go 1 we can throw away the release.* code.

[rsc, 2012-03-27 04:17:54]

*** Submitted as 515964d0c89b ***

cmd/go: update for go1 tag format

R=golang-dev, r, r
CC=golang-dev
http://codereview.appspot.com/5919048