From: Elias Naur <mail@eliasnaur.com>
Date: Tue, 30 Apr 2019 18:00:23 +0000 (+0200)
Subject: cmd/dist: detect GOHOSTARCH on iOS
X-Git-Tag: go1.13beta1~478
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=888bac1421d098b6cbe58d4fde5d6ba516833f23;p=gostls13.git

cmd/dist: detect GOHOSTARCH on iOS

cmd/dist defaults to GOHOSTARCH=amd64 on darwin because no other
darwin host could build Go. With the upcoming self-hosted iOS
builders, GOHOSTARCH=arm64 is also possible.

Updates #31722

Change-Id: I9af47d9f8c57ea45475ce498acefbfe6bf4815b9
Reviewed-on: https://go-review.googlesource.com/c/go/+/174306
Run-TryBot: Elias Naur <mail@eliasnaur.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
---

diff --git a/src/cmd/dist/main.go b/src/cmd/dist/main.go
index 7a7ffe8f0c..07361d828f 100644
--- a/src/cmd/dist/main.go
+++ b/src/cmd/dist/main.go
@@ -60,9 +60,6 @@ func main() {
 		// uname -m doesn't work under AIX
 		gohostarch = "ppc64"
 	case "darwin":
-		// Even on 64-bit platform, darwin uname -m prints i386.
-		// We don't support any of the OS X versions that run on 32-bit-only hardware anymore.
-		gohostarch = "amd64"
 		// macOS 10.9 and later require clang
 		defaultclang = true
 	case "freebsd":
@@ -107,6 +104,11 @@ func main() {
 			gohostarch = "amd64"
 		case strings.Contains(out, "86"):
 			gohostarch = "386"
+			if gohostos == "darwin" {
+				// Even on 64-bit platform, some versions of macOS uname -m prints i386.
+				// We don't support any of the OS X versions that run on 32-bit-only hardware anymore.
+				gohostarch = "amd64"
+			}
 		case strings.Contains(out, "aarch64"), strings.Contains(out, "arm64"):
 			gohostarch = "arm64"
 		case strings.Contains(out, "arm"):
@@ -128,8 +130,8 @@ func main() {
 		case strings.Contains(out, "s390x"):
 			gohostarch = "s390x"
 		case gohostos == "darwin":
-			if strings.Contains(run("", CheckExit, "uname", "-v"), "RELEASE_ARM_") {
-				gohostarch = "arm"
+			if strings.Contains(run("", CheckExit, "uname", "-v"), "RELEASE_ARM64_") {
+				gohostarch = "arm64"
 			}
 		default:
 			fatalf("unknown architecture: %s", out)