On Tue, Apr 15, 2008 at 10:53:25PM +0100, Daniel P. Berrange wrote:
> A long time ago I proposed a syntax for serial / parallel port handling
> in libvirt XML.
Patch looks good to me +1
> + if (def->nserials == 0) {
> + obj = xmlXPathEval(BAD_CAST "/domain/devices/console", ctxt);
> + if ((obj != NULL) && (obj->type == XPATH_NODESET) &&
> + (obj->nodesetval != NULL) && (obj->nodesetval->nodeNr == 1)) {
just a small suggestion, use "/domain/devices/console[1]" in case someone
duplicates the <console/> tags
Also I think adding a new section in the XML format description need
to be added to the documentation. Just a copy of the mail explanations
would be sufficient, plus one example in the full domain context.
thanks !
Daniel
--
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard | virtualization library http://libvirt.org/
veillard@xxxxxxxxxx | libxml GNOME XML XSLT toolkit http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/
--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list