[Piglit] [PATCH] all.tests: remove glean/teapot

Brian Paul brianp at vmware.com
Wed Oct 10 07:52:28 PDT 2012


On 10/09/2012 06:26 PM, Marek Olšák wrote:
> It doesn't test anything.
> ---
>   tests/all.tests |    1 -
>   1 file changed, 1 deletion(-)
>
> diff --git a/tests/all.tests b/tests/all.tests
> index 964c84c..2c99171 100644
> --- a/tests/all.tests
> +++ b/tests/all.tests
> @@ -115,7 +115,6 @@ glean['rgbTriStrip'] = GleanTest('rgbTriStrip', res_dir)
>   glean['scissor'] = GleanTest('scissor', res_dir)
>   glean['shaderAPI'] = GleanTest('shaderAPI', res_dir)
>   glean['stencil2'] = GleanTest('stencil2', res_dir)
> -glean['teapot'] = GleanTest('teapot', res_dir)
>   glean['texCombine'] = GleanTest('texCombine', res_dir)
>   glean['texCube'] = GleanTest('texCube', res_dir)
>   glean['texEnv'] = GleanTest('texEnv', res_dir)

I believe that teapot was added as a performance test (several glean 
tests are perf tests).

The focus of piglit has been correctness but I think it would be nice 
to develop a suite of performance measurement tests too.  But that's a 
whole other discussion.

I'm fine with this patch though.

Reviewed-by: Brian Paul <brianp at vmware.com>


More information about the Piglit mailing list