Skip to content

Commit 22481b0

Browse files
author
Bryan C. Mills
committed
dashboard: update host-ios-arm64-corellium-ios to match its previous env
In CL 353549 I refactored environment-handling in the buildlet such that an explicit per-host GOROOT_BOOTSTRAP environment configured in dashboard/builders.go overrides the latent GOROOT_BOOTSTRAP value on the host. (The entry in dashboard/builders.go may omit GOROOT_BOOTSTRAP entirely to defer to the host environment.) That change caught out a previously-unnoticed configuration mismatch for the host-ios-arm64-corellium-ios builder: its entry in dashboard/builders.go indicated "GOROOT_BOOTSTRAP=/var/mobile/go-ios-arm64-bootstrap", but (as shown in previous builder logs¹) it was actually using "GOROOT_BOOTSTRAP=/var/root/go-ios-arm64-bootstrap". The latter empirically exists on the builder, while the former does not. ¹https://build.golang.org/log/4da918256e30f3442be7885019dc2d17bc718e0d For golang/go#33598. Change-Id: I09e5f83b46f23d5a4744c752fb66fcd4b33e4cff Reviewed-on: https://go-review.googlesource.com/c/build/+/354752 Trust: Bryan C. Mills <[email protected]> Run-TryBot: Bryan C. Mills <[email protected]> TryBot-Result: Go Bot <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]>
1 parent ecfecf7 commit 22481b0

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

dashboard/builders.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -623,7 +623,7 @@ var Hosts = map[string]*HostConfig{
623623
IsReverse: true,
624624
ExpectNum: 3,
625625
env: []string{
626-
"GOROOT_BOOTSTRAP=/var/mobile/go-ios-arm64-bootstrap",
626+
"GOROOT_BOOTSTRAP=/var/root/go-ios-arm64-bootstrap",
627627
},
628628
},
629629
"host-android-arm64-corellium-android": &HostConfig{

0 commit comments

Comments
 (0)