Re: [spice v2 2/8] tests: migrate: use uri for default's spicy client

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 9/25/19 1:02 PM, Victor Toso wrote:
From: Victor Toso <me@xxxxxxxxxxxxxx>

Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx>
Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
---
  tests/migrate.py | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/migrate.py b/tests/migrate.py
index 0c302c92..5a0f5930 100755
--- a/tests/migrate.py
+++ b/tests/migrate.py
@@ -96,8 +96,8 @@ def start_qemu(qemu_exec, image, spice_port, qmp_filename, incoming_port=None, e
      return proc
def start_client(client, spice_port):
-    return Popen(("%(client)s -h localhost -p %(port)d" % dict(port=spice_port,
-        client=client)).split(), executable=client)
+    client_cmd = "spicy --uri spice://localhost:%s" % (spice_port)
+    return Popen(client_cmd.split(), executable=client)

Hi,

Since you remote-viewer is made available as a client on the next patch,
you can leave '%(client)s ' as is and not replace it with 'spicy'.

That also makes the next patch smaller.

Uri.

def wait_active(q, active):
      events = ["RESUME"] if active else ["STOP"]


_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]