|
|
|
|
|
|
|
|
|
|
xen-devel
Re: [Xen-devel] [PATCH 8/9] xl: xl block-attach -N (dry run) option
On Fri, 2011-06-03 at 12:16 +0100, Ian Jackson wrote:
[...]
> Just the entire structure.
[...]
> JSON would be a good concrete syntax.
[...]
> If we had a competent JSON pretty-printer it would do as the
> human-readable output too.
Should be pretty easy. QMP support in libxl is also going to require a
JSON library (for internal use by libxl, not exposed to libxl users),
which hopefully has functions which make the pretty part trivial.
> We shouldn't be encouraging XML :-).
:-D
> > What do you think? (I think I lean towards the final option, unless a
> > single hardcoded syntax is thought to be sufficient).
>
> The prototype you suggest is not sufficient because field members may
> be structures, so the whole thing needs to be recursive. If you want
> reasonable output from a recursive pretty-printer you need to pass an
> indent level etc.
Oh yeah. Add more params to callback as necessary ;-)
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
<Prev in Thread] |
Current Thread |
[Next in Thread>
|
- [Xen-devel] [PATCH 11/11] xl: new "check-xl-disk-parse" test script for disk parser, (continued)
- Re: [Xen-devel] [PATCH 10/11] xl: xl block-attach -N (dry run) option, Ian Campbell
- Re: [Xen-devel] [PATCH 09/11] xl: implement -N (dry run) global option for create and cpupool-create, Ian Campbell
- Re: [Xen-devel] [PATCH 08/11] xl: new global -N option for dry run, Ian Campbell
- Re: [Xen-devel] [PATCH 8/9] xl: xl block-attach -N (dry run) option, Ian Campbell
- Re: [Xen-devel] [PATCH 8/9] xl: xl block-attach -N (dry run) option, Ian Jackson
- Re: [Xen-devel] [PATCH 8/9] xl: xl block-attach -N (dry run) option, Ian Campbell
- Re: [Xen-devel] [PATCH 8/9] xl: xl block-attach -N (dry run) option, Ian Campbell
- Re: [Xen-devel] [PATCH 8/9] xl: xl block-attach -N (dry run) option, Ian Jackson
- Re: [Xen-devel] [PATCH 8/9] xl: xl block-attach -N (dry run) option,
Ian Campbell <=
- Re: [Xen-devel] [PATCH 7/9] libxl: disks: allow specification of "backendtype=phy|tap|qdisk", Ian Campbell
- Re: [Xen-devel] [PATCH 7/9] libxl: disks: allow specification of "backendtype=phy|tap|qdisk", Ian Jackson
- Re: [Xen-devel] [PATCH 7/9] libxl: disks: allow specification of "backendtype=phy|tap|qdisk", Ian Campbell
- Re: [Xen-devel] [PATCH 6/9] xl: disks: replace block-attach disk config parser with call to xlu_parse_disk, Ian Campbell
- Re: [Xen-devel] [PATCH 5/9] xl: disks: replace config file disk spec parser with call to xlu_disk_parse, Ian Campbell
- Re: [Xen-devel] [PATCH 4/9] docs: update xl-disk-configuration.txt to describe new syntax, Ian Campbell
- Re: [Xen-devel] [PATCH 3/9] libxl: disks: commit libxlu_disk_l.[ch] flex output, Ian Campbell
- Re: [Xen-devel] [PATCH 2/9] libxl: disks: new xlu_disk_parse function, Ian Campbell
- Re: [Xen-devel] [PATCH 2/9] libxl: disks: new xlu_disk_parse function, Ian Jackson
- Re: [Xen-devel] [PATCH 2/9] libxl: disks: new xlu_disk_parse function, Ian Campbell
|
|
|
|
|