Page MenuHomePhabricator

Build C code in seeder with PIE_FLAGS
ClosedPublic

Authored by freetrader on Oct 4 2017, 20:50.

Details

Summary

This avoids a relocation error for libbitcoin_seeder_a-dns.o when building
the seeder shared library on some Debian environments (rare).

The build problem was observed on Debian 8 x86_64 derivate with gcc 7.1.0:

/usr/bin/ld: libbitcoin_seeder.a(libbitcoin_seeder_a-dns.o): relocation R_X86_64_32 against '.rodata' can not be used when making a shared object; recompile with -fPIC

Depends D581 .

Test Plan

make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
add_pie_flag_to_seeder_includes
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 949
Build 949: arc lint + arc unit

Event Timeline

Herald added 1 blocking reviewer(s): Restricted Project. · View Herald TranscriptOct 4 2017, 20:50
deadalnix requested changes to this revision.Oct 5 2017, 12:47
deadalnix added inline comments.
src/Makefile.am
28 ↗(On Diff #1488)

That's definitively not an include flag.

This revision now requires changes to proceed.Oct 5 2017, 12:47
freetrader edited edge metadata.

Move PIE_FLAGS to more appropriate location

deadalnix requested changes to this revision.Oct 5 2017, 15:08

I don't think this even builds...

src/Makefile.am
28 ↗(On Diff #1496)

fix

This revision now requires changes to proceed.Oct 5 2017, 15:08
freetrader edited edge metadata.

Fix dropped parenthesis

This revision is now accepted and ready to land.Oct 5 2017, 21:01
This revision was automatically updated to reflect the committed changes.