Re: [Qemu-devel] [PATCH] qemu.py: Fix syntax error

2017-09-18 Thread Eduardo Habkost
On Mon, Sep 18, 2017 at 07:25:24AM +0200, Kevin Wolf wrote: > Python requires parentheses around multiline expression. This fixes the > breakage of all Python-based qemu-iotests cases that was introduced in > commit dab91d9aa0. > > Signed-off-by: Kevin Wolf > --- > > Eduardo, I think I'm going t

Re: [Qemu-devel] [PATCH] qemu.py: Fix syntax error

2017-09-18 Thread Peter Maydell
On 18 September 2017 at 06:25, Kevin Wolf wrote: > Python requires parentheses around multiline expression. This fixes the > breakage of all Python-based qemu-iotests cases that was introduced in > commit dab91d9aa0. > > Signed-off-by: Kevin Wolf > --- > > Eduardo, I think I'm going to include th

Re: [Qemu-devel] [PATCH] qemu.py: Fix syntax error

2017-09-18 Thread Alex Bennée
Kevin Wolf writes: > Python requires parentheses around multiline expression. This fixes the > breakage of all Python-based qemu-iotests cases that was introduced in > commit dab91d9aa0. > > Signed-off-by: Kevin Wolf Heh, just sent an identical patch. Reviewed-by: Alex Bennée Tested-by: Alex

Re: [Qemu-devel] [PATCH] qemu.py: Fix syntax error

2017-09-18 Thread Stefan Hajnoczi
On Mon, Sep 18, 2017 at 07:25:24AM +0200, Kevin Wolf wrote: > Python requires parentheses around multiline expression. This fixes the > breakage of all Python-based qemu-iotests cases that was introduced in > commit dab91d9aa0. > > Signed-off-by: Kevin Wolf > --- > > Eduardo, I think I'm going t

[Qemu-devel] [PATCH] qemu.py: Fix syntax error

2017-09-17 Thread Kevin Wolf
Python requires parentheses around multiline expression. This fixes the breakage of all Python-based qemu-iotests cases that was introduced in commit dab91d9aa0. Signed-off-by: Kevin Wolf --- Eduardo, I think I'm going to include this patch in a block layer pull request today, just to stop the C