Commit 7ec5499d authored by David Symonds's avatar David Symonds

debug/gosym: more carefully build the test binary.

TBR=r
CC=golang-dev
https://golang.org/cl/5676062
parent aed20a69
...@@ -6,6 +6,7 @@ package gosym ...@@ -6,6 +6,7 @@ package gosym
import ( import (
"debug/elf" "debug/elf"
"fmt"
"os" "os"
"os/exec" "os/exec"
"runtime" "runtime"
...@@ -27,7 +28,9 @@ func dotest() bool { ...@@ -27,7 +28,9 @@ func dotest() bool {
// the resulting binary looks like it was built from pclinetest.s, // the resulting binary looks like it was built from pclinetest.s,
// but we have renamed it to keep it away from the go tool. // but we have renamed it to keep it away from the go tool.
pclinetestBinary = os.TempDir() + "/pclinetest" pclinetestBinary = os.TempDir() + "/pclinetest"
cmd := exec.Command("sh", "-c", "go tool 6a pclinetest.asm && go tool 6l -E main -o "+pclinetestBinary+" pclinetest.6") command := fmt.Sprintf("go tool 6a -o %s.6 pclinetest.asm && go tool 6l -E main -o %s %s.6",
pclinetestBinary, pclinetestBinary, pclinetestBinary)
cmd := exec.Command("sh", "-c", command)
cmd.Stdout = os.Stdout cmd.Stdout = os.Stdout
cmd.Stderr = os.Stderr cmd.Stderr = os.Stderr
if err := cmd.Run(); err != nil { if err := cmd.Run(); err != nil {
...@@ -185,8 +188,8 @@ func TestPCLine(t *testing.T) { ...@@ -185,8 +188,8 @@ func TestPCLine(t *testing.T) {
t.Logf("off is %d", off) t.Logf("off is %d", off)
if fn == nil { if fn == nil {
t.Errorf("failed to get line of PC %#x", pc) t.Errorf("failed to get line of PC %#x", pc)
} else if !strings.HasSuffix(file, "pclinetest.s") { } else if !strings.HasSuffix(file, "pclinetest.asm") {
t.Errorf("expected %s (%s) at PC %#x, got %s (%s)", "pclinetest.s", sym.Name, pc, file, fn.Name) t.Errorf("expected %s (%s) at PC %#x, got %s (%s)", "pclinetest.asm", sym.Name, pc, file, fn.Name)
} else if line != wantLine || fn != sym { } else if line != wantLine || fn != sym {
t.Errorf("expected :%d (%s) at PC %#x, got :%d (%s)", wantLine, sym.Name, pc, line, fn.Name) t.Errorf("expected :%d (%s) at PC %#x, got :%d (%s)", wantLine, sym.Name, pc, line, fn.Name)
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment