]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/dist: skip -race w/ external linkage on windows 2008
authorThan McIntosh <thanm@google.com>
Tue, 22 Nov 2022 15:42:25 +0000 (10:42 -0500)
committerGopher Robot <gobot@golang.org>
Tue, 22 Nov 2022 16:07:28 +0000 (16:07 +0000)
Add a skip for the external-linkage part of the race detector test for
elderly versions of windows, until the underlying cause for the problem
can be determined.

Updates #56904.

Change-Id: I3e8650ff66f34efefabcd6bc343d57124539901b
Reviewed-on: https://go-review.googlesource.com/c/go/+/452763
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Than McIntosh <thanm@google.com>
Auto-Submit: Than McIntosh <thanm@google.com>

src/cmd/dist/test.go

index eadca9c08feb538da282b38e68d0cffde46935e4..316a9b9dddfbb2d157b5b138cb92caae83325ef6 100644 (file)
@@ -1602,14 +1602,18 @@ func (t *tester) registerRaceTests() {
                // t.registerTest("race:misc/cgo/test", hdr, &goTest{dir: "../misc/cgo/test", race: true, env: []string{"GOTRACEBACK=2"}})
        }
        if t.extLink() {
-               // Test with external linking; see issue 9133.
-               t.registerTest("race:external", hdr,
-                       &goTest{
-                               race:     true,
-                               ldflags:  "-linkmode=external",
-                               runTests: "TestParse|TestEcho|TestStdinCloseRace",
-                               pkgs:     []string{"flag", "os/exec"},
-                       })
+               if strings.HasPrefix(os.Getenv("GO_BUILDER_NAME"), "windows-amd64-2008") {
+                       log.Printf("skipping -race with external linkage on older windows builder, see https://github.com/golang/go/issues/56904 for details")
+               } else {
+                       // Test with external linking; see issue 9133.
+                       t.registerTest("race:external", hdr,
+                               &goTest{
+                                       race:     true,
+                                       ldflags:  "-linkmode=external",
+                                       runTests: "TestParse|TestEcho|TestStdinCloseRace",
+                                       pkgs:     []string{"flag", "os/exec"},
+                               })
+               }
        }
 }