Re: [Intel-gfx] [PATCH] list-workarounds: Convert to python3

2013-05-11 Thread Ben Widawsky
On Sat, May 11, 2013 at 05:38:03PM +0100, Damien Lespiau wrote: > On Fri, May 10, 2013 at 03:28:29PM -0700, Ben Widawsky wrote: > > - print("%s: %s" % (wa, ', '.join(workarounds[wa]))) > > + print(("%s: %s" % (wa, ', '.join(workarounds[wa] > > This one adds

Re: [Intel-gfx] [PATCH] list-workarounds: Convert to python3

2013-05-11 Thread Damien Lespiau
On Fri, May 10, 2013 at 03:28:29PM -0700, Ben Widawsky wrote: > - print("%s: %s" % (wa, ', '.join(workarounds[wa]))) > + print(("%s: %s" % (wa, ', '.join(workarounds[wa] This one adds extra () that are not necessary. Other than that, ship it! Reviewed-b

Re: [Intel-gfx] [PATCH] list-workarounds: Convert to python3

2013-05-10 Thread Kenneth Graunke
On 05/10/2013 03:28 PM, Ben Widawsky wrote: The rest of the tool suite that uses python already uses python3. The tool configure requires python >= 3 (which is confusing because of the no backward compat problem). The world is slowly moving to python3. Converted with 2to3. CC: Damien Lespiau

[Intel-gfx] [PATCH] list-workarounds: Convert to python3

2013-05-10 Thread Ben Widawsky
The rest of the tool suite that uses python already uses python3. The tool configure requires python >= 3 (which is confusing because of the no backward compat problem). The world is slowly moving to python3. Converted with 2to3. CC: Damien Lespiau CC: Kenneth Graunke Signed-off-by: Ben Widaws