[Piglit] [PATCH 3/4] all.py: fix arb_geometry_shader4-program-parameter-vertices-out tests

Ilia Mirkin imirkin at alum.mit.edu
Tue Apr 14 13:58:09 PDT 2015


I checked the test, and it does indeed expect these as separate
arguments. However it's not illegal for an argument to have a space in
it (or a binary -- what happens when I stick my piglit checkout into
'/My Awesome Directory With Spaces/piglit').

Patches 1-3: Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>
[haven't looked at 4 yet]

On Tue, Apr 14, 2015 at 4:52 PM, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> These had a space in them.
>
> Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
> ---
>  tests/all.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/all.py b/tests/all.py
> index a1d18d2..3612de8 100755
> --- a/tests/all.py
> +++ b/tests/all.py
> @@ -4000,8 +4000,8 @@ with profile.group_manager(
>                       'GL_TRIANGLES_ADJACENCY', 'GL_TRIANGLE_STRIP_ADJACENCY']:
>              g(['arb_geometry_shader4-ignore-adjacent-vertices', draw, prim])
>
> -    for mode in ['1', 'tf 1', 'max', 'tf max']:
> -        g(['arb_geometry_shader4-program-parameter-vertices-out', mode])
> +    for mode in [['1'], ['tf', '1'], ['max'], ['tf', 'max']]:
> +        g(['arb_geometry_shader4-program-parameter-vertices-out'] + mode)
>
>  with profile.group_manager(
>          PiglitGLTest,
> --
> 2.3.5
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list