[Piglit] [Patch v2 1/4] framework: allow loading a piglit.conf from more locations

Dylan Baker baker.dylan.c at gmail.com
Tue Jul 15 07:02:36 PDT 2014


[snip]
> > +        else:
> > +            if __debug__:
> > +                print('Warning: piglit.conf not found!\n'
> > +                      '(searching current dir, $HOME/.config, '
> > +                      '$XDG_CONFIG_HOME, and piglit source dir)',
> > +                      file=sys.stderr)
> 
> Did you mean to strip this debug warning? I don't think a warning
> message is needed if piglit.conf doesn't exist.
> 
> -Jordan

I had meant to leave it, I thought it was useful to have a warning that piglit 
hadn't found a config file, but if it bothers people (I know that Carl had 
also mentioned it annoyed him) I don't have a problem removing it.

[snip]
-------------- 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/20140715/ceaa561d/attachment-0001.sig>


More information about the Piglit mailing list