code review 66130043: cmd/pack: add 'c' command to create archive

cmd/pack: add 'c' command to create archive

When Go 1.3 is released, this will keep existing
Go 1.2 build scripts that use 'go tool pack grc' working.
For efficiency, such scripts should be changed to
use 6g -pack instead, but keeping the old behavior
available enables a more graceful transition.

[rsc, 2014-02-19 20:32:06]

Hello r (cc: golang-codereviews@googlegroups.com),

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

[r, 2014-02-19 21:36:05]

LGTM

https://codereview.appspot.com/66130043/diff/20001/src/cmd/pack/pack.go
File src/cmd/pack/pack.go (right):

https://codereview.appspot.com/66130043/diff/20001/src/cmd/pack/pack.go#newco...
src/cmd/pack/pack.go:38: Where op is one of cprtx optionally followed by v for
verbose output.
For compatibility with old Go build environments, the op string grc is accepted
as a synonym for c.

https://codereview.appspot.com/66130043/diff/20001/src/cmd/pack/pack.go#newco...
src/cmd/pack/pack.go:362: }
ar.pad = 0
(not necessary, i guess, but do it anyway)

[rsc, 2014-02-19 22:08:29]

done

[rsc, 2014-02-19 22:08:47]

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

cmd/pack: add 'c' command to create archive

When Go 1.3 is released, this will keep existing
Go 1.2 build scripts that use 'go tool pack grc' working.
For efficiency, such scripts should be changed to
use 6g -pack instead, but keeping the old behavior
available enables a more graceful transition.

LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/66130043

[gobot, 2014-02-19 22:34:05]

This CL appears to have broken the plan9-386-cnielsen builder.