[Piglit] [PATCH] framework/exectest.py: Set returncode to 0 when executable is missing.

Brian Paul brianp at vmware.com
Tue Jun 10 12:26:39 PDT 2014


On 06/10/2014 01:16 PM, jfonseca at vmware.com wrote:
> From: José Fonseca <jfonseca at vmware.com>
>
> Just like piglit_report_result(PIGLIT_SKIP) does.
>
> Otherwise these tests are considered failures or -- with my commit
> 058e0f8a1e536b68ef43d27ada80645845a39e19 -- crashes.
> ---
>   framework/exectest.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/framework/exectest.py b/framework/exectest.py
> index a833066..a5e06ae 100644
> --- a/framework/exectest.py
> +++ b/framework/exectest.py
> @@ -205,7 +205,7 @@ class Test(object):
>                   out = ("PIGLIT: {'result': 'skip'}\n"
>                          "Test executable not found.\n")
>                   err = ""
> -                returncode = None
> +                returncode = 0
>               else:
>                   raise e
>
>

Reviewed-by: Brian Paul <brianp at vmware.com>



More information about the Piglit mailing list