[PATCH] Link Clang/ARM/Arch Linux builder into Zorg

[Renato Golin]

LGTM, thanks!

--renato


On 25 November 2013 12:31, Mikael Lyngvig <mikael at lyngvig.org> wrote:

> Here's what I hope to be the final patch.  I've even proof-read it line by
> line (I always do a diff before committing, but obviously I must have been
> skimming too fast).
>
> Yes, I can imagine what a nightmare it must have been.  Feverish coders en
> masse battling for supremacy with each their own ill thought through
> patches.
>
> I apologize for my numerous errors this day, but I've been up too long,
> working too hard, so now the failure rate is increasing exponentially, it
> seems.  Time to shut down for today.
>
>
> 2013/11/25 Renato Golin <renato.golin at linaro.org>
>
>> On 25 November 2013 12:21, Mikael Lyngvig <mikael at lyngvig.org> wrote:
>>
>>> I did the unthinkable and edited the previous patch when I enabled the
>>> test so now that change got lost when I sent you the last patch.  Do you
>>> mind editing the "test=False" to become "test=True" in the patch I just
>>> sent you?
>>>
>>
>> Heh, I was just about to comment on that. ;)
>>
>> You can send a final patch to the list with both changes in. I prefer if
>> Galina commits the Zorg patches, since she knows when the buildmaster will
>> restart.
>>
>> The changes look good to me with 4 jobs and test=true.
>>
>> cheers,
>> --renato
>>
>> PS: I can't remember how it was like before peer review, but it surely
>> was horrible!
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131125/0d553d2a/attachment.html>