[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] qemu-img: Add --pkgversion common option
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] qemu-img: Add --pkgversion common option |
Date: |
Thu, 08 Jan 2015 09:41:20 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 |
On 01/08/2015 07:40 AM, Peter Maydell wrote:
> On 8 January 2015 at 14:30, Don Slutz <address@hidden> wrote:
>> This will output the QEMU_PKGVERSION which is set by configure's
>> --with-pkgversion=
>
>> + /* Report pkgversion */
>> + if (!strcmp(cmdname, "--pkgversion")) {
>> + argc--; argv++; /* Drop special arg. */
>> + printf("qemu-img version " QEMU_VERSION
>> + " pkgversion " QEMU_PKGVERSION "\n");
>> + if (argc < 2) {
>> + return 0;
>> + }
>> + cmdname = argv[1]; /* Update */
>> + }
>
> Wouldn't it be better to just report the QEMU_PKGVERSION
> as part of the version info we already report in --version?
> This would be consistent with both linux-user and the softmmu
> emulators, which do:
>
> printf("QEMU emulator version " QEMU_VERSION QEMU_PKGVERSION ",
> Copyright (c) 2003-2008 Fabrice Bellard\n");
I agree - output the information unconditionally on --version, rather
than inventing a new option.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature