On 4/7/21 9:50 AM, Emanuele Giuseppe Esposito wrote:
Signed-off-by: Emanuele Giuseppe Esposito <[email protected]>
---
  python/qemu/machine.py | 2 +-
  python/qemu/qtest.py   | 4 +++-
  2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/python/qemu/machine.py b/python/qemu/machine.py
index c721e07d63..18d32ebe45 100644
--- a/python/qemu/machine.py
+++ b/python/qemu/machine.py
@@ -109,7 +109,7 @@ def __init__(self,
self._binary = binary
          self._args = list(args)
-        self._wrapper = wrapper
+        self._wrapper = list(wrapper)

Unrelated change?

(I'm assuming you want to copy the user's input to explicitly avoid sharing state. Commit message blurb for this would be good.)

          self._name = name or "qemu-%d" % os.getpid()
          self._test_dir = test_dir
diff --git a/python/qemu/qtest.py b/python/qemu/qtest.py
index 0d01715086..4c90daf430 100644
--- a/python/qemu/qtest.py
+++ b/python/qemu/qtest.py
@@ -111,6 +111,7 @@ class QEMUQtestMachine(QEMUMachine):
      def __init__(self,
                   binary: str,
                   args: Sequence[str] = (),
+                 wrapper: Sequence[str] = (),
                   name: Optional[str] = None,
                   test_dir: str = "/var/tmp",
                   socket_scm_helper: Optional[str] = None,
@@ -119,7 +120,8 @@ def __init__(self,
              name = "qemu-%d" % os.getpid()
          if sock_dir is None:
              sock_dir = test_dir
-        super().__init__(binary, args, name=name, test_dir=test_dir,
+        super().__init__(binary, args, wrapper=wrapper, name=name,
+                         test_dir=test_dir,
                           socket_scm_helper=socket_scm_helper,
                           sock_dir=sock_dir)
          self._qtest: Optional[QEMUQtestProtocol] = None


ACK


Reply via email to