[Piglit] [PATCH] framework/summary.py: Always print number of changes in console mode

2015-09-09 Thread Tom Stellard
The counts for changes, fixes, and regressions are now reported as zero when no changes are detect rather than being omitted from the output. This makes it easier for scripts to parse the output of piglit summary console, becuase now when there are no regressions piglit always outputs:

Re: [Piglit] [PATCH] framework/summary.py: Always print number of changes in console mode

2015-09-09 Thread Tom Stellard
On Wed, Sep 09, 2015 at 01:23:22PM -0700, Dylan Baker wrote: > On Wed, Sep 09, 2015 at 03:57:55PM -0400, Tom Stellard wrote: > > On Wed, Sep 09, 2015 at 12:52:22PM -0700, Dylan Baker wrote: > > > > I have a buildbot seti up to run piglit and the way it detects whether > > > > or no there have been

Re: [Piglit] [PATCH] framework/summary.py: Always print number of changes in console mode

2015-09-09 Thread Dylan Baker
Reviewed-by: Dylan Baker On a related note, I have a series that seriously overhauls the summary code, including fixing the console/text summary, producing a column for each result. It does include changes, fixes, and regressions in all cases like this patch. I'm hoping

Re: [Piglit] [PATCH] framework/summary.py: Always print number of changes in console mode

2015-09-09 Thread Tom Stellard
On Wed, Sep 09, 2015 at 11:05:55AM -0700, Dylan Baker wrote: > Reviewed-by: Dylan Baker > > On a related note, I have a series that seriously overhauls the summary > code, including fixing the console/text summary, producing a column for > each result. It does include

Re: [Piglit] [PATCH] framework/summary.py: Always print number of changes in console mode

2015-09-09 Thread Dylan Baker
On Wed, Sep 09, 2015 at 03:25:50PM -0400, Tom Stellard wrote: > On Wed, Sep 09, 2015 at 11:05:55AM -0700, Dylan Baker wrote: > > Reviewed-by: Dylan Baker > > > > On a related note, I have a series that seriously overhauls the summary > > code, including fixing the

Re: [Piglit] [PATCH] framework/summary.py: Always print number of changes in console mode

2015-09-09 Thread Dylan Baker
> I have a buildbot seti up to run piglit and the way it detects whether > or no there have been piglit regressions is by running piglit summary > console and then parsing the output. > > The CSV summary seems to only accept a single result file, unless I'm > doing something wrong: > >

Re: [Piglit] [PATCH] framework/summary.py: Always print number of changes in console mode

2015-09-09 Thread Tom Stellard
On Wed, Sep 09, 2015 at 12:52:22PM -0700, Dylan Baker wrote: > > I have a buildbot seti up to run piglit and the way it detects whether > > or no there have been piglit regressions is by running piglit summary > > console and then parsing the output. > > > > The CSV summary seems to only accept a