Skip to content

Commit 706d84f

Browse files
committed
cmd/link: suppress -no_pie deprecation warning on darwin
Apparently the new darwin linker starts to emit a warning about -no_pie deprecation. Maybe we want to switch to PIE by default. For now, suppress the warning. This also makes it easier for backporting to previous releases. For golang#54482. Change-Id: I1a3b74c237a9d00ec3b030fc3a9940a31e5cd37e Reviewed-on: https://go-review.googlesource.com/c/go/+/430937 Run-TryBot: Cherry Mui <[email protected]> Reviewed-by: Than McIntosh <[email protected]> TryBot-Result: Gopher Robot <[email protected]>
1 parent 5abf200 commit 706d84f

File tree

1 file changed

+7
-0
lines changed
  • src/cmd/link/internal/ld

1 file changed

+7
-0
lines changed

src/cmd/link/internal/ld/lib.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1775,6 +1775,13 @@ func (ctxt *Link) hostlink() {
17751775
if len(out) > 0 {
17761776
// always print external output even if the command is successful, so that we don't
17771777
// swallow linker warnings (see https://golang.org/issue/17935).
1778+
if ctxt.IsDarwin() && ctxt.IsAMD64() {
1779+
const noPieWarning = "ld: warning: -no_pie is deprecated when targeting new OS versions\n"
1780+
if i := bytes.Index(out, []byte(noPieWarning)); i >= 0 {
1781+
// swallow -no_pie deprecation warning, issue 54482
1782+
out = append(out[:i], out[i+len(noPieWarning):]...)
1783+
}
1784+
}
17781785
ctxt.Logf("%s", out)
17791786
}
17801787

0 commit comments

Comments
 (0)