[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

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



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.