Re: [Qemu-devel] [PATCH v2 02/10] iotests: Fix 237 for Python 2.x

[Philippe Mathieu-Daudé]

On 1/30/19 4:24 PM, Max Reitz wrote:
> math.ceil() returns an integer on Python 3.x, but a float on Python 2.x.
> range() always needs integers, so we need an explicit conversion on 2.x
> (which does not hurt on 3.x).
> 
> It is not quite clear whether we want to support Python 2.x for any
> prolonged time, but this may as well be fixed along with the other
> issues some iotests have right now.
> 
> Signed-off-by: Max Reitz <address@hidden>
> Reviewed-by: John Snow <address@hidden>

Reviewed-by: Philippe Mathieu-Daudé <address@hidden>

> ---
>  tests/qemu-iotests/237 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/qemu-iotests/237 b/tests/qemu-iotests/237
> index fe0dd0f461..06897f8c87 100755
> --- a/tests/qemu-iotests/237
> +++ b/tests/qemu-iotests/237
> @@ -224,7 +224,7 @@ with iotests.FilePath('t.vmdk') as disk_path, \
>              iotests.log("= %s %d =" % (subfmt, size))
>              iotests.log("")
>  
> -            num_extents = math.ceil(size / 2.0**31)
> +            num_extents = int(math.ceil(size / 2.0**31))
>              extents = [ "ext%d" % (i) for i in range(1, num_extents + 1) ]
>  
>              vm.launch()
>