]> xenbits.xensource.com Git - qemu-upstream-4.4-testing.git/commitdiff
pty: unbreak libvirt
authorGerd Hoffmann <kraxel@redhat.com>
Thu, 3 Jan 2013 13:23:03 +0000 (14:23 +0100)
committerAnthony Liguori <aliguori@us.ibm.com>
Thu, 3 Jan 2013 18:53:41 +0000 (12:53 -0600)
Commit 586502189edf9fd0f89a83de96717a2ea826fdb0 breaks libvirt pty
support because it tried to figure the pts name from stderr output.

Fix this by moving the label to the end of the line, this way the
libvirt parser does still recognise the message.  libvirt looks
for "char device redirected to ${ptsname}<whitespace>".

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
qemu-char.c

index 331ad5c087e900397aaffc9fec8bfba8e18f117e..f41788c9efbf3b0d05b635cd5d82721768821bc6 100644 (file)
@@ -1012,10 +1012,11 @@ static CharDriverState *qemu_chr_open_pty(QemuOpts *opts)
     qemu_opt_set(opts, "path", q_ptsname(master_fd));
 
     label = qemu_opts_id(opts);
-    fprintf(stderr, "char device%s%s redirected to %s\n",
-            label ? " " : "",
-            label ?: "",
-            q_ptsname(master_fd));
+    fprintf(stderr, "char device redirected to %s%s%s%s\n",
+            q_ptsname(master_fd),
+            label ? " (label " : "",
+            label ? label      : "",
+            label ? ")"        : "");
 
     s = g_malloc0(sizeof(PtyCharDriver));
     chr->opaque = s;