[Piglit] Set default platform in pigilt.conf

Dylan Baker baker.dylan.c at gmail.com
Mon Aug 25 20:39:19 PDT 2014


On Monday, August 25, 2014 06:20:00 PM Jordan Justen wrote:
> On Tue, Aug 19, 2014 at 1:24 PM, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> > This series adds support for setting the default platform used by waffle
> > in piglit.conf. It seems like a rather long series, but the majority of
> > the series is either refactoring code to allow changes, or working on
> > the unit tests, (In fact, 11 of the 18 patches in this series much with
> > unit tests).
> >
> > Patches 2-7: Fix bugs in both the unit tests for loading piglit.conf,
> >              and the actual code for loading piglit files, these patches
> >                          are needed since without them patch 18 will exercise bugs
> >                          that they fix them.
> > Patches 8-15: Make additional cleanups to the tests for piglit.conf,
> > Patches 16-17: Do the legwork of the series, 16 makes the majority of
> >                the changes necessary to use the config file to set
> >                            default values, while patch 18 actually implements the
> >                            first user of this feature.
> >
> > This is available at my github:
> > https://github.com/dcbaker/piglit submit/conf-defaults
> >
> > Jordan -- This is what you wanted, correct?
> 
> Uh, yeah. Patch 18 anyway. :)

Like all good things it just takes a little time to get there :)

Thanks for reviewing this.

> 
> Series Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20140825/747a5759/attachment.sig>


More information about the Piglit mailing list