From: Austin Clements Date: Thu, 25 May 2023 13:25:05 +0000 (-0400) Subject: cmd/dist: drop wasitest from special list X-Git-Tag: go1.21rc1~233 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=f4086d36862d8fd41af90356c894d04d3ff58203;p=gostls13.git cmd/dist: drop wasitest from special list CL 498255 made runtime/internal/wasitest compatible with all platforms so that "go test std" works again. This means it no longer has to be in the special dist test list. While we're here explain the purpose of this list better and implore people to please not expand it, since almost any addition is a sign that "go test std cmd" no longer works. Change-Id: I31c7fb767787fa587f65c2697aed9ed43e95fb18 Reviewed-on: https://go-review.googlesource.com/c/go/+/498256 Reviewed-by: Russ Cox Run-TryBot: Austin Clements TryBot-Result: Gopher Robot Auto-Submit: Austin Clements --- diff --git a/src/cmd/dist/test.go b/src/cmd/dist/test.go index f16bf32bbf..f148fb97b5 100644 --- a/src/cmd/dist/test.go +++ b/src/cmd/dist/test.go @@ -573,9 +573,17 @@ func (t *tester) registerRaceBenchTest(pkg string) { func (t *tester) registerTests() { // registerStdTestSpecially tracks import paths in the standard library // whose test registration happens in a special way. + // + // These tests *must* be able to run normally as part of "go test std cmd", + // even if they are also registered separately by dist, because users often + // run go test directly. Use skips or build tags in preference to expanding + // this list. registerStdTestSpecially := map[string]bool{ - "runtime/internal/wasitest": true, // Registered at the bottom as a host test. - "cmd/internal/testdir": true, // Registered at the bottom with sharding. + // testdir can run normally as part of "go test std cmd", but because + // it's a very large test, we register is specially as several shards to + // enable better load balancing on sharded builders. Ideally the build + // system would know how to shard any large test package. + "cmd/internal/testdir": true, } // Fast path to avoid the ~1 second of `go list std cmd` when