Re: [Qemu-devel] Merging the quorum block driver

[Daniel P. Berrange]

On Thu, Sep 19, 2013 at 08:51:39AM -0600, Eric Blake wrote:
> On 09/19/2013 07:21 AM, Kevin Wolf wrote:
> 
> > 
> > I was thinking more along the lines of:
> > 
> >     -drive if=none,file.driver=quorum,format=raw,\
> >            file.children[0].file.filename=/nfs1/test.qcow2, \
> 
> Note that this requires shell quoting to protect you from globbing
> changing the argv handed to qemu based on an (unusual) file name in the
> current directory.  Not necessarily the end of the world (someone like
> libvirt that uses exec rather than a shell command line isn't bothered
> by it, and the command line is so long that most users will not be
> typing it directly), but worth thinking about whether we can come up
> with an alternate syntax that doesn't require shell quoting.  (As is,
> your use of \-newline and whitespace is not quite right for how the
> actual command line would appear; but it does make discussion easier in
> the meantime)

Just don't use brackets - use a . for the array index separator too
eg

 -drive if=none,file.driver=quorum,format=raw,\
    file.children.0.file.filename=foo,\
    file.children.1.file.filename=bar,\
    file.children.2.file.filename=wiz,\

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|