[Piglit] [PATCH 1/2] Ported the glsl1 test from Glean to Piglit.

Laura Ekstrand laura at jlekstrand.net
Wed Oct 15 11:24:38 PDT 2014


It's really not a good use of time to check for overlaps between the glsl1
tests and the shader runner tests, because that would take a lot of
searching.  So, the basic question is: do we want to convert these tests to
shader runner (and not care about overlap), keep glsl1.c, or just throw
glsl1.c away?

I'm honestly not sure how to answer this question.  These shader tests are
pretty basic, but there are a lot of them, and they might hit on places
that piglit doesn't already cover.

On Wed, Oct 15, 2014 at 9:39 AM, Matt Turner <mattst88 at gmail.com> wrote:

> On Wed, Oct 15, 2014 at 8:13 AM, Brian Paul <brianp at vmware.com> wrote:
> > I'm not sure that porting this test to Piglit is worthwhile.  I'd bet
> that
> > most of the little subtests here are already sufficiently covered by
> > existing piglit shader tests.  It would probably be simpler to make
> shader
> > test scripts for the few cases here that might not be covered already.
>
> I've already deleted probably a hundred tests from this that are clear
> duplicated of existing shader_tests (though there might be some more
> as well).
>
> I completely agree that we'd really like to replace these tests with
> individual shader_tests where possible.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20141015/046041b6/attachment.html>


More information about the Piglit mailing list