Skip to content

Commit f0fb708

Browse files
committed
dashboard: add windows- and darwin- amd64-regabi builders
This adds a builder for the GOEXPERIMENT=regabi configuration of the compiler. We'll want this builder until we enable regabi by default. Update comments and buildsRepo test, since it's now merged with master. Updates golang/go#40724. Change-Id: I424ca7bf6243e6b4660f4ec54c248a1813b3a83d Reviewed-on: https://go-review.googlesource.com/c/build/+/305849 Trust: David Chase <[email protected]> Run-TryBot: David Chase <[email protected]> TryBot-Result: Go Bot <[email protected]> Reviewed-by: Austin Clements <[email protected]>
1 parent 305e1e3 commit f0fb708

File tree

1 file changed

+39
-6
lines changed

1 file changed

+39
-6
lines changed

dashboard/builders.go

Lines changed: 39 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1687,12 +1687,45 @@ func init() {
16871687
HostType: "host-linux-buster",
16881688
Notes: "builder with GOEXPERIMENT=regabi, see golang.org/issue/40724",
16891689
buildsRepo: func(repo, branch, goBranch string) bool {
1690-
// Make sure we don't break things on master.
1691-
// The actual experiment is on dev.regabi
1692-
// (until we merge to master). And dev.regabi
1693-
// is being merged into dev.typeparams, so
1694-
// test that, too.
1695-
return repo == "go" && (branch == "master" || branch == "dev.regabi" || branch == "dev.typeparams")
1690+
return repo == "go" && branch == "master"
1691+
},
1692+
env: []string{
1693+
"GO_DISABLE_OUTBOUND_NETWORK=1",
1694+
"GOEXPERIMENT=regabi",
1695+
},
1696+
GoDeps: []string{
1697+
"89b44b4e2bb2f88474d6b8476f5c28ea2aea9b28", // A master commit from 2020/12/20, just before a merge into dev.regabi, before the GOEXPERIMENT is used anywhere
1698+
},
1699+
})
1700+
addBuilder(BuildConfig{
1701+
Name: "windows-amd64-regabi",
1702+
HostType: "host-windows-amd64-2016",
1703+
Notes: "builder with GOEXPERIMENT=regabi, see golang.org/issue/40724",
1704+
buildsRepo: func(repo, branch, goBranch string) bool {
1705+
return repo == "go" && branch == "master"
1706+
},
1707+
env: []string{
1708+
"GO_DISABLE_OUTBOUND_NETWORK=1",
1709+
"GOEXPERIMENT=regabi",
1710+
"GOARCH=amd64",
1711+
"GOHOSTARCH=amd64",
1712+
// cmd/go takes ~188 seconds on windows-amd64
1713+
// now, which is over the 180 second default
1714+
// dist test timeout. So, bump this builder
1715+
// up:
1716+
"GO_TEST_TIMEOUT_SCALE=2",
1717+
},
1718+
GoDeps: []string{
1719+
"89b44b4e2bb2f88474d6b8476f5c28ea2aea9b28", // A master commit from 2020/12/20, just before a merge into dev.regabi, before the GOEXPERIMENT is used anywhere
1720+
},
1721+
})
1722+
addBuilder(BuildConfig{
1723+
Name: "darwin-amd64-regabi",
1724+
HostType: "host-darwin-10_15",
1725+
distTestAdjust: macTestPolicy,
1726+
Notes: "builder with GOEXPERIMENT=regabi, see golang.org/issue/40724",
1727+
buildsRepo: func(repo, branch, goBranch string) bool {
1728+
return repo == "go" && branch == "master"
16961729
},
16971730
env: []string{
16981731
"GO_DISABLE_OUTBOUND_NETWORK=1",

0 commit comments

Comments
 (0)