]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: don't run TestEnsureDropM on windows or plan9
authorIan Lance Taylor <iant@golang.org>
Mon, 18 Jan 2016 17:01:48 +0000 (09:01 -0800)
committerIan Lance Taylor <iant@golang.org>
Mon, 18 Jan 2016 18:40:45 +0000 (18:40 +0000)
This is testing code in asm_GOARCH.s, so it's not necessary to run the
test on systems where it doesn't build.

Fixes #13991.

Change-Id: Ia7a2d3a34b32e6987dc67428c1e09e63baf0518a
Reviewed-on: https://go-review.googlesource.com/18707
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/runtime/crash_cgo_test.go
src/runtime/testdata/testprogcgo/dropm.go

index 92b4f0ca71c921173df66f02c037e76b0fbe3fa1..d7b367f941f4a35e89a8d77828e105e2757dd97a 100644 (file)
@@ -137,6 +137,10 @@ func TestCgoExecSignalMask(t *testing.T) {
 
 func TestEnsureDropM(t *testing.T) {
        // Test for issue 13881.
+       switch runtime.GOOS {
+       case "windows", "plan9":
+               t.Skipf("skipping dropm test on %s", runtime.GOOS)
+       }
        got := runTestProg(t, "testprogcgo", "EnsureDropM")
        want := "OK\n"
        if got != want {
index 80ccdcc60832de78f96b80fe7af8a4da6d798efa..75984ea75f3c2684e169a58a71d250ff8b16b8a8 100644 (file)
@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style
 // license that can be found in the LICENSE file.
 
+// +build !plan9,!windows
+
 // Test that a sequence of callbacks from C to Go get the same m.
 // This failed to be true on arm and arm64, which was the root cause
 // of issue 13881.