[asterisk-dev] [Code Review] Sets -rpath linker option when appropriate

[David Lee]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2132/
-----------------------------------------------------------

(Updated Sept. 27, 2012, 8:52 a.m.)


Review request for Asterisk Developers.


Changes
-------

Suggestion by Jeff Ollie: Change detection from libdir=[/usr]/lib to prefix=/usr.


Summary (updated)
-------

Sets -rpath linker option when appropriate.

 * For OS's other than OS X, add the -rpath linker option if prefix is
   anything other than /usr. This can be disabled by passing
   --disable-rpath to ./configure.
 * For OS X, we have macosx_version_min set to 10.4, which doesn't
   support rpath. So instead, the patch adds the -dynamiclib and
   -install_name flags for linking libasteriskssl.dylib, along with
   cleaning up some of the other OS X specific linker flags.


This addresses bug ASTERSIK-20407.
    https://issues.asterisk.org/jira/browse/ASTERSIK-20407


Diffs (updated)
-----

  /branches/11/Makefile 373907 
  /branches/11/configure UNKNOWN 
  /branches/11/configure.ac 373907 
  /branches/11/main/Makefile 373907 
  /branches/11/makeopts.in 373907 

Diff: https://reviewboard.asterisk.org/r/2132/diff


Testing
-------

Built on OS X and Ubuntu Precise, --prefix=/opt/Asterisk, both with and without --disable-rpath.


Thanks,

David

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120927/7b32af20/attachment.htm>