<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 />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On September 26th, 2012, 2:24 p.m., <b>Tilghman Lesher</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <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>
 </blockquote>







</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">When merging to trunk, I plan on raising it to 10.5 (Leopard). (See my comment on the mailing list[1]). That&#39;s enough to add rpath support. I&#39;d rather not make that change in 11, since that&#39;s exactly the sort of change that looks like it should be fine, but will turn around and bite us in ways we don&#39;t expect.

Apple&#39;s doesn&#39;t officially acknowledge end-of-support for OS versions; you just have to watch their security updates. Their most recent update[2] was released for 10.6 (Snow Leopard) and later. But Leopard received an update[3] in May of this year, and browser stats[4] show that it still has 10% of the overall Mac market. That&#39;s enough for me to say that 10.5 is a high enough minimum (until we find some other compiler flag we want to turn on that requires a higher minimum version).

 [1]: http://lists.digium.com/pipermail/asterisk-dev/2012-September/057013.html
 [2]: http://support.apple.com/kb/HT1222
 [3]: http://support.apple.com/kb/HT5283
 [4]: http://www.netmarketshare.com/operating-system-market-share.aspx?qprid=10&amp;qpcustomb=*2</pre>
<br />








<p>- David</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&#39;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&#39;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>