[Piglit] [PATCH 13/13] summary.py: Remove work-arounds for version 0 result bugs

Ilia Mirkin imirkin at alum.mit.edu
Sat Jun 21 07:24:20 PDT 2014


On Sat, Jun 21, 2014 at 8:06 AM, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> These work-arounds are no longer necessary since they are handled by the
> result format updater on load.
>
> Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>

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

Is there also some stuff in the templates? Not sure, just wanted to mention it.

I'll look at 12/13 later today at some point.

> ---
>  framework/summary.py | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/framework/summary.py b/framework/summary.py
> index fa98267..0db2a80 100644
> --- a/framework/summary.py
> +++ b/framework/summary.py
> @@ -484,9 +484,6 @@ class Summary:
>                      if not path.exists(temp_path):
>                          os.makedirs(temp_path)
>
> -                    if isinstance(value.get('dmesg'), list):
> -                        value['dmesg'] = "\n".join(value['dmesg'])
> -
>                      if value.get('time') is not None:
>                          value['time'] = datetime.timedelta(0, value['time'])
>
> --
> 2.0.0
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list