]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/dist: do not run on ppc64le
authorRuss Cox <rsc@golang.org>
Fri, 5 Aug 2022 18:37:34 +0000 (14:37 -0400)
committerGopher Robot <gobot@golang.org>
Fri, 5 Aug 2022 19:51:08 +0000 (19:51 +0000)
Also don't run on all the other tiny slow boxes we have.
Should fix the remaining ppc64le broken builders.

For #44505 (or at least somehow provoked by it).

Change-Id: I72e8086cb641f3f7da3a872140a752bf328eec1f
Reviewed-on: https://go-review.googlesource.com/c/go/+/421438
Reviewed-by: Heschi Kreinick <heschi@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Russ Cox <rsc@golang.org>
Run-TryBot: Russ Cox <rsc@golang.org>

src/cmd/dist/test.go

index da5b17980bcece2b44eb31ead125db8dd4cb85bc..536a214773fe526ac7f5f3aa68d67486de1de76f 100644 (file)
@@ -866,12 +866,12 @@ func (t *tester) registerTests() {
                        })
                }
        }
-       // Only run the API check on fast development platforms. Android, iOS, and JS
-       // are always cross-compiled, and the filesystems on our only plan9 builders
-       // are too slow to complete in a reasonable timeframe. Every platform checks
-       // the API on every GOOS/GOARCH/CGO_ENABLED combination anyway, so we really
-       // only need to run this check once anywhere to get adequate coverage.
-       if goos != "android" && !t.iOS() && goos != "js" && goos != "plan9" {
+       // Only run the API check on fast development platforms.
+       // Every platform checks the API on every GOOS/GOARCH/CGO_ENABLED combination anyway,
+       // so we really only need to run this check once anywhere to get adequate coverage.
+       // To help developers avoid trybot-only failures, we try to run on typical developer machines
+       // which is darwin/linux/windows and amd64/arm64.
+       if (goos == "darwin" || goos == "linux" || goos == "windows") && (goarch == "amd64" || goarch == "arm64") {
                t.tests = append(t.tests, distTest{
                        name:    "api",
                        heading: "API check",