From: Eric Blake <eblake@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>, qemu-devel@nongnu.org
Cc: patches@linaro.org, Stefan Hajnoczi <stefanha@redhat.com>,
Fabrice Bellard <fabrice@bellard.org>,
Anthony Liguori <aliguori@amazon.com>
Subject: Re: [Qemu-devel] [PATCH v2 for-2.7] Update ancient copyright string in -version output
Date: Thu, 4 Aug 2016 09:32:10 -0600 [thread overview]
Message-ID: <57A35FFA.8060003@redhat.com> (raw)
In-Reply-To: <1470309276-5012-1-git-send-email-peter.maydell@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 2117 bytes --]
On 08/04/2016 05:14 AM, Peter Maydell wrote:
> Currently the -version command line argument prints a string ending
> with "Copyright (c) 2003-2008 Fabrice Bellard". This is now some
> eight years out of date; abstract it out of the several places that
> print the string and update it to:
>
> Copyright (c) 2003-2016 Fabrice Bellard and the QEMU Project developers
>
> to reflect the work by all the QEMU Project contributors over the
> last decade.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
Reviewed-by: Eric Blake <eblake@redhat.com>
> The aim here is to (1) update the dates and (2) acknowledge
> the work of all our contributors. I'm open to bikeshedding
> on the exact wording (or on which header file we should
> put the #define in...)
>
> I only pulled out the copyright string proper into the #define
> because a GUI About box is going to want just that, with no
> leading ',' or trailing newline.
>
> Fabrice: I have cc'd you since this is proposing an update
> to your copyright info.
>
> v1->v2 changes: add the qemu-img.c string.
> ---
> bsd-user/main.c | 3 ++-
> include/qemu-common.h | 4 ++++
> linux-user/main.c | 2 +-
> qemu-img.c | 2 +-
> vl.c | 3 ++-
> 5 files changed, 10 insertions(+), 4 deletions(-)
qemu-io has a --version option that probably ought to output copyright
information, but as it currently does not do so, this patch need not be
the one that tweaks it.
qemu-nbd has a --version option, where the copyright info is currently:
Copyright (C) 2006 Anthony Liguori <anthony@codemonkey.ws>.
I don't know if we want to rework that one in light of this patch, or if
we can just blindly use this code. Anthony?
> +/* Copyright string for -version arguments, About dialogs, etc */
> +#define QEMU_COPYRIGHT "Copyright (c) 2003-2016 " \
> + "Fabrice Bellard and the QEMU Project developers"
> +
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]
next prev parent reply other threads:[~2016-08-04 15:32 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-04 11:14 [Qemu-devel] [PATCH v2 for-2.7] Update ancient copyright string in -version output Peter Maydell
2016-08-04 15:32 ` Eric Blake [this message]
2016-08-11 16:52 ` Peter Maydell
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=57A35FFA.8060003@redhat.com \
--to=eblake@redhat.com \
--cc=aliguori@amazon.com \
--cc=fabrice@bellard.org \
--cc=patches@linaro.org \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
/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.