glslparsertest.c has a bug where it assumes that when the
requested glsl version is 1.00 the GL_ARB_ES2_compatibility
extension must be present. For GLES2 this should not be
required. The piglit_is_gles() call
checks the version string from glGetString(GL_VERSION).

Signed-off-by: Tom Gall <[email protected]>
---
 tests/glslparsertest/glslparsertest.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/glslparsertest/glslparsertest.c 
b/tests/glslparsertest/glslparsertest.c
index 1fd75d1..6e638da 100644
--- a/tests/glslparsertest/glslparsertest.c
+++ b/tests/glslparsertest/glslparsertest.c
@@ -385,7 +385,7 @@ piglit_init(int argc, char**argv)
        if (glsl_version_string != NULL)
                glsl_version = parse_glsl_version(glsl_version_string);
 
-       if (requested_version == 100) {
+       if (requested_version == 100 && !piglit_is_gles()) {
                piglit_require_extension("GL_ARB_ES2_compatibility");
        } else if (requested_version == 300) {
                piglit_require_extension("GL_ARB_ES3_compatibility");
-- 
1.7.10.4

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to