[Piglit] [PATCH 1/1] glslparsertest: On GLES don't require GL_ARB_ES2_compatibility

Tom Gall tom.gall at linaro.org
Mon Feb 18 11:00:44 PST 2013


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 <tom.gall at linaro.org>
---
 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



More information about the Piglit mailing list