Re: [Qemu-devel] [PATCH v3] QMP: Allow dot separated dict path arguments in qmp-shell

2014-02-27 Thread Luiz Capitulino
On Wed, 12 Feb 2014 11:05:13 +0800 Fam Zheng wrote: > As another convinience to allow using commands that expect a dict as > argument, this patch adds support for foo.bar=value syntax, similar to > command line argument style: > > (QEMU) blockdev-add options.driver=file options.id=drive1 > op

Re: [Qemu-devel] [PATCH v3] QMP: Allow dot separated dict path arguments in qmp-shell

2014-02-14 Thread Stefan Hajnoczi
On Wed, Feb 12, 2014 at 11:05:13AM +0800, Fam Zheng wrote: > As another convinience to allow using commands that expect a dict as > argument, this patch adds support for foo.bar=value syntax, similar to > command line argument style: > > (QEMU) blockdev-add options.driver=file options.id=drive1

Re: [Qemu-devel] [PATCH v3] QMP: Allow dot separated dict path arguments in qmp-shell

2014-02-12 Thread Fam Zheng
On Wed, 02/12 11:14, Eric Blake wrote: > On 02/11/2014 08:05 PM, Fam Zheng wrote: > > As another convinience to allow using commands that expect a dict as > > You missed this, even though I pointed it out in v2: > > s/convinience/convenience/ Oops, I'm sorry for that. Luiz, would you fix this wh

Re: [Qemu-devel] [PATCH v3] QMP: Allow dot separated dict path arguments in qmp-shell

2014-02-12 Thread Eric Blake
On 02/11/2014 08:05 PM, Fam Zheng wrote: > As another convinience to allow using commands that expect a dict as You missed this, even though I pointed it out in v2: s/convinience/convenience/ > argument, this patch adds support for foo.bar=value syntax, similar to > command line argument style:

[Qemu-devel] [PATCH v3] QMP: Allow dot separated dict path arguments in qmp-shell

2014-02-11 Thread Fam Zheng
As another convinience to allow using commands that expect a dict as argument, this patch adds support for foo.bar=value syntax, similar to command line argument style: (QEMU) blockdev-add options.driver=file options.id=drive1 options.filename=... Signed-off-by: Fam Zheng --- v3: Fix error me