Re: [Qemu-devel] [PATCH 1/1] monitor: Fix QMP ABI breakage around "id"

[Eric Blake]

On 06/08/2015 02:44 AM, Markus Armbruster wrote:
> Commit 65207c5 accidentally dropped a line of code we need along with
> a comment that became wrong then.  This made QMP reject "id":
> 
>     {"execute": "system_reset", "id": "1"}
>     {"error": {"class": "GenericError", "desc": "QMP input object member 'id' 
> is unexpected"}}
> 
> Put the lost line right back, so QMP again accepts and returns "id",
> as promised by the ABI:
> 
>     {"execute": "system_reset", "id": "1"}
>     {"return": {}, "id": "1"}
> 
> Reported-by: Fabio Fantoni <address@hidden>
> Signed-off-by: Markus Armbruster <address@hidden>
> Signed-off-by: Don Slutz <address@hidden>
> Tested-by: Fabio Fantoni <address@hidden>
> Signed-off-by: Wen Congyang <address@hidden>
> Signed-off-by: Pavel Fedin <address@hidden>
> Tested-by: Eric Blake <address@hidden>
> Signed-off-by: Markus Armbruster <address@hidden>

Double S-o-b by Markus is odd.  But it's already landed in master, I'm
too late for fixing it :)

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature