[Piglit] [PATCH 1/5] glsl-es-1.00: move contents of glsl-1.00 to glsl-es-1.00

Paul Berry stereotype441 at gmail.com
Thu Feb 14 15:12:05 PST 2013


On 14 February 2013 15:03, Tom Gall <tom.gall at linaro.org> wrote:

> Change all.tests to reflect new location of glsl-1.00 to
> glsl-es-1.00 and add es as part of the group name.
>
> In all.tests, add glsl-es-1.00/execution as a shader test.
>
> Move contents of glsl-1.00/compiler/arithmetic-operators/
> to glsl-es-1.00/compiler/arithmetic-operators.
>
> Remove blank line at end of
> tests/spec/glsl-1.00/compiler/assignment-operators/modulus-assign-00.frag
>
> v2: switch to glsl-es-1.00 instead of glsl-1.00 based on list
> discussion
>
> Signed-off-by: Tom Gall <tom.gall at linaro.org>
>

I assume you split this into 5 patches in order to avoid the mailing list's
size limit?

If so, I would recommend squashing it down to one patch and generating the
e-mail with "git format-patch --find-renames" so that the diff is small.
That will make it much easier to review.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20130214/c438009d/attachment.html>


More information about the Piglit mailing list