From: "Daniel P. Berrange" <berrange@redhat.com>
To: Eric Blake <eblake@redhat.com>
Cc: qemu-devel@nongnu.org, Paolo Bonzini <pbonzini@redhat.com>,
"open list:Network Block Dev..." <qemu-block@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH 1/2] qemu-nbd: Update version string
Date: Fri, 21 Jul 2017 15:03:29 +0100 [thread overview]
Message-ID: <20170721140329.GR17693@redhat.com> (raw)
In-Reply-To: <20170721135047.25005-2-eblake@redhat.com>
On Fri, Jul 21, 2017 at 08:50:46AM -0500, Eric Blake wrote:
> qemu-io and qemu-img already mirror the qemu version string,
> time to make qemu-nbd do the same.
>
> Reported-by: 陳培泓 <pahome.chen@mirlab.org>
> Signed-off-by: Eric Blake <eblake@redhat.com>
> ---
> qemu-nbd.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/qemu-nbd.c b/qemu-nbd.c
> index 78d05bea2d..1086934a95 100644
> --- a/qemu-nbd.c
> +++ b/qemu-nbd.c
> @@ -36,6 +36,7 @@
> #include "io/channel-socket.h"
> #include "crypto/init.h"
> #include "trace/control.h"
> +#include "qemu-version.h"
>
> #include <getopt.h>
> #include <libgen.h>
> @@ -129,10 +130,10 @@ static void usage(const char *name)
> static void version(const char *name)
> {
> printf(
> -"%s version 0.0.1\n"
> +"%s " QEMU_VERSION QEMU_PKGVERSION "\n"
wow, suprised that bogus version has existed for so long :-)
> "Written by Anthony Liguori.\n"
> "\n"
> -"Copyright (C) 2006 Anthony Liguori <anthony@codemonkey.ws>.\n"
> +QEMU_COPYRIGHT "\n"
> "This is free software; see the source for copying conditions. There is NO\n"
> "warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.\n"
> , name);
Reviewed-by: Daniel P. Berrange <berrange@redhat.com>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
next prev parent reply other threads:[~2017-07-21 14:03 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-21 13:50 [Qemu-devel] [PATCH for-2.10 0/2] Update qemu-nbd version Eric Blake
2017-07-21 13:50 ` [Qemu-devel] [PATCH 1/2] qemu-nbd: Update version string Eric Blake
2017-07-21 14:03 ` Daniel P. Berrange [this message]
2017-07-21 13:50 ` [Qemu-devel] [PATCH 2/2] maint: Reorder include directives for qemu-{nbd, io} Eric Blake
2017-07-24 12:07 ` [Qemu-devel] [PATCH for-2.10 0/2] Update qemu-nbd version Eric Blake
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170721140329.GR17693@redhat.com \
--to=berrange@redhat.com \
--cc=eblake@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.