diff options
| author | Ian Lance Taylor <iant@golang.org> | 2023-01-31 13:55:19 -0800 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2023-02-01 10:06:08 +0000 |
| commit | 56a14ad4bc19d5ee9d4257f370a570377e81e544 (patch) | |
| tree | e69d2bef76c350beadcca04bb5d64ec28d71e2ad /src/cmd/dist/test.go | |
| parent | 23c0121e4eb259cc1087d0f79a0803cbc71f500b (diff) | |
| download | go-56a14ad4bc19d5ee9d4257f370a570377e81e544.tar.gz go-56a14ad4bc19d5ee9d4257f370a570377e81e544.zip | |
cmd/dist, internal/platform: remove some target modes that fail
Before CL 463992 there were some cases that cmd/dist did not test
but that platform.BuildModeSupport permitted. In CL 463992 those
conflicts were resolved in favor of platform.BuildModeSupport.
However, further testing has uncovered some cases that do not in
fact work. Adjust those in both cmd/dist and internal/platform.
In particular, mark android-arm and android-arm64 as not supporting
plugin mode. Sample failure:
https://build.golang.org/log/ebba858ea9f94f076966d8cfd42348a0e9345095
Mark ios as not supporting c-archive mode. Sample failure:
https://build.golang.org/log/e78a58189d94b90dc6d4b2e01a1b4a0b35d63792
Fixes #58170
Fixes #58172
Change-Id: Ic1bd18e36382cac0779aad48cb6e7b1de8eeb10d
Reviewed-on: https://go-review.googlesource.com/c/go/+/464339
Reviewed-by: Bryan Mills <bcmills@google.com>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
Auto-Submit: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Ian Lance Taylor <iant@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Diffstat (limited to 'src/cmd/dist/test.go')
| -rw-r--r-- | src/cmd/dist/test.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/cmd/dist/test.go b/src/cmd/dist/test.go index dcb6881171..b540984e38 100644 --- a/src/cmd/dist/test.go +++ b/src/cmd/dist/test.go @@ -1143,7 +1143,7 @@ func (t *tester) supportedBuildmode(mode string) bool { return false } switch goos { - case "aix", "darwin", "ios", "windows": + case "aix", "darwin", "windows": return true case "linux": switch goarch { @@ -1182,7 +1182,7 @@ func (t *tester) supportedBuildmode(mode string) bool { switch pair { case "linux-386", "linux-amd64", "linux-arm", "linux-arm64", "linux-s390x", "linux-ppc64le": return true - case "android-386", "android-amd64", "android-arm", "android-arm64": + case "android-386", "android-amd64": return true case "darwin-amd64", "darwin-arm64": return true |