]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/objdump: skip PIE test when cgo is disabled and PIE needs external linking
authorCherry Mui <cherryyz@google.com>
Thu, 3 Nov 2022 16:21:58 +0000 (12:21 -0400)
committerCherry Mui <cherryyz@google.com>
Thu, 3 Nov 2022 16:57:09 +0000 (16:57 +0000)
On some platforms, building PIE needs external linking, which
cannot run if cgo is not available.

Change-Id: I6d504aed0f0442cda0355d0beac606ad365e2046
Reviewed-on: https://go-review.googlesource.com/c/go/+/447616
Reviewed-by: Than McIntosh <thanm@google.com>
Run-TryBot: Cherry Mui <cherryyz@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>

src/cmd/objdump/objdump_test.go

index fa6a1b9a68cd8c3cca417ff14f1635e9e1e7b7c7..b747d0d542e8506b1fcbaf6620c575fe1d6d3c80 100644 (file)
@@ -290,6 +290,10 @@ func TestDisasmPIE(t *testing.T) {
        if !platform.BuildModeSupported("gc", "pie", runtime.GOOS, runtime.GOARCH) {
                t.Skipf("skipping on %s/%s, PIE buildmode not supported", runtime.GOOS, runtime.GOARCH)
        }
+       if !platform.InternalLinkPIESupported(runtime.GOOS, runtime.GOARCH) {
+               // require cgo on platforms that PIE needs external linking
+               testenv.MustHaveCGO(t)
+       }
        t.Parallel()
        testDisasm(t, "fmthello.go", false, false, "-buildmode=pie")
 }