<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">ok thanks.<div><br></div><div>-Chris</div><div><br><div><div>On May 22, 2011, at 9:56 AM, İsmail Dönmez wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><font size="2"><font face="tahoma,sans-serif">Oh nm, its not a typo. Here comes the ld(1):<br></font></font><div><font class="Apple-style-span" face="tahoma, sans-serif"><br></font></div><div>--no-add-needed </div><div><br>

</div><div>This option affects the treatment of dynamic libraries from ELF DT_NEEDED tags in dynamic libraries mentioned on the command line after the --no-add-needed option. Normally, the linker will add a DT_NEEDED tag for each dynamic library from DT_NEEDED tags. --no-add-needed causes DT_NEEDED tags will never be emitted for thoselibraries from DT_NEEDED tags. --add-needed restores the default behaviour.</div>

<div><br></div><div>Sorry! ;)</div><div><br><div class="gmail_quote">On Sun, May 22, 2011 at 6:55 PM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div style="word-wrap:break-word">Can a redhat user test this?<div><br></div><div>-Chris</div><div><br><div><div><div></div><div class="h5"><div>On May 22, 2011, at 9:53 AM, İsmail Dönmez wrote:</div><br></div></div><blockquote type="cite">

<div><div></div><div class="h5"><font size="2"><font face="tahoma,sans-serif">Thanks, here is one for the Redhat typo I mentioned.<br></font></font><div><font face="tahoma, sans-serif"><br></font></div><div><font face="tahoma, sans-serif">Regards.<br>



</font></div><div><div class="gmail_quote"><br></div><div class="gmail_quote">On Sun, May 22, 2011 at 6:50 PM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com" target="_blank">clattner@apple.com</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Applied in r131857, thanks<div><br><div><div><div></div><div><div>On May 22, 2011, at 9:40 AM, İsmail Dönmez wrote:</div>



<br></div></div><blockquote type="cite"><div><div></div><div><font size="2"><font face="tahoma,sans-serif">Hi Chris;<br></font></font><br><div class="gmail_quote">On Sun, May 22, 2011 at 6:32 PM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com" target="_blank">clattner@apple.com</a>></span> wrote:<br>





<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hi Ismail,<div><br></div><div>This doesn't apply cleanly to mainline.  The changes required look trivial, but I'd prefer you to do it because I can't test this.</div>





</div></blockquote><div><br></div><div>New patch attached, but this line from Toolchains.cpp:</div><div><br></div><div>  if (IsRedhat(Distro))</div><div>    ExtraOpts.push_back("--no-add-needed");</div><div><br>





</div><div>looks like a typo, correct option is "--no-as-needed".</div><div><br></div><div>Regards,</div><div>ismail</div><div> </div></div>
</div></div><span><toolchain-update.patch></span></blockquote></div><br></div></div></blockquote></div><br></div>
</div></div><span><redhat-typo.patch></span></blockquote></div><br></div></div></blockquote></div><br></div>
</blockquote></div><br></div></body></html>