Commit 754e98cb authored by Rahul Chaudhry's avatar Rahul Chaudhry Committed by Hyang-Ah Hana Kim

cmd/dist: de-dup iOS detection

Change-Id: I89778988baec1cf4a35d9342c7dbe8c4c08ff3cd
Reviewed-on: https://go-review.googlesource.com/9893
Run-TryBot: Hyang-Ah Hana Kim <hyangah@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: default avatarHyang-Ah Hana Kim <hyangah@gmail.com>
parent b1d144e1
...@@ -244,9 +244,7 @@ func (t *tester) registerTests() { ...@@ -244,9 +244,7 @@ func (t *tester) registerTests() {
}, },
}) })
iOS := t.goos == "darwin" && (t.goarch == "arm" || t.goarch == "arm64") if t.cgoEnabled && t.goos != "android" && !t.iOS() {
if t.cgoEnabled && t.goos != "android" && !iOS {
// Disabled on android and iOS. golang.org/issue/8345 // Disabled on android and iOS. golang.org/issue/8345
t.tests = append(t.tests, distTest{ t.tests = append(t.tests, distTest{
name: "cgo_stdio", name: "cgo_stdio",
...@@ -265,7 +263,7 @@ func (t *tester) registerTests() { ...@@ -265,7 +263,7 @@ func (t *tester) registerTests() {
}, },
}) })
} }
if t.cgoEnabled && t.goos != "android" && !iOS { if t.cgoEnabled && t.goos != "android" && !t.iOS() {
// TODO(crawshaw): reenable on android and iOS // TODO(crawshaw): reenable on android and iOS
// golang.org/issue/8345 // golang.org/issue/8345
// //
...@@ -295,7 +293,7 @@ func (t *tester) registerTests() { ...@@ -295,7 +293,7 @@ func (t *tester) registerTests() {
heading: "../misc/cgo/testso", heading: "../misc/cgo/testso",
fn: t.cgoTestSOWindows, fn: t.cgoTestSOWindows,
}) })
} else if t.hasBash() && t.goos != "android" && !iOS { } else if t.hasBash() && t.goos != "android" && !t.iOS() {
t.registerTest("testso", "../misc/cgo/testso", "./test.bash") t.registerTest("testso", "../misc/cgo/testso", "./test.bash")
} }
if t.supportedBuildmode("c-archive") { if t.supportedBuildmode("c-archive") {
...@@ -310,23 +308,23 @@ func (t *tester) registerTests() { ...@@ -310,23 +308,23 @@ func (t *tester) registerTests() {
if t.gohostos == "linux" && t.goarch == "amd64" { if t.gohostos == "linux" && t.goarch == "amd64" {
t.registerTest("testasan", "../misc/cgo/testasan", "go", "run", "main.go") t.registerTest("testasan", "../misc/cgo/testasan", "go", "run", "main.go")
} }
if t.hasBash() && t.goos != "android" && !iOS && t.gohostos != "windows" { if t.hasBash() && t.goos != "android" && !t.iOS() && t.gohostos != "windows" {
t.registerTest("cgo_errors", "../misc/cgo/errors", "./test.bash") t.registerTest("cgo_errors", "../misc/cgo/errors", "./test.bash")
} }
if t.gohostos == "linux" && t.extLink() { if t.gohostos == "linux" && t.extLink() {
t.registerTest("testsigfwd", "../misc/cgo/testsigfwd", "go", "run", "main.go") t.registerTest("testsigfwd", "../misc/cgo/testsigfwd", "go", "run", "main.go")
} }
} }
if t.hasBash() && t.goos != "nacl" && t.goos != "android" && !iOS { if t.hasBash() && t.goos != "nacl" && t.goos != "android" && !t.iOS() {
t.registerTest("doc_progs", "../doc/progs", "time", "go", "run", "run.go") t.registerTest("doc_progs", "../doc/progs", "time", "go", "run", "run.go")
t.registerTest("wiki", "../doc/articles/wiki", "./test.bash") t.registerTest("wiki", "../doc/articles/wiki", "./test.bash")
t.registerTest("codewalk", "../doc/codewalk", "time", "./run") t.registerTest("codewalk", "../doc/codewalk", "time", "./run")
t.registerTest("shootout", "../test/bench/shootout", "time", "./timing.sh", "-test") t.registerTest("shootout", "../test/bench/shootout", "time", "./timing.sh", "-test")
} }
if t.goos != "android" && !iOS { if t.goos != "android" && !t.iOS() {
t.registerTest("bench_go1", "../test/bench/go1", "go", "test") t.registerTest("bench_go1", "../test/bench/go1", "go", "test")
} }
if t.goos != "android" && !iOS { if t.goos != "android" && !t.iOS() {
// TODO(bradfitz): shard down into these tests, as // TODO(bradfitz): shard down into these tests, as
// this is one of the slowest (and most shardable) // this is one of the slowest (and most shardable)
// tests. // tests.
...@@ -336,7 +334,7 @@ func (t *tester) registerTests() { ...@@ -336,7 +334,7 @@ func (t *tester) registerTests() {
fn: t.testDirTest, fn: t.testDirTest,
}) })
} }
if t.goos != "nacl" && t.goos != "android" && !iOS { if t.goos != "nacl" && t.goos != "android" && !t.iOS() {
t.tests = append(t.tests, distTest{ t.tests = append(t.tests, distTest{
name: "api", name: "api",
heading: "API check", heading: "API check",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment