Skip to content

Commit 9693b94

Browse files
prattmicgopherbot
authored andcommitted
runtime: include stderr when objdump fails
If objdump panics, we want the panic included in the test log. Change-Id: I6a6a636c13c5ba08acaa1c6c8714541a8e91542b Reviewed-on: https://go-review.googlesource.com/c/go/+/704338 Reviewed-by: Florian Lehner <[email protected]> Auto-Submit: Michael Pratt <[email protected]> Reviewed-by: Michael Knyszek <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]>
1 parent 8616981 commit 9693b94

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/runtime/unsafepoint_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ func TestUnsafePoint(t *testing.T) {
4343
cmd := exec.Command(testenv.GoToolPath(t), "tool", "objdump", "-s", "setGlobalPointer", os.Args[0])
4444
out, err := cmd.CombinedOutput()
4545
if err != nil {
46-
t.Fatalf("can't objdump %v", err)
46+
t.Fatalf("can't objdump %v:\n%s", err, out)
4747
}
4848
lines := strings.Split(string(out), "\n")[1:]
4949

0 commit comments

Comments
 (0)