[Piglit] [PATCH 2/2] framework: Don't try to parse GL/GLSL version from wflinfo.

[Ilia Mirkin]

There's a bin/glinfo that it should be using. Relying on
glxinfo/wflinfo/etc seems totally inappropriate.

On Mon, Dec 7, 2015 at 9:26 AM, Jose Fonseca <jfonseca at vmware.com> wrote:
> It's making many assumptions about the wflinfo which are not true.
>
> So completely disable it as a workaround.
>
> Though I wonder if there's really any merit in adding a depending on
> wflinfo.  IMO, if piglit cares for the advertised GL/GLSL versions, it
> should have its own internally utility program to dump every thing needed.
> Parsing output from external utilities is begging for trouble.
> ---
>  framework/test/opengl.py | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/framework/test/opengl.py b/framework/test/opengl.py
> index 29da2d1..0a7b2d7 100644
> --- a/framework/test/opengl.py
> +++ b/framework/test/opengl.py
> @@ -80,6 +80,11 @@ class WflInfo(object):
>          opts -- arguments to pass to wflinfo other than verbose and platform
>
>          """
> +
> +        # FIXME: The version parsing below is full of bugs.  Disable all
> +        # wflinfo invocation until they are addressed
> +        raise StopWflinfo('OSError')
> +
>          with open(os.devnull, 'w') as d:
>              try:
>                  raw = subprocess.check_output(
> --
> 2.5.0
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit