]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: expand TestGdbPythonCgo skip to include mips64le
authorBryan C. Mills <bcmills@google.com>
Fri, 7 Jan 2022 21:35:31 +0000 (16:35 -0500)
committerBryan Mills <bcmills@google.com>
Mon, 10 Jan 2022 17:06:19 +0000 (17:06 +0000)
The failure mode in #37794 does not match the failure mode described
in #18784. However, since the test is currently skipped on all other
MIPS variants, it may be that they suffer from the same underlying GDB
bug. Ideally one of the Go MIPS maintainers should file an upstream
bug and remove the skip once it is fixed; in the meantime, there is no
point in continuing to let the test fail on just one of the four MIPS
variants.

For #37794

Change-Id: I570f51cc04cbb7ef1ed7efd526e26886af53bfb6
Reviewed-on: https://go-review.googlesource.com/c/go/+/376654
Trust: Bryan Mills <bcmills@google.com>
Run-TryBot: Bryan Mills <bcmills@google.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>

src/runtime/runtime-gdb_test.go

index 63d3160ee47716196e748d990cc4518ddfd88fc6..7e8723e15f85a67174f616f8e8a34c0728654e1c 100644 (file)
@@ -153,8 +153,8 @@ func TestGdbPython(t *testing.T) {
 }
 
 func TestGdbPythonCgo(t *testing.T) {
-       if runtime.GOARCH == "mips" || runtime.GOARCH == "mipsle" || runtime.GOARCH == "mips64" {
-               testenv.SkipFlaky(t, 18784)
+       if strings.HasPrefix(runtime.GOARCH, "mips") {
+               testenv.SkipFlaky(t, 37794)
        }
        testGdbPython(t, true)
 }