Page MenuHomePhabricator

[CMAKE] Add PIC/PIE flags on non Windows platforms
ClosedPublic

Authored by Fabien on Mon, Mar 4, 16:15.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Commits
rABCcb22eb3fee4a: [CMAKE] Add PIC/PIE flags on non Windows platforms
Summary

This diffs adds the -fPIC and -fPIE compiler flags, as well as the
-pie compiler flag.

Test Plan
mkdir buildcmake && cd buildcmake
cmake -GNinja ..
ninja
../contrib/devtools/security-check.py src/bitcoind

Before the patch, the output would contain PIE, no longer after the
patch.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien created this revision.Mon, Mar 4, 16:15
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Mar 4, 16:15
Herald added a subscriber: schancel. · View Herald Transcript
deadalnix accepted this revision.Mon, Mar 4, 17:40
This revision is now accepted and ready to land.Mon, Mar 4, 17:40
This revision was automatically updated to reflect the committed changes.