[Mesa-dev] [PATCH] all.tests: add ignore for missing configuration files.

[Ian Romanick]

On 01/08/2012 01:25 PM, Tolga Dalman wrote:
> On 01/06/12 14:44, Dylan Baker wrote:
>> if you don't have an /etc/drirc or ~/.drirc then all of the tests will
>> give a status of 'warn', this addes that warning to the ignored list.
>> ---
>> tests/all.tests | 1 +
>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/tests/all.tests b/tests/all.tests
>> index b713de0..f8d942b 100644
>> --- a/tests/all.tests
>> +++ b/tests/all.tests
>> @@ -2077,3 +2077,4 @@
>> Test.ignoreErrors.append(re.compile("stw_(init|cleanup).*"))
>> Test.ignoreErrors.append(re.compile("OpenGLInfo..*"))
>> Test.ignoreErrors.append(re.compile("AdapterInfo..*"))
>> Test.ignoreErrors.append(re.compile("No memory leaks detected."))
>> +Test.ignoreErrors.append(re.complie("libGL: Can't open configuration
>> file*"))
>
> s/complie/compile/

And I think it should end with 'file.*', not 'file*'.  The later will 
let it end with zero or more e's, not zero or more arbitrary characters. 
  Did you verify that this patch actually fixes the issue?

Also, piglit patches should be sent to piglit at lists.freedesktop.org.

> Best regards
> Tolga Dalman
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev