dashboard: remove iostest special case
There are no tethered iOS builders left. If we add some in the future,
they should use all.bash directly.
Change-Id: Ieef7454af706c7e99c0d10fc37a337fb2c3ba130
Reviewed-on: https://go-review.googlesource.com/c/build/+/259357
Run-TryBot: Elias Naur <mail@eliasnaur.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Trust: Elias Naur <mail@eliasnaur.com>
diff --git a/dashboard/builders.go b/dashboard/builders.go
index 2bd35fe..a406614 100644
--- a/dashboard/builders.go
+++ b/dashboard/builders.go
@@ -1063,9 +1063,6 @@
if strings.HasPrefix(c.Name, "nacl-") {
return "src/nacltest.bash"
}
- if strings.HasPrefix(c.Name, "darwin-arm") && !strings.Contains(c.Name, "corellium") {
- return "src/iostest.bash"
- }
if strings.HasPrefix(c.Name, "misc-compile") {
return "src/buildall.bash"
}
@@ -1087,10 +1084,8 @@
// These we've verified to work.
return true
}
- // TODO(bradfitz): make iostest.bash work too. And
- // buildall.bash should really just be N small container jobs
- // instead of a "buildall.bash". Then we can delete this whole
- // method.
+ // TODO(bradfitz): buildall.bash should really just be N small container
+ // jobs instead of a "buildall.bash". Then we can delete this whole method.
return false
}
@@ -1246,9 +1241,6 @@
// AllScriptArgs returns the set of arguments that should be passed to the
// all.bash-equivalent script. Usually empty.
func (c *BuildConfig) AllScriptArgs() []string {
- if strings.HasPrefix(c.Name, "darwin-arm") && !strings.Contains(c.Name, "corellium") {
- return []string{"-restart"}
- }
return append([]string(nil), c.allScriptArgs...)
}
