Skip to content

Commit

Permalink
Only run TestDisasm on amd64
Browse files Browse the repository at this point in the history
It seems objdump is not able to dump other architecture object.
  • Loading branch information
zhsj committed Oct 7, 2020
1 parent 1066cbb commit 924778f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions internal/binutils/binutils_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -217,8 +217,8 @@ func testDisasm(t *testing.T, intelSyntax bool) {
}

func TestDisasm(t *testing.T) {
if runtime.GOOS != "linux" && runtime.GOOS != "darwin" {
t.Skip("This test only works on Linux or macOS")
if (runtime.GOOS != "linux" && runtime.GOOS != "darwin") || runtime.GOARCH != "amd64" {
t.Skip("This test only works on x86-64 Linux or macOS")
}
testDisasm(t, false)
}
Expand Down

0 comments on commit 924778f

Please sign in to comment.