<div dir="ltr">Oh yeah, that's fine.  I won't take that code out.<div><br></div><div>Hmm at least some of the builds went through this weekend, I made a number of changes Saturday morning (US Pacific time) that I saw go through the Ubuntu 14.04 cmake bot.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 14, 2015 at 6:29 AM, Pavel Labath <span dir="ltr"><<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
we've had an unrelated breaking change, so the buildbots were red over<br>
the weekend. I've fixed it now, and it seems to be turning green.<br>
We've also had power outage during the weekend and not all of the<br>
buildbots are back up yet, as we need to wait for MTV to wake up. I'd<br>
like to give this at least one more day, to give them a chance to<br>
stabilize. Is this blocking you from making further changes to the<br>
test event system?<br>
<span class="HOEnZb"><font color="#888888"><br>
pl<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On 12 December 2015 at 00:20, Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br>
> Hey Pavel and/or Tamas,<br>
><br>
> Let me know when we're definitely all clear on the expected timeout support<br>
> I added to the (now once again) newer default test results.<br>
><br>
> As soon as we don't need the legacy summary results anymore, I'm going to<br>
> strip out the code that manages it.  It is quite messy and duplicates the<br>
> content that is better handled by the test event system.<br>
><br>
> Thanks!<br>
><br>
> -Todd<br>
><br>
> On Fri, Dec 11, 2015 at 2:03 PM, Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br>
>><br>
>> I went ahead and added the expected timeout support in r255363.<br>
>><br>
>> I'm going to turn back on the new BasicResultsFormatter as the default.<br>
>> We can flip this back off if it is still not doing everything we need, but I<br>
>> *think* we cover the issue you saw now.<br>
>><br>
>> -Todd<br>
>><br>
>> On Fri, Dec 11, 2015 at 10:14 AM, Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br>
>>><br>
>>> Hi Pavel,<br>
>>><br>
>>> I'm going to adjust the new summary output for expected timeouts.  I hope<br>
>>> to do that in the next hour or less.  I'll put that in and flip the default<br>
>>> back on for using the new summary output.<br>
>>><br>
>>> I'll do those two changes separately, so you can revert the flip back on<br>
>>> to flip it back off if we still have an issue.<br>
>>><br>
>>> Sound good?<br>
>>><br>
>>> (This can be orthogonal to the new work to mark up expected timeouts).<br>
>>> --<br>
>>> -Todd<br>
>><br>
>><br>
>><br>
>><br>
>> --<br>
>> -Todd<br>
><br>
><br>
><br>
><br>
> --<br>
> -Todd<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div>