Skip to content

Commit e47b0b2

Browse files
prattmicgopherbot
authored andcommitted
dashboard: remove needsGoProxy
This field was added in CL 166218 but never used. For golang/go#35678. Change-Id: I197fd3835136e7649b20a7ca47d4179591cb49a5 Reviewed-on: https://go-review.googlesource.com/c/build/+/482335 Auto-Submit: Michael Pratt <mpratt@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org> Run-TryBot: Michael Pratt <mpratt@google.com>
1 parent b88b71f commit e47b0b2

File tree

1 file changed

+0
-14
lines changed

1 file changed

+0
-14
lines changed

dashboard/builders.go

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -828,11 +828,6 @@ type BuildConfig struct {
828828
// storage.
829829
StopAfterMake bool
830830

831-
// needsGoProxy is whether this builder should have GOPROXY set.
832-
// Currently this is only for the longtest builder, which needs
833-
// to run cmd/go tests fetching from the network.
834-
needsGoProxy bool
835-
836831
// privateGoProxy for builder has it's own Go proxy instead of
837832
// proxy.golang.org, after setting this builder will respect
838833
// GOPROXY environment value.
@@ -1151,8 +1146,6 @@ func (c *BuildConfig) AllScript() string {
11511146

11521147
func (c *BuildConfig) IsTryOnly() bool { return c.tryOnly }
11531148

1154-
func (c *BuildConfig) NeedsGoProxy() bool { return c.needsGoProxy }
1155-
11561149
// PrivateGoProxy for builder has it's own Go proxy instead of proxy.golang.org
11571150
func (c *BuildConfig) PrivateGoProxy() bool { return c.privateGoProxy }
11581151

@@ -1920,7 +1913,6 @@ func init() {
19201913
// For golang.org/x repos, don't test non-latest versions.
19211914
return repo == "go" || (branch == "master" && goBranch == "master")
19221915
},
1923-
needsGoProxy: true, // for cmd/go module tests
19241916
env: []string{
19251917
"GO_TEST_TIMEOUT_SCALE=5", // give them lots of time
19261918
},
@@ -1938,7 +1930,6 @@ func init() {
19381930
}
19391931
return b
19401932
},
1941-
needsGoProxy: true, // for cmd/go module tests
19421933
env: []string{
19431934
"GO_TEST_TIMEOUT_SCALE=5", // Inherited from the longtest builder.
19441935
},
@@ -1960,7 +1951,6 @@ func init() {
19601951
}
19611952
return b
19621953
},
1963-
needsGoProxy: true, // for cmd/go module tests
19641954
env: []string{
19651955
"GOARCH=386",
19661956
"GOHOSTARCH=386",
@@ -2493,7 +2483,6 @@ func init() {
24932483
}
24942484
return b
24952485
},
2496-
needsGoProxy: true, // for cmd/go module tests
24972486
env: []string{
24982487
"GO_TEST_TIMEOUT_SCALE=5", // give them lots of time
24992488
},
@@ -2523,7 +2512,6 @@ func init() {
25232512
}
25242513
return b
25252514
},
2526-
needsGoProxy: true, // for cmd/go module tests
25272515
env: []string{
25282516
"GO_TEST_TIMEOUT_SCALE=5", // give them lots of time
25292517
},
@@ -2641,7 +2629,6 @@ func init() {
26412629
}
26422630
return b
26432631
},
2644-
needsGoProxy: true, // for cmd/go module tests
26452632
env: []string{
26462633
"GO_TEST_TIMEOUT_SCALE=5", // give them lots of time
26472634
},
@@ -2820,7 +2807,6 @@ func init() {
28202807
}
28212808
return b
28222809
},
2823-
needsGoProxy: true, // for cmd/go module tests
28242810
env: []string{
28252811
"GO_TEST_TIMEOUT_SCALE=5", // give them lots of time
28262812
},

0 commit comments

Comments
 (0)