Re: [libvirt] Extend virConfParser to allow parsing VMware VMX config files

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

 



On Fri, Jun 19, 2009 at 05:54:47PM +0200, Matthias Bolte wrote:
> 2009/6/19 Daniel P. Berrange <berrange@xxxxxxxxxx>:
> > On Fri, Jun 19, 2009 at 10:01:20AM +0200, Matthias Bolte wrote:
> >> Hi,
> >>
> >> As part of the VMware ESX driver I need to parse VMware VMX config
> >> files. This files have basically the same syntax as Xen config files,
> >> but the key part of the key=value format is allowed to contain ':' and
> >> '.'.
> >>
> >> The attached patch adds a flag to the virConfParser to enabled a VMX
> >> mode for the parser to allow ':' and '.' in the key, as suggested by
> >> Daniel P. Berrange on IRC.
> >
> > ACK, looks fine to me - though I'm wondering if you might also want to
> > explicitly turn off support for lists ? IIIRC the VMware config values
> > were only scalars, so for safety you might want the parser to refuse
> > to parse / generate lists.
> >
> > Daniel
> >
> 
> Good advice. I've done this now and attached the patch for it. The
> first patch got already commited, thanks to Daniel Veillard.

  Okay, makes sense too and the flag renaming is a good idea,

    applied and commited !

    thanks !

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]