Skip to content

Commit 86463c1

Browse files
committed
cmd/vet/all: delete
The work of running full vet on std and cmd during local development has moved to go test, which of course runs during all.bash. For errors in other GOOS/GOARCH combinations, the misc-compile builders (running buildall.bash) also now run go vet std cmd. The vetall builder need not do anything anymore. Make it a no-op until it can be retired, and remove cmd/vet/all and its special case in the go command. Fixes #31916. Change-Id: I8f30d184c382ea7c2c8f520e5618f680db633968 Reviewed-on: https://go-review.googlesource.com/c/go/+/176440 Run-TryBot: Russ Cox <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> Reviewed-by: Brad Fitzpatrick <[email protected]>
1 parent 4f76871 commit 86463c1

File tree

5 files changed

+1
-464
lines changed

5 files changed

+1
-464
lines changed

src/cmd/dist/test.go

-16
Original file line numberDiff line numberDiff line change
@@ -383,22 +383,6 @@ func (t *tester) registerRaceBenchTest(pkg string) {
383383
var stdOutErrAreTerminals func() bool
384384

385385
func (t *tester) registerTests() {
386-
if strings.HasSuffix(os.Getenv("GO_BUILDER_NAME"), "-vetall") {
387-
// Run vet over std and cmd and call it quits.
388-
for k := range cgoEnabled {
389-
osarch := k
390-
t.tests = append(t.tests, distTest{
391-
name: "vet/" + osarch,
392-
heading: "cmd/vet/all",
393-
fn: func(dt *distTest) error {
394-
t.addCmd(dt, "src/cmd/vet/all", "go", "run", "main.go", "-p="+osarch)
395-
return nil
396-
},
397-
})
398-
}
399-
return
400-
}
401-
402386
// Fast path to avoid the ~1 second of `go list std cmd` when
403387
// the caller lists specific tests to run. (as the continuous
404388
// build coordinator does).

src/cmd/go/internal/modload/init.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -357,7 +357,7 @@ func InitMod() {
357357
func modFileToBuildList() {
358358
Target = modFile.Module.Mod
359359
targetPrefix = Target.Path
360-
if rel := search.InDir(cwd, cfg.GOROOTsrc); rel != "" && rel != filepath.FromSlash("cmd/vet/all") {
360+
if rel := search.InDir(cwd, cfg.GOROOTsrc); rel != "" {
361361
targetInGorootSrc = true
362362
if Target.Path == "std" {
363363
targetPrefix = ""

0 commit comments

Comments
 (0)