[Piglit] [Patch v2 1/2] piglit-summary-html.py: Fix regression from constant conversion

Ilia Mirkin imirkin at alum.mit.edu
Tue Apr 8 14:16:40 PDT 2014


On Tue, Apr 8, 2014 at 4:36 PM, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> This fixes a regression introduced in ec37e0fa951
>
> If piglit-summary-html.py is run with -e all it will except. This is
> because I missed converting the various statuses to constants from
> classes
>
> Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>

Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>

> ---
>  piglit-summary-html.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/piglit-summary-html.py b/piglit-summary-html.py
> index a37b337..4a51721 100755
> --- a/piglit-summary-html.py
> +++ b/piglit-summary-html.py
> @@ -71,8 +71,8 @@ def main():
>      if args.exclude_details:
>          # If exclude-results has all, then change it to be all
>          if 'all' in args.exclude_details:
> -            args.exclude_details = [status.Skip(), status.Pass(), status.Warn(),
> -                                    status.Crash(), status.Fail()]
> +            args.exclude_details = [status.SKIP, status.PASS, status.WARN,
> +                                    status.CRASH, status.FAIL]
>          else:
>              args.exclude_details = [status.status_lookup(i) for i in
>                                      args.exclude_details]
> --
> 1.9.1
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list