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

Dylan Baker baker.dylan.c at gmail.com
Sat Feb 15 01:11:54 PST 2014


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.


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.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20140215/abc3387c/attachment.html>


More information about the Piglit mailing list