code review 176530043: tools/cmd/vet: check that cgo is enabled before testing it

tools/cmd/vet: check that cgo is enabled before testing it
Should fix the tools builders.

[r, 2014-12-01 23:00:34]

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

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

[bradfitz, 2014-12-01 23:03:27]

LGTM
 On Dec 1, 2014 3:00 PM, <r@golang.org> wrote:

> Reviewers: rsc,
>
> Message:
> Hello rsc (cc: golang-codereviews@googlegroups.com),
>
> I'd like you to review this change to
> https://code.google.com/p/go.tools
>
>
> Description:
> tools/cmd/vet: check that cgo is enabled before testing it
> Should fix the tools builders.
>
> Please review this at https://codereview.appspot.com/176530043/
>
> Affected files (+5, -0 lines):
>   M cmd/stringer/endtoend_test.go
>
>
> Index: cmd/stringer/endtoend_test.go
> ===================================================================
> --- a/cmd/stringer/endtoend_test.go
> +++ b/cmd/stringer/endtoend_test.go
> @@ -6,6 +6,7 @@
>
>  import (
>         "fmt"
> +       "go/build"
>         "io"
>         "io/ioutil"
>         "os"
> @@ -45,6 +46,10 @@
>                         t.Errorf("%s is not a Go file", name)
>                         continue
>                 }
> +               if name == "cgo.go" && !build.Default.CgoEnabled {
> +                       t.Logf("cgo is no enabled for %s", name)
> +                       continue
> +               }
>                 // Names are known to be ASCII and long enough.
>                 typeName := fmt.Sprintf("%c%s", name[0]+'A'-'a',
> name[1:len(name)-len(".go")])
>                 stringerCompileAndRun(t, dir, stringer, typeName, name)
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "golang-codereviews" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to golang-codereviews+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>

[r, 2014-12-01 23:15:31]

*** Submitted as
https://code.google.com/p/go/source/detail?r=c3d959302c8a&repo=tools ***

tools/cmd/vet: check that cgo is enabled before testing it
Should fix the tools builders.

LGTM=bradfitz
R=rsc, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/176530043