|
|
|
|
|
|
|
|
|
|
xen-devel
Re: [Xen-devel] [PATCH] Replace config file parser for "xl"
On 02/03/10 19:24, Ian Jackson wrote:
This provides a replacement config file parser for "xl" based on bison
and flex.
Benefits:
* proper error reporting with line numbers
* parser can understand nearly all "xm" configuration files directly
(doesn't understand Python code but should do everything else)
* parser also understands the ;-infested "xl" style files
* removes the dependency on libconfig
* better checking for certain kinds of mistakes
* eliminates the strange "massage file and try again" code
This is intended to support all config files currently supported by
"xl" and almost all files supported by "xm". (NB that whether a
feature works depends on the implementation of that feature in
xl/libxl of course.)
This patch also introduces a new library "libxlutil" which is mainly
for the benefit of "xl". Users of libxl do not need to use libxlutil,
but they can do so if they want to parse "xl" files without being
"xl".
this is merely replacing a library dependency by a compilation
dependency, plus the burden on maintaining a ad-hoc grammar. At least it
would probably be useful to take the same approch as the linux kernel,
i.e. checking bison/flex output in the repository to account for bison
annoying volatility.
however since it's well separated from libxl,
Acked-by: Vincent Hanquez <vincent.hanquez@xxxxxxxxxxxxx>
--
Vincent
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
|
|