code review 13250047: cmd/go: implement -x correctly for 'go vet', 'go fmt', ...

cmd/go: implement -x correctly for 'go vet', 'go fmt', and so on

Fixes issue 5676.

[rsc, 2013-09-10 16:24:28]

Hello golang-dev@googlegroups.com,

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

[bradfitz, 2013-09-10 16:28:40]

LGTM
 On Sep 10, 2013 9:24 AM, <rsc@golang.org> wrote:

> Reviewers: golang-dev1,
>
> Message:
> Hello golang-dev@googlegroups.com,
>
> I'd like you to review this change to
> https://code.google.com/p/go/
>
>
> Description:
> cmd/go: implement -x correctly for 'go vet', 'go fmt', and so on
>
> Fixes issue 5676.
>
> Please review this at
https://codereview.appspot.**com/13250047/<https://codereview.appspot.com/132...
>
> Affected files (+1, -1 lines):
>   M src/cmd/go/main.go
>
>
> Index: src/cmd/go/main.go
> ==============================**==============================**=======
> --- a/src/cmd/go/main.go
> +++ b/src/cmd/go/main.go
> @@ -360,7 +360,7 @@
>
>  func run(cmdargs ...interface{}) {
>         cmdline := stringList(cmdargs...)
> -       if buildN || buildV {
> +       if buildN || buildX {
>                 fmt.Printf("%s\n", strings.Join(cmdline, " "))
>                 if buildN {
>                         return
>
>
> --
>
> ---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<golang-dev%2Bunsubscribe@googlegrou...
> .
> For more options, visit
https://groups.google.com/**groups/opt_out<https://groups.google.com/groups/o...
> .
>

[rsc, 2013-09-10 16:47:55]

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

cmd/go: implement -x correctly for 'go vet', 'go fmt', and so on

Fixes issue 5676.

R=golang-dev, bradfitz
CC=golang-dev
https://codereview.appspot.com/13250047