Page MenuHomeFreeBSD

Deorbit ALLOW_SHARED_TEXTREL
ClosedPublic

Authored by emaste on May 11 2016, 4:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 12, 2:14 PM
Unknown Object (File)
Tue, Nov 11, 11:41 AM
Unknown Object (File)
Tue, Nov 11, 5:27 AM
Unknown Object (File)
Tue, Nov 4, 2:26 AM
Unknown Object (File)
Sun, Nov 2, 11:54 AM
Unknown Object (File)
Sun, Nov 2, 11:54 AM
Unknown Object (File)
Sun, Nov 2, 11:54 AM
Unknown Object (File)
Sun, Nov 2, 7:43 AM
Subscribers

Details

Summary

We want to avoid .text relocations in shared objects. libcrypto was the only consumer and it is now fixed, so remove the now-unused support for disabling the linker warning.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste retitled this revision from to Deorbit ALLOW_SHARED_TEXTREL.
emaste updated this object.
emaste edited the test plan for this revision. (Show Details)
emaste added a reviewer: kib.
emaste added a subscriber: jkim.
kib edited edge metadata.

This might make harder future imports of buggy libraries in future. OTOH, the LD_FATAL_WARNING knob provides the same functionality still.

This revision is now accepted and ready to land.May 11 2016, 5:38 PM

This might make harder future imports of buggy libraries in future. OTOH, the LD_FATAL_WARNING knob provides the same functionality still.

Yeah, my hope is that this might encourage a future importer of a buggy library to fix it first, and if not at least the ld warning will come out in the log.

This revision was automatically updated to reflect the committed changes.