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

Chad Versace chad.versace at linux.intel.com
Wed Jan 16 12:55:41 PST 2013


On 01/16/2013 11:25 AM, Tom Gall wrote:
> 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>

Patch is committed.



More information about the Piglit mailing list