[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH] Revive -version 'QEMU PC Emulator...'
From: |
Cole Robinson |
Subject: |
[Qemu-devel] [PATCH] Revive -version 'QEMU PC Emulator...' |
Date: |
Wed, 12 May 2010 16:29:21 -0400 |
Commit f75ca1ae205f24dae296c82d534c37746f87232f changed the version
string from:
QEMU PC Emulator version x.yy.z
to
QEMU Emulator version x.yy.z
libvirt is overly sensitive to the format of this string, and barfs when
trying to parse qemu -help output. While libvirt should certainly be more
robust here, changing the output format of -version for cosmetic reasons
doesn't seem like the best idea, so let's revert the change and add a
comment explaining the issue.
Signed-off-by: Cole Robinson <address@hidden>
---
vl.c | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/vl.c b/vl.c
index 85bcc84..8592949 100644
--- a/vl.c
+++ b/vl.c
@@ -2012,7 +2012,9 @@ static void main_loop(void)
static void version(void)
{
- printf("QEMU emulator version " QEMU_VERSION QEMU_PKGVERSION ", Copyright
(c) 2003-2008 Fabrice Bellard\n");
+ /* Change with care, at least libvirt parses this string and is
+ * historically sensitive to its format */
+ printf("QEMU PC emulator version " QEMU_VERSION QEMU_PKGVERSION ",
Copyright (c) 2003-2008 Fabrice Bellard\n");
}
static void help(int exitcode)
--
1.6.6.1
- [Qemu-devel] [PATCH] Revive -version 'QEMU PC Emulator...',
Cole Robinson <=
- [Qemu-devel] Re: [PATCH] Revive -version 'QEMU PC Emulator...', Jes Sorensen, 2010/05/12
- [Qemu-devel] Re: [PATCH] Revive -version 'QEMU PC Emulator...', Cole Robinson, 2010/05/12
- [Qemu-devel] Re: [PATCH] Revive -version 'QEMU PC Emulator...', Jes Sorensen, 2010/05/13
- [Qemu-devel] Re: [PATCH] Revive -version 'QEMU PC Emulator...', Cole Robinson, 2010/05/13
- [Qemu-devel] Re: [PATCH] Revive -version 'QEMU PC Emulator...', Jes Sorensen, 2010/05/13
- Re: [Qemu-devel] Re: [PATCH] Revive -version 'QEMU PC Emulator...', Daniel P. Berrange, 2010/05/13
- Re: [Qemu-devel] Re: [PATCH] Revive -version 'QEMU PC Emulator...', Luiz Capitulino, 2010/05/13
- Re: [Qemu-devel] Re: [PATCH] Revive -version 'QEMU PC Emulator...', Paul Brook, 2010/05/13
- Re: [Qemu-devel] Re: [PATCH] Revive -version 'QEMU PC Emulator...', Blue Swirl, 2010/05/13
- Re: [Qemu-devel] Re: [PATCH] Revive -version 'QEMU PC Emulator...', Stuart Brady, 2010/05/13