cmd/link: skip TestDWARF when cgo is disabled

While we're here, fix a Skip/Skipf error I noticed.

Fixes #19796.

Change-Id: I59b1f5b5ea727fc314acfee8445b3de0b5af1e46
Reviewed-on: https://go-review.googlesource.com/38992
Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>

diff --git a/src/cmd/link/dwarf_test.go b/src/cmd/link/dwarf_test.go
index 4c7e2e42326066a41f503414c1ec578d4fdb4e2b..f88aecc7c7ecd3c8ea475be94ac6e7198068e935 100644 (file)
--- a/src/cmd/link/dwarf_test.go
+++ b/src/cmd/link/dwarf_test.go
@@ -20,6 +20,7 @@ import (
 )
 
 func TestDWARF(t *testing.T) {
+       testenv.MustHaveCGO(t)
        testenv.MustHaveGoBuild(t)
 
        if runtime.GOOS == "plan9" {

diff --git a/src/cmd/link/linkbig_test.go b/src/cmd/link/linkbig_test.go
index d793c2f5f24cd34da754d455e5d63d5dc1564f4e..960d89fd21f9543d55f03fa329fe2d269a071a9d 100644 (file)
--- a/src/cmd/link/linkbig_test.go
+++ b/src/cmd/link/linkbig_test.go
@@ -21,7 +21,7 @@ import (
 
 func TestLargeText(t *testing.T) {
        if testing.Short() || (obj.GOARCH != "ppc64le" && obj.GOARCH != "ppc64" && obj.GOARCH != "arm") {
-               t.Skip("Skipping large text section test in short mode or on %s", obj.GOARCH)
+               t.Skipf("Skipping large text section test in short mode or on %s", obj.GOARCH)
        }
        testenv.MustHaveGoBuild(t)
 

diff --git a/src/internal/testenv/testenv.go b/src/internal/testenv/testenv.go
index 4cd8a2b541ec4f5653581b4f4bcf20f559b032d6..1a13ac3f2b51e88efa50e6e559979a4048c0cdfe 100644 (file)
--- a/src/internal/testenv/testenv.go
+++ b/src/internal/testenv/testenv.go
@@ -151,6 +151,15 @@ func MustHaveExternalNetwork(t *testing.T) {
        }
 }
 
+var haveCGO bool
+
+// MustHaveCGO calls t.Skip if cgo is not available.
+func MustHaveCGO(t *testing.T) {
+       if !haveCGO {
+               t.Skipf("skipping test: no cgo")
+       }
+}
+
 // HasSymlink reports whether the current system can use os.Symlink.
 func HasSymlink() bool {
        ok, _ := hasSymlink()

diff --git a/src/internal/testenv/testenv_cgo.go b/src/internal/testenv/testenv_cgo.go
new file mode 100644 (file)
index 0000000..e3d4d16
--- /dev/null
+++ b/src/internal/testenv/testenv_cgo.go
@@ -0,0 +1,11 @@
+// Copyright 2017 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+// +build cgo
+
+package testenv
+
+func init() {
+       haveCGO = true
+}