[Piglit] [PATCH] Alter shader_runner GL/GLSL ES version requirement syntax.

Tom Gall tom.gall at linaro.org
Wed Jan 16 11:25:52 PST 2013


Thanks for the discussion on this one. Let's move forward on it.

On Wed, Jan 16, 2013 at 12:47 PM, Chad Versace
<chad.versace at linux.intel.com> wrote:
> On 01/15/2013 07:25 PM, Stuart Abercrombie wrote:
>> The current syntax isn't compatible with the same shader_test
>> supporting GL and GLES in the future.
>>
>> Modify existing GL ES tests to use the new syntax, and remove
>> explicit #version directives, which will instead be inserted based on
>> GLSL >= requirements.
>>
>> v2 Added check for trailing chars. Formatting.
>> ---
>>  tests/shaders/shader_runner.c                      |   24 +++++++++----------
>>  .../spec/glsl-es-1.00/execution/sanity.shader_test |    7 +----
>>  .../spec/glsl-es-3.00/execution/sanity.shader_test |    8 +-----
>>  3 files changed, 15 insertions(+), 24 deletions(-)
>
> Reviewed-by: Chad Versace <chad.versace at linux.intel.com>
>

Reviewed-by: Tom Gall <tom.gall at linaro.org>

--
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


More information about the Piglit mailing list