diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2016-04-24 13:33:33 +0200 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2016-05-03 12:35:09 +0000 |
| commit | babdbfb8260bbe8c6305c9d3023d83cc0b3645bf (patch) | |
| tree | c0797119ac2c86d583d9fcf6c8e89e8d4b820a0e /src/cmd/trace | |
| parent | caa21475328999c1cd108b71ceb6efb7f4cf8fc4 (diff) | |
| download | go-babdbfb8260bbe8c6305c9d3023d83cc0b3645bf.tar.xz | |
cmd/trace: make binary argument optional
1.7 traces embed symbol info and we now generate symbolized pprof profiles,
so we don't need the binary. Make binary argument optional as 1.5 traces
still need it.
Change-Id: I65eb13e3d20ec765acf85c42d42a8d7aae09854c
Reviewed-on: https://go-review.googlesource.com/22410
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
Reviewed-by: Austin Clements <austin@google.com>
Diffstat (limited to 'src/cmd/trace')
| -rw-r--r-- | src/cmd/trace/main.go | 21 |
1 files changed, 14 insertions, 7 deletions
diff --git a/src/cmd/trace/main.go b/src/cmd/trace/main.go index cfd222e132..2735bf13ea 100644 --- a/src/cmd/trace/main.go +++ b/src/cmd/trace/main.go @@ -14,7 +14,7 @@ Example usage: Generate a trace file with 'go test': go test -trace trace.out pkg View the trace in a web browser: - go tool trace pkg.test trace.out + go tool trace trace.out */ package main @@ -37,7 +37,9 @@ Given a trace file produced by 'go test': go test -trace=trace.out pkg Open a web browser displaying trace: - go tool trace [flags] pkg.test trace.out + go tool trace [flags] [pkg.test] trace.out +[pkg.test] argument is required for traces produced by Go 1.6 and below. +Go 1.7 does not require the binary argument. Flags: -http=addr: HTTP service address (e.g., ':6060') @@ -58,12 +60,17 @@ func main() { } flag.Parse() - // Usage information when no arguments. - if flag.NArg() != 2 { + // Go 1.7 traces embed symbol info and does not require the binary. + // But we optionally accept binary as first arg for Go 1.5 traces. + switch flag.NArg() { + case 1: + traceFile = flag.Arg(0) + case 2: + programBinary = flag.Arg(0) + traceFile = flag.Arg(1) + default: flag.Usage() } - programBinary = flag.Arg(0) - traceFile = flag.Arg(1) ln, err := net.Listen("tcp", *httpFlag) if err != nil { @@ -91,7 +98,7 @@ var loader struct { func parseEvents() ([]*trace.Event, error) { loader.once.Do(func() { - tracef, err := os.Open(flag.Arg(1)) + tracef, err := os.Open(traceFile) if err != nil { loader.err = fmt.Errorf("failed to open trace file: %v", err) return |