diff options
author | Josh Bleecher Snyder <josharian@gmail.com> | 2017-03-30 15:07:05 -0700 |
---|---|---|
committer | Austin Clements <austin@google.com> | 2017-04-05 18:11:34 +0000 |
commit | d7989b784e685eb51af1601af0f8a91d3bd53e5d (patch) | |
tree | 28a7bb3034d6b611cad60830bc12d64dd963ff80 | |
parent | 056be9f79c37076e1a75639163016182b8b31843 (diff) | |
download | go-d7989b784e685eb51af1601af0f8a91d3bd53e5d.tar.gz go-d7989b784e685eb51af1601af0f8a91d3bd53e5d.zip |
[release-branch.go1.8] cmd/link: skip TestDWARF when cgo is disabled
While we're here, fix a Skip/Skipf error I noticed.
Fixes #19796.
(This fixes failures on the release branch introduced by cherry-pick
CL 39605.)
Change-Id: I59b1f5b5ea727fc314acfee8445b3de0b5af1e46
Reviewed-on: https://go-review.googlesource.com/39612
Run-TryBot: Austin Clements <austin@google.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
-rw-r--r-- | src/cmd/link/dwarf_test.go | 1 | ||||
-rw-r--r-- | src/cmd/link/linkbig_test.go | 2 | ||||
-rw-r--r-- | src/internal/testenv/testenv.go | 9 | ||||
-rw-r--r-- | src/internal/testenv/testenv_cgo.go | 11 |
4 files changed, 22 insertions, 1 deletions
diff --git a/src/cmd/link/dwarf_test.go b/src/cmd/link/dwarf_test.go index f10eaedaf5..32fa3a32a3 100644 --- a/src/cmd/link/dwarf_test.go +++ b/src/cmd/link/dwarf_test.go @@ -24,6 +24,7 @@ func TestDWARF(t *testing.T) { t.Skip("DWARF is not supported on Windows") } + 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 d793c2f5f2..960d89fd21 100644 --- 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 10384b6206..f7c4ad2682 100644 --- a/src/internal/testenv/testenv.go +++ b/src/internal/testenv/testenv.go @@ -138,6 +138,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 index 0000000000..e3d4d16b33 --- /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 +} |