[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] input: move input-send-event into experimental
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH] input: move input-send-event into experimental namespace |
Date: |
Tue, 25 Nov 2014 09:35:26 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 |
On 11/25/2014 04:58 AM, Markus Armbruster wrote:
> Amos Kong <address@hidden> writes:
>
>> On Tue, Nov 25, 2014 at 09:06:34AM +0100, Markus Armbruster wrote:
>>> Gerd Hoffmann <address@hidden> writes:
>>>
>>>> Ongoing discussions on how we are going to specify the console,
>>>> so tag the command as experiemntal so we can refine things in
>>>> the 2.3 development cycle.
>>>>
>>>> Signed-off-by: Gerd Hoffmann <address@hidden>
>>>> ---
>>>> qmp-commands.hx | 12 ++++++------
>>>
>>> Don't you need to patch qapi-schema.json, too?
>>
>> Not necessary in function level.
>
> s/need to/want to/?
Yes, if you are going to mark the HMP command experimental, please also
mark the QMP command as experimental.
>
>>> Do we actually explain "x- means experimental" anywhere?
Hmm, In my (not yet finished) series for improving qapi generation type
safety, I have added:
$ git grep -A1 'x-' docs/
...
docs/qapi-code-gen.txt:Any command, type, or field name beginning with
"x-" is marked
docs/qapi-code-gen.txt-experimental, and may be withdrawn in a future
release. Downstream
...
but I obviously need to find time to revisit my series since that text
is not in upstream yet.
>>
>> What's the official way to make command experimental?
>
> I think we have an understanding / convention that an "x-" prefix marks
> names as unstable API. But I can't find it spelled out anywhere.
> Anyway, separate issue.
Yep, and on my plate to finish it.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature