From a99c38d66381b2a6abbc0d9c88feb3f6291cb245 Mon Sep 17 00:00:00 2001 From: Michael Pratt Date: Fri, 14 Jan 2022 15:22:51 -0500 Subject: Revert "cmd/dist: log CPU model when testing" Fails TestRepeatBootstrap. This reverts CL 371474. For #50146. Change-Id: Ie4adda4e0229e153471301ca00fe2c1c694b4b2d Reviewed-on: https://go-review.googlesource.com/c/go/+/378587 Reviewed-by: Bryan Mills Reviewed-by: Ian Lance Taylor Trust: Michael Pratt Run-TryBot: Michael Pratt TryBot-Result: Gopher Robot --- src/cmd/dist/test.go | 19 ------------------- 1 file changed, 19 deletions(-) (limited to 'src/cmd/dist/test.go') diff --git a/src/cmd/dist/test.go b/src/cmd/dist/test.go index fed83120ed..50a2e5936c 100644 --- a/src/cmd/dist/test.go +++ b/src/cmd/dist/test.go @@ -218,15 +218,6 @@ func (t *tester) run() { } } - if err := t.maybeLogMetadata(); err != nil { - t.failed = true - if t.keepGoing { - log.Printf("Failed logging metadata: %v", err) - } else { - fatalf("Failed logging metadata: %v", err) - } - } - for _, dt := range t.tests { if !t.shouldRunTest(dt.name) { t.partial = true @@ -277,16 +268,6 @@ func (t *tester) shouldRunTest(name string) bool { return false } -func (t *tester) maybeLogMetadata() error { - if t.compileOnly { - // We need to run a subprocess to log metadata. Don't do that - // on compile-only runs. - return nil - } - t.out("Test execution environment.") - return logMetadata() -} - // short returns a -short flag value to use with 'go test' // or a test binary for tests intended to run in short mode. // It returns "true", unless the environment variable -- cgit v1.2.3-54-g00ecf