[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [PATCH v2 00/24] block: Command line option -blockdev

From: Markus Armbruster
Subject: Re: [Qemu-block] [PATCH v2 00/24] block: Command line option -blockdev
Date: Tue, 28 Feb 2017 22:55:57 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

Markus Armbruster <address@hidden> writes:

> Actually, the command line option is the least part of this series.
> Its bulk is about building infrastructure and getting errors out of
> the JSON parser[*].  The latter part could be spun out in its own
> series, if that helps.  We'll see.
> The design of the command line interface was discussed here:
> Subject: Non-flat command line option argument syntax
> Message-ID: <address@hidden>
> https://lists.gnu.org/archive/html/qemu-devel/2017-02/msg00555.html
> This series is based on "[PULL 00/26] QAPI patches for 2017-02-28".
> Available as branch blockdev-cmdline in my public repo
> <http://repo.or.cz/w/qemu/armbru.git>.
> [*] Who'd have thunk anyone actually wants to know what's wrong with
> his JSON!
> v2: Address review by Eric and Kevin

Forgot to mention: patches reordered, PATCH 21-23 are now PATCH 17-19.

> PATCH 03: Comment fixes, memory leak plugged
> PATCH 05: Comment fix
> PATCH 08+12: Announce followup work in commit message 
> PATCH 18: Comment fix
> PATCH 20: A missing test for failure fixed
> PATCH 21: Option help text improved, loop to call qmp_blockdev_add()
>           moved to the right place
> PATCH 22: Commit message fix
> PATCH 23: Commit message fix, qobject_input_get_keyval() fixed to set
>           v->keyval, memory leaks plugged, test case tweaked to cover
>         leading zero, keyval_listify() thinko fixed (this is the
>         only non-trivial change in v2)

reply via email to

[Prev in Thread] Current Thread [Next in Thread]