Skip to content

Commit 0682cd6

Browse files
cherrymuibradfitz
authored andcommitted
[release-branch.go1.19] 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. Fixes golang#55114. Updates golang#55112, 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]> (cherry picked from commit 706d84f) Reviewed-on: https://go-review.googlesource.com/c/go/+/431517
1 parent 08f29e8 commit 0682cd6

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

0 commit comments

Comments
 (0)