Re: [Xen-devel] [PATCH 10/14] docs: update xl-disk-configuration.txt to describe new syntax

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Fri, 2011-05-20 at 11:21 +0100, Ian Jackson wrote:
> Ian Campbell writes ("Re: [Xen-devel] [PATCH 10/14] docs: update 
> xl-disk-configuration.txt to describe new syntax"):
> > I think it would be ok if you correct the * placements etc. Having some
> > sort of (semi-)formal syntax declaration in addition to the wordy
> > description is useful IMHO.
> > 
> > Would the syntax be clearer as a BNF style thing rather than a regexp-y
> > thing? e.g. this would allow "[<target>, [<format>, [<vdev>,
> > [<access>]]]]," to be declared as "POSITIONAL_PARAMETERS := ..." making
> > the syntax a little more self-documenting and "KEY := ..." could be
> > defined with reference to the relevant following sections etc.
> 
> The trouble with bnf is that it all comes out of order -
> self-assembly, so to speak.
> 
> > Should the second "<key>=<value>" usage also have a "|flag"?
> 
> Yes.
> 
> > I presume "key" can also == "target" and can be the non-ultimate key so
> > long as the value does not contain any problematic characters. In this
> > case the explicit target= needs to be marked as optional.
> 
> No, general "key" may not be "target".  "target=" always eats the rest
> of the string.

Oh, yes, that's pretty obvious now I think about it...



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel