On 27.6.2018 13:47, Pavel Hrdina wrote:
On Wed, Jun 27, 2018 at 01:03:03PM +0200, Miro Hrončok wrote:
> From 0d3b7cc9ba787a139ecbe6dac490b1f5bb021b7f Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Miro=20Hron=C4=8Dok?= <miro(a)hroncok.cz>
> Date: Wed, 27 Jun 2018 13:00:28 +0200
> Subject: [PATCH] Fix syntax error on Python 3.7
>
> async is a keyword now.
>
> `asyncio import ensure_future` works on Python 3.4 to 3.7
>
> `from asyncio import async as ensure_future` is not needed.
> ---
> libvirtaio.py | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
Thanks for the patch, but as noted in this review [1] using only
'ensure_future' is not good enough.
Pavel
[1] <
https://www.redhat.com/archives/libvir-list/2018-June/msg01700.html>
In that case you need to get async dynamically:
From 70e4ab8cc02019487b9740812d68fd9d9d021192 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Miro=20Hron=C4=8Dok?= <miro(a)hroncok.cz>
Date: Wed, 27 Jun 2018 13:00:28 +0200
Subject: [PATCH] Fix syntax error on Python 3.7
async is a keyword now.
`asyncio import ensure_future` works on Python 3.4.4 to 3.7.
`from asyncio import async as ensure_future` is needed on Debian
with Python 3.4.2, but we cannot type that, so we use getattr instead.
---
libvirtaio.py | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/libvirtaio.py b/libvirtaio.py
index 1c432dd..45fa0c5 100644
--- a/libvirtaio.py
+++ b/libvirtaio.py
@@ -46,7 +46,10 @@ import libvirt
try:
from asyncio import ensure_future
except ImportError:
- from asyncio import async as ensure_future
+ # debina has python 3.4.2 without the above
+ # we cannot from asyncio import async, because of python 3.7+
+ import asyncio
+ ensure_future = getattr(asyncio, 'async')
class Callback(object):
--
2.17.0
--
Miro Hrončok
--
Phone: +420777974800
IRC: mhroncok