[Piglit] V2 Finish replacing summary

Jose Fonseca jfonseca at vmware.com
Wed Jul 3 11:18:33 PDT 2013


Yes.

Reviewed-by: Jose Fonseca <jfonseca at vmware.com>

Jose

----- Original Message -----
> does that mean I have a reviewed-by for the two junit patches?
> 
> 
> On Fri, Jun 28, 2013 at 1:11 PM, Jose Fonseca <jfonseca at vmware.com> wrote:
> 
> >
> >
> > ----- Original Message -----
> > > Ah, I knew I forgot something.
> > >
> > > https://github.com/crymson/piglit.git mako-complete-v2
> >
> > Looks great.  Output is 100% identical to before.
> >
> > Thanks.
> >
> > Jose
> >
> > >
> > >
> > > On Fri, Jun 28, 2013 at 9:34 AM, Jose Fonseca <jfonseca at vmware.com>
> > wrote:
> > >
> > > >
> > > > ----- Original Message -----
> > > > > This series finishes replacing the old Summary class with the
> > NewSummary
> > > > > That was added for the new HTML backend.
> > > > >
> > > > > Patches 1 and 4 are groundwork changes that should be transparent to
> > end
> > > > > users
> > > > > Patches 2 and 3 Do a little bit of cleanup for HTML summary
> > generation,
> > > > > namely they stop generating pages when there is no content.
> > > > > Patches 5-9 Convert piglit-summary.py to use the NewSummary class
> > > >
> > > > > Patches 10-11 Change the behavior of piglit-summary-juint.py. These
> > > > changes
> > > > > follow discussion with Jose and constitute the majority of the V2
> > changes
> > > >
> > > > Thanks. It sounds good. Can I pull this from somewhere?
> > > >
> > > > Jose
> > > >
> > > > > Patch 12 renames the NewSummary class to just Summary
> > > > >
> > > >
> > >
> >
> 


More information about the Piglit mailing list