<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/2132/">https://reviewboard.asterisk.org/r/2132/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Just by way of a note, when we first added the osx minimum version flag, it was meant to raise the floor, because at the time, if not specified, the Apple compiler would assume compatibility with 10.0. At this point, since 10.8 is out, we could potentially raise that floor further, to 10.5 or 10.6, depending upon what minimum version of the Apple linker supports the -rpath option.</pre>
<br />
<p>- Tilghman</p>
<br />
<p>On September 26th, 2012, 1:28 p.m., David Lee wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By David Lee.</div>
<p style="color: grey;"><i>Updated Sept. 26, 2012, 1:28 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Sets -rpath linker option when appropriate.
* For OS's other than OS X, add the -rpath linker option if libdir is
anything other than /lib or /usr/lib. 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.
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Built on OS X and Ubuntu Precise, --prefix=/opt/Asterisk, both with and without --disable-rpath.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/ASTERSIK-20407">ASTERSIK-20407</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/branches/11/Makefile <span style="color: grey">(373734)</span></li>
<li>/branches/11/configure <span style="color: grey">(UNKNOWN)</span></li>
<li>/branches/11/configure.ac <span style="color: grey">(373734)</span></li>
<li>/branches/11/main/Makefile <span style="color: grey">(373734)</span></li>
<li>/branches/11/makeopts.in <span style="color: grey">(373734)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2132/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>