[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 09/11] authz: add QAuthZListFile object type
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v7 09/11] authz: add QAuthZListFile object type for a file access control list |
Date: |
Thu, 6 Dec 2018 18:48:23 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.3.1 |
On 12/6/18 6:07 PM, Eric Blake wrote:
> On 12/6/18 9:58 AM, Philippe Mathieu-Daudé wrote:
>
>>>>> "refresh": "yes"
>>>>
>>>> I had to use "refresh": yes (value unquoted) to avoid:
>>>
>>> Do you mean "refresh":true? (bare yes is not valid JSON, but bare true
>>> is the correct way for a JSON bool).
>>
>> I first tried 'true' then was curious and tried 'yes' and it worked
>> too...
>
> Ah, so you're using a python wrapper rather than direct QMP, and
> benefitting from a rather-forgiving parser that turns yes into the
> correct QMP true over the wire to the actual monitor.
I'm not sure I'm using a python wrapper, I use rlwrap following:
https://wiki.qemu.org/Documentation/QMP#Trying_it
Regardless, let's use the correct JSON bool for the documentation.