<div dir="ltr"><div>The patch has been updated and I am re-submitting everything I've submitted to Galina as well so that it goes through a full peer review.</div><div><br></div>Phew, thank you a ton for catching that!  <div>
<br></div><div>As far as the CC and CXX environment variables goes, I did it to retain backwards compatibility with the existing three LLD builders:</div><div><br></div><div><div>-                               env={</div>
<div>-                                    'CXX': "clang++",</div><div>-                                    'C':   "clang"},</div><div>+                               env=merged_env,</div>
</div><div><br></div><div>So they already got overriden, I have simply made the override overriddable :-)  Now you can specify that you want to use ccache in the env argument, something you could not do before.  I am also going to use ccache.  I bet your lld builds haven't been using ccache, even if you thought they did.</div>
<div><br></div><div>Please also notice that my builder uses GNU Gold - also through an override of env.</div><div><br></div><div><br></div><div>-- Mikael</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
2013/12/9 Dmitri Gribenko <span dir="ltr"><<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sat, Dec 7, 2013 at 2:48 AM, Mikael Lyngvig <<a href="mailto:mikael@lyngvig.org">mikael@lyngvig.org</a>> wrote:<br>
> I am finally getting somewhere on my project of setting up a PowerPC lld<br>
> builder and for that I need this patch to be included in Zorg.  It simply<br>
> adds the 'env' parameter to getLLDBuildFactory() that most of the other<br>
> build factory functions have.<br>
<br>
+                   'C'    : "clang",<br>
<br>
This should be 'CC'.<br>
<br>
Also, I think that this change might disable ccache on my builders,<br>
because this change overrides system's CC and CXX when the environment<br>
configured in zorg is empty.  I think that there should be no defaults<br>
for CC and CXX.  These can be overridden in per-builder configuration.<br>
<span class="HOEnZb"><font color="#888888"><br>
Dmitri<br>
<br>
--<br>
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if<br>
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>>*/<br>
</font></span></blockquote></div><br></div>