Re: [Qemu-devel] [PATCH 2/2] qapi: Create an include directive for use in the JSON description files.

2014-03-26 Thread Wenchao Xia
Just saw the discuss of Lluís's patch, either { 'include': './include/include.json' } or include(include-cycle-b.json) is OK to me, but hope only one way is kepted in the end.

Re: [Qemu-devel] [PATCH 2/2] qapi: Create an include directive for use in the JSON description files.

2014-03-26 Thread Eric Blake
On 03/21/2014 10:01 AM, Benoît Canet wrote: The new directive in the form { 'include': 'path/to/file.json' } will trigger the parsing of path/to/file.json. The directive will be replaced by the result of the parsing. This will allow for easy modularisation of qapi JSON descriptions files.

Re: [Qemu-devel] [PATCH 2/2] qapi: Create an include directive for use in the JSON description files.

2014-03-26 Thread Eric Blake
On 03/26/2014 07:49 PM, Wenchao Xia wrote: Just saw the discuss of Lluís's patch, either { 'include': './include/include.json' } or include(include-cycle-b.json) is OK to me, but hope only one way is kepted in the end. We already had this discussion; we want the { 'include':

[Qemu-devel] [PATCH 2/2] qapi: Create an include directive for use in the JSON description files.

2014-03-21 Thread Benoît Canet
The new directive in the form { 'include': 'path/to/file.json' } will trigger the parsing of path/to/file.json. The directive will be replaced by the result of the parsing. This will allow for easy modularisation of qapi JSON descriptions files. The qapi commands now takes the input file path