[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt |
Date: |
Thu, 26 Jun 2014 07:11:47 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 |
On 06/26/2014 07:01 AM, Paolo Bonzini wrote:
> Il 26/06/2014 14:46, Luiz Capitulino ha scritto:
>>> >
>>> > Michal's hack at least ensures that we have event documentation,
>>> even if
>>> > the format changed compared to the 2.0 docs, and even if we don't have
>>> > time to get something better in place before 2.1 goes out. So all of
>>> > this conversation on ways to do better is nice, but if we don't get
>>> > there quickly, I could at least live with Michal's patch in the
>>> short term.
>> I honestly don't know what's the best thing to do here. I can live with
>> this patch too, although having some introductory comments at the top of
>> the file saying what it is would be nicer.
>
> Can we just revert to the pre-conversion qmp-events.txt file? This is
> similar to the situation with qmp-commands.hx.
As in, carrying redundant information between two files? True that it's
no worse than qmp-commands.hx; it would also ensure we aren't regressing
on quality of user documentation in relation to prior releases. We
could do the restore of that file, but if we do, we should fix the bugs
in pre-conversion qmp-events.txt (a couple events were undocumented,
there were some grammar errors, all fixed as part of the conversion), as
well as document events that have been added post-conversion.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Michal Privoznik, 2014/06/25
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Paolo Bonzini, 2014/06/25
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Eric Blake, 2014/06/25
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Markus Armbruster, 2014/06/25
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Luiz Capitulino, 2014/06/25
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Eric Blake, 2014/06/25
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Luiz Capitulino, 2014/06/26
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Paolo Bonzini, 2014/06/26
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt,
Eric Blake <=
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Paolo Bonzini, 2014/06/26
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Luiz Capitulino, 2014/06/26
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Paolo Bonzini, 2014/06/26
- [Qemu-devel] [PATCH] qmp: add qmp-events.txt back, Luiz Capitulino, 2014/06/26
- Re: [Qemu-devel] [PATCH] qmp: add qmp-events.txt back, Paolo Bonzini, 2014/06/26
- Re: [Qemu-devel] [PATCH] qmp: add qmp-events.txt back, Eric Blake, 2014/06/26
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Eric Blake, 2014/06/26
- Re: [Qemu-devel] [PATCH] Makefile: Install qmp-events.txt, Paolo Bonzini, 2014/06/26