code review 9611048: cmd/go: support new location for vet

cmd/go: support new location for vet
Also delete the special case for exp, which isn't necessary any more.

Fixes issue 5529.

[r, 2013-05-21 22:05:35]

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

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

[ioe, 2013-05-21 22:23:46]

Could you confirm, that one can now build, package and install Go into to 
system providing a proper goroot.

LATER, go get  code.google.com/p/go.tools/cmd/vet, which builds vet.

Then package vet from from that build, install it into the proper system 
goroot and it will all work?

So it will work like any other packaging done for go, just that now the 
binary is installed in the goroot instead of e.g. /usr/bin

If it now works that way: Yes and big thanks! :-)

On Wednesday, May 22, 2013 12:05:35 AM UTC+2, r...@golang.org wrote:
>
> Reviewers: rsc, 
>
> Message: 
> Hello rsc (cc: golan...@googlegroups.com <javascript:>), 
>
> I'd like you to review this change to 
> https://code.google.com/p/go/ 
>
>
> Description: 
> cmd/go: support new location for vet 
>

[r, 2013-05-21 22:35:05]

I'm not sure what you're asking, but I think it's what this CL does:
install vet in the tool directory as it has always has been, but built
from the new location.

-rob

[rsc, 2013-05-21 23:43:06]

LGTM

[r, 2013-05-22 04:18:18]

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

cmd/go: support new location for vet
Also delete the special case for exp, which isn't necessary any more.

Fixes issue 5529.

R=rsc, nightlyone
CC=golang-dev
https://codereview.appspot.com/9611048