Page MenuHomeFreeBSD

svn: support building as PIE
ClosedPublic

Authored by emaste on Feb 19 2019, 3:21 PM.

Diff Detail

Repository
rS FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

emaste created this revision.Feb 19 2019, 3:21 PM
cem accepted this revision.Feb 19 2019, 5:56 PM

Why is the MK_PIE PIE_SUFFIX “_pic”? That seems inconsistent. Otherwise, LGTM.

(Seems like we could just dump all of these “.a” into a single library as well to simplify the build, but that is an orthogonal improvement.)

This revision is now accepted and ready to land.Feb 19 2019, 5:56 PM
In D19246#411794, @cem wrote:

Why is the MK_PIE PIE_SUFFIX “_pic”? That seems inconsistent. Otherwise, LGTM.

Leftover from an earlier iteration of D18423

usr.bin/svn/Makefile.inc
44 ↗(On Diff #54069)

Will s/_pic/_pie/ here

emaste added inline comments.Feb 19 2019, 10:27 PM
usr.bin/svn/Makefile.inc
5 ↗(On Diff #54069)

Oh, need to remove this as well :)

This revision was automatically updated to reflect the committed changes.