[llvm] r193463 - lit: handle late multiprocessing errors gracefully

[Alp Toker]

On 29/10/2013 07:47, Sylvestre Ledru wrote:
> On 28/10/2013 11:44, Alp Toker wrote:
>> On 28/10/2013 10:09, Sylvestre Ledru wrote:
>>> Hello,
>>>
>>> Could you fix that ?
>> Fixed in r193516..
> Excellent!
>
>> I'm a little surprised because that commit shouldn't have enabled
>> anything new outside of BSD.
>>
>> Are you sure the test suite wasn't already failing on your builder
>> before r193463?
> Depends what you mean by "before" :)
> It was working a few days before this commit and, with your fix, the
> code coverage is back in business.
> http://buildd-clang.debian.net/coverage/

For the record, I suspect your build was broken earlier when the feature
was actually enabled, in r193279:

    [lit] Use multiprocessing based parallelism by default, on Unix.

My commits have just been dealing with the fallout from that.


>
> Thanks again,

My pleasure!

> Sylvestre
>
>

-- 
http://www.nuanti.com
the browser experts