[Piglit] [PATCH 2/2] Removed Glean read pixels sanity test.

Laura Ekstrand laura at jlekstrand.net
Mon Oct 20 15:49:24 PDT 2014


Thanks for the catch.  I took a look at sanity.py, and though I don't know
much Python yet, it looks like there are only two tests in there.  The
Glean basic test will get deleted soon because my ultimate goal is to get
rid of Glean altogether.  Since readPixSanity is the only other test, do we
really need sanity.py anymore?  (I sent you this in an IRC private message
on this subject this morning.)

Thanks.

Laura

On Sun, Oct 19, 2014 at 8:07 PM, Michel Dänzer <michel at daenzer.net> wrote:

> On 18.10.2014 06:24, Laura Ekstrand wrote:
>
>>
>> diff --git a/tests/all.py b/tests/all.py
>> index 6c5588a..eb317de 100644
>> --- a/tests/all.py
>> +++ b/tests/all.py
>> @@ -126,7 +126,6 @@ glean['pointAtten'] = GleanTest('pointAtten')
>>   glean['pointSprite'] = GleanTest('pointSprite')
>>   # exactRGBA is not included intentionally, because it's too strict and
>>   # the equivalent functionality is covered by other tests
>> -glean['readPixSanity'] = GleanTest('readPixSanity')
>>   glean['shaderAPI'] = GleanTest('shaderAPI')
>>   glean['stencil2'] = GleanTest('stencil2')
>>   glean['texCombine'] = GleanTest('texCombine')
>>
>
> This also needs to be removed from tests/sanity.py, and the new test
> should probably be added there instead.
>
>
> --
> Earthling Michel Dänzer            |                  http://www.amd.com
> Libre software enthusiast          |                Mesa and X developer
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20141020/d246dc9d/attachment.html>


More information about the Piglit mailing list