Skip to content

Commit

Permalink
cmd/objdump: ensure that test executable files are distinct
Browse files Browse the repository at this point in the history
This fixes test failures introduced by CL 176901.

Change-Id: I133299ba3be3a15ced076c95e4833ba6070d7eb7
Reviewed-on: https://go-review.googlesource.com/c/go/+/176903
Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
  • Loading branch information
josharian committed May 13, 2019
1 parent 3378683 commit f68244e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/cmd/objdump/objdump_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
package main

import (
"crypto/md5"
"flag"
"fmt"
"go/build"
Expand Down Expand Up @@ -101,7 +102,8 @@ func testDisasm(t *testing.T, printCode bool, flags ...string) {
goarch = f[1]
}

hello := filepath.Join(tmp, "hello.exe")
hash := md5.Sum([]byte(fmt.Sprintf("%v-%v", flags, printCode)))
hello := filepath.Join(tmp, fmt.Sprintf("hello-%x.exe", hash))
args := []string{"build", "-o", hello}
args = append(args, flags...)
args = append(args, "testdata/fmthello.go")
Expand Down

0 comments on commit f68244e

Please sign in to comment.