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

Message ID 1361214044-6301-1-git-send-email-tom.gall@linaro.org
State New
Headers show

Commit Message

Tom Gall Feb. 18, 2013, 7 p.m.
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@linaro.org>
---
 tests/glslparsertest/glslparsertest.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch

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");