[Piglit] [PATCH] log: Replace console output with a simpler output

Ilia Mirkin imirkin at alum.mit.edu
Sat Feb 15 01:14:05 PST 2014


On Sat, Feb 15, 2014 at 4:11 AM, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> This kind of feedback would have been really useful before it landed. It was
> on the list for a couple weeks, and I let it sit after Jordan had reviewed
> if for a couple of days. I'll go ahead and revert it, I'm just saying
> showing up to the party late and then complaining that the party started
> before you got there gets old.

Actually I've sent out patches that address various people's
concerns... Eric's should be addressed by one of them.

>
>
> On Fri, Feb 14, 2014 at 11:24 AM, Eric Anholt <eric at anholt.net> wrote:
>>
>> Dylan Baker <baker.dylan.c at gmail.com> writes:
>>
>> > This replaces the console spewer with a simpler console reporting
>> > mechanism inspired by the output of ninja. This reduces code, remove all
>> > singleton instances and speeds up piglit runs. There is a drawback, the
>> > output is much more terse than the previous implementation, giving only
>> > the following output:
>> > [<finished>/<total>] Running Test(s): <test number>
>> > [16008/16011] Running Test(s): 16007 16008 16009 16010
>>
>> I don't like this and don't want to see it land.  Seeing pass-fail
>> scrolling is really imporatnt to me, and saves me more time than you're
>> proposing to save with this patch.
>
>
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
>


More information about the Piglit mailing list