code review 9496043: cmd/vet: delete; it now lives in the go.tools subrepo

cmd/vet: delete; it now lives in the go.tools subrepo

[r, 2013-05-17 20:04:22]

Hello golang-dev@googlegroups.com,

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

[dsymonds, 2013-05-17 20:09:42]

Should the dir be "govet", so installing it with the go tool will use the
correct binary name?

[rsc, 2013-05-17 20:14:18]

I don't see what this has to do with the CL at hand, but the dir (in the
other repo) should continue to be vet, and whatever we use to build the
full distribution (go tool or some other script) should know where to put
it.

Russ

[iant, 2013-05-17 20:32:39]

https://codereview.appspot.com/9496043/diff/1/test/bench/shootout/timing.log
File test/bench/shootout/timing.log (right):

https://codereview.appspot.com/9496043/diff/1/test/bench/shootout/timing.log#...
test/bench/shootout/timing.log:6: # sudo apt-get install libpcre3-dev
This seems like an unrelated change.

[r, 2013-05-17 20:33:29]

Hello golang-dev@googlegroups.com, dsymonds@golang.org, rsc@golang.org,
iant@golang.org (cc: golang-dev@googlegroups.com),

Please take another look.

[r, 2013-05-17 20:33:49]

indeed it was. i had a CL to hold that hostage but the CL is gone. i'm having a
bad hg day.

[dave_cheney.net, 2013-05-17 20:42:59]

Now there is a new subrepo, can someone with permissions please
redeploy build.golang.org to cover it.

On Sat, May 18, 2013 at 6:33 AM,  <r@golang.org> wrote:
> indeed it was. i had a CL to hold that hostage but the CL is gone. i'm
> having a bad hg day.
>
>
>
> https://codereview.appspot.com/9496043/
>
> --
>
> ---You received this message because you are subscribed to the Google Groups
> "golang-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to golang-dev+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/groups/opt_out.
>
>

[iant, 2013-05-17 20:48:29]

LGTM

[r, 2013-05-17 20:53:26]

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

cmd/vet: delete; it now lives in the go.tools subrepo

R=golang-dev, dsymonds, rsc, iant, dave
CC=golang-dev
https://codereview.appspot.com/9496043