]> Cypherpunks repositories - gostls13.git/commitdiff
[dev.typeparams] cmd/link: include "go build" output in test logs
authorMatthew Dempsky <mdempsky@google.com>
Wed, 2 Jun 2021 00:00:26 +0000 (17:00 -0700)
committerMatthew Dempsky <mdempsky@google.com>
Wed, 2 Jun 2021 22:34:36 +0000 (22:34 +0000)
If running "go build" outputs anything, write it to the test log even
if the test succeeds. This makes it easier to diagnose errors within
the compiler by adding print statements and finding them in the test
log, even if the compiler exits successfully.

Change-Id: Id04716c4e1dcd9220c35ea0040ea516c1dd5237c
Reviewed-on: https://go-review.googlesource.com/c/go/+/324329
Trust: Matthew Dempsky <mdempsky@google.com>
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Cherry Mui <cherryyz@google.com>
src/cmd/link/internal/ld/dwarf_test.go

index 2f59c2fe0aa9484a0294170ad7e21fa6a0710ca0..543dd5caac7454ade0419564fdc0bff028e38e15 100644 (file)
@@ -101,8 +101,11 @@ func gobuild(t *testing.T, dir string, testfile string, gcflags string) *builtFi
        }
 
        cmd := exec.Command(testenv.GoToolPath(t), "build", gcflags, "-o", dst, src)
-       if b, err := cmd.CombinedOutput(); err != nil {
-               t.Logf("build: %s\n", b)
+       b, err := cmd.CombinedOutput()
+       if len(b) != 0 {
+               t.Logf("## build output:\n%s", b)
+       }
+       if err != nil {
                t.Fatalf("build error: %v", err)
        }