[Piglit] [Patch v2 1/4] framework: allow loading a piglit.conf from more locations
Jordan Justen
jljusten at gmail.com
Tue Jul 15 16:44:31 PDT 2014
On Tue, Jul 15, 2014 at 7:02 AM, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> [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.
Should I be concerned if piglit.conf is missing?
I suspect not, and therefore it doesn't seem like a warning is needed.
-Jordan
More information about the Piglit
mailing list