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 <[email protected]>
[haven't looked at 4 yet]

On Tue, Apr 14, 2015 at 4:52 PM, Dylan Baker <[email protected]> wrote:
> These had a space in them.
>
> Signed-off-by: Dylan Baker <[email protected]>
> ---
>  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
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/piglit
_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to