<div dir="ltr"><div class="gmail_extra">Sorry for the very slow response to this patch.</div><div class="gmail_extra"><br></div><div class="gmail_extra">The patch looks good, except for the part </div><div class="gmail_extra">


<br></div><div class="gmail_extra"><div class="gmail_extra">+  def include_sysroot(self):</div><div class="gmail_extra">+    # FIXME: Is this the correct precedence for isysroot and sysroot?</div><div class="gmail_extra">


+    if self.sysroot:</div><div class="gmail_extra">+      return self.sysroot</div><div class="gmail_extra">+    return self.isysroot</div><div class="gmail_extra"><br></div><div class="gmail_extra">where the correct precedence is the other way around.</div>


<div class="gmail_extra"><br></div><div class="gmail_extra">I have changed that part so that isysroot takes precedence, and submitted this CL as revision 783 in the distcc svn repository.</div><br><div class="gmail_quote">

On Tue, Dec 20, 2011 at 6:43 PM, Rafael Ávila de Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">The attached patch is not complete, but it adds enough features and<br>



fixes to let distcc pump build a slightly patched version of firefox<br>
on OS X.<br>
<br>
What the patch does is<br>
<br>
* add support for -isysroot and -arch<br>
* Fix a really nasty corner case:<br>
  * CompressFiles::Compress uses os.makedirs to create a directory<br>
(foo)  for one compilation unit<br>
  * Another compilation unit does the equivalent of '#include<br>
"foo/../bar.h"'<br>
  * The compilation unit uses no other file in foo<br>
  * We conclude that we don't have to create a dummy<br>
foo/forcing_technique_271828<br>
<br>
Cheers,<br>
Rafael<br>
<span><font color="#888888"><br>
--<br>
You received this message because you are subscribed to the "distcc-patches" list.<br>
To post to this list, send email to <<a href="mailto:distcc-patches@googlegroups.com" target="_blank">distcc-patches@googlegroups.com</a>>.<br>
To unsubscribe from this list, send email to <<a href="mailto:distcc-patches-unsubscribe@googlegroups.com" target="_blank">distcc-patches-unsubscribe@googlegroups.com</a>>.<br>
For archives and more options, see <<a href="http://groups.google.com/group/distcc-patches" target="_blank">http://groups.google.com/group/distcc-patches</a>>.</font></span></blockquote></div><br><br clear="all"><div>


<br></div>-- <br><div>Fergus Henderson <<a href="mailto:fergus@google.com" target="_blank">fergus@google.com</a>></div><div><br></div><div><div><span style="font-family:arial,sans-serif;font-size:13px;border-collapse:collapse">"Defend the user, exclude no one, and create magic." -- Eric Schmidt.</span></div>


</div><div><span style="font-family:arial,sans-serif;font-size:13px;border-collapse:collapse"><br></span></div>
</div></div>