[Intel-gfx] [PATCH i-g-t] list-workarounds: Fix python 2 print statement

Dylan Baker baker.dylan.c at gmail.com
Mon Feb 8 18:16:16 UTC 2016


Looks good to me.

Reviewed-by: Dylan Baker <dylanx.c.baker at intel.com>

Quoting Damien Lespiau (2016-02-08 04:08:29)
> That script is a python 3 script, so we can't use the python 2 print
> statement, it's a function now.
> 
> I missed it in the review because reviewing a diff without additional
> context gives you a partial story.
> 
> Cc: Sameer Kibey <sameer.kibey at intel.com>
> Cc: Dylan Baker <baker.dylan.c at gmail.com>
> Signed-off-by: Damien Lespiau <damien.lespiau at intel.com>
> ---
>  scripts/list-workarounds | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/scripts/list-workarounds b/scripts/list-workarounds
> index 8b41ae5..70c026d 100755
> --- a/scripts/list-workarounds
> +++ b/scripts/list-workarounds
> @@ -96,7 +96,7 @@ def print_workarounds(project_root, driver_dir, project):
>                 sys.exit(1)
>  
>         parse(work_arounds)
> -       print "\nList of workarounds found in %s:" % project
> +       print("\nList of workarounds found in %s:" % project)
>         for wa in sorted(workarounds.keys()):
>                 if not options.platform:
>                         print("%s: %s" % (wa, ', '.join(workarounds[wa])))
> -- 
> 2.4.3
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/intel-gfx/attachments/20160208/70fd8e90/attachment-0001.sig>


More information about the Intel-gfx mailing list