On Mon, Dec 17, 2012 at 12:35 PM, Chad Versace
<chad.vers...@linux.intel.com> wrote:
> On 12/14/2012 12:01 PM, Tom Gall wrote:
>> This is a small change in two parts to repair support for
>> gles2 by glslparsertest.
>>
>> To glslparsertest.c, add 100 into the list of versions the switch
>> handles when passed from the command line instead of the default
>> path. Next in piglit_init when version 100 is used, check for
>> gles and if in use don't check for GL_ARB_ES2_compatibility.
>>
>> Rename the created testcase binary to be gles-2.0-glslparsertest
>> so it's consistant with the gles-3.0 tests. (As well as consistant
>> with the proposed new gles-2.0 tests)
>>
>> Signed-off-by: Tom Gall <tom.g...@linaro.org>
>> ---
>> tests/glslparsertest/CMakeLists.gles2.txt | 4 +---
>> tests/glslparsertest/glslparsertest.c | 6 +++++-
>> 2 files changed, 6 insertions(+), 4 deletions(-)
>>
>> diff --git a/tests/glslparsertest/CMakeLists.gles2.txt
>> b/tests/glslparsertest/CMakeLists.gles2.txt
>> index 1e37bed..7c24fcc 100644
>> --- a/tests/glslparsertest/CMakeLists.gles2.txt
>> +++ b/tests/glslparsertest/CMakeLists.gles2.txt
>> @@ -3,8 +3,6 @@ link_libraries(
>> ${OPENGL_gles2_LIBRARY}
>> )
>>
>> -piglit_add_executable(glslparsertest_${piglit_target_api}
>> - glslparsertest.c
>> - )
>> +piglit_add_executable(${piglit_target_api}-glslparsertest glslparsertest.c)
>
> Nak the name change. The naming convention used throughout Piglit for
> libraries,
> executables, and source is to append the basename with the api. For example,
> libpilitutil_gles2. This one executable, glslparsertest, doesn't warrant an
> exception.
Ok.
I'll fix up my gles2 submission and this one. FWIW the gles3
submission that went inhas gles3 prefixed the same way. So that should
be fixed too.
>>
>> # vim: ft=cmake:
>> diff --git a/tests/glslparsertest/glslparsertest.c
>> b/tests/glslparsertest/glslparsertest.c
>> index ff32da7..c91a56b 100644
>> --- a/tests/glslparsertest/glslparsertest.c
>> +++ b/tests/glslparsertest/glslparsertest.c
>> @@ -43,6 +43,10 @@ PIGLIT_GL_TEST_CONFIG_BEGIN
>> const unsigned int int_version = parse_glsl_version(argv[3]);
>>
>> switch (int_version) {
>> + case 100:
>> + config.supports_gl_compat_version = 10;
>> + config.supports_gl_es_version = 20;
>> + break;
>> case 110:
>> case 120:
>> case 130:
>> @@ -351,7 +355,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 (!piglit_is_gles() && requested_version == 100) {
>> piglit_require_extension("GL_ARB_ES2_compatibility");
>> } else if (requested_version == 300) {
>> piglit_require_extension("GL_ARB_ES3_compatibility");
>
> Analogous changes will later be for GLES3. I see that your patch
> series is solely focused on implementing GLES2 support, and that's fine. I'm
> just making a note of where future additional changes.
>
> Other than the name change, the patch is:
> Reviewed-by: Chad Versace <chad.vers...@linux.intel.com>
--
Regards,
Tom
"Where's the kaboom!? There was supposed to be an earth-shattering
kaboom!" Marvin Martian
Graphics Working Group | Linaro.org │ Open source software for ARM SoCs
w) tom.gall att linaro.org
h) tom_gall att mac.com
_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit
