Re: [xml] [Libvir] [PATCH] virDomainMigrate version 4 (for discussion only!)
- From: Daniel Veillard <veillard redhat com>
- To: "Richard W.M. Jones" <rjones redhat com>
- Cc: libvir-list redhat com, xml gnome org, "Daniel P. Berrange" <berrange redhat com>
- Subject: Re: [xml] [Libvir] [PATCH] virDomainMigrate version 4 (for discussion only!)
- Date: Tue, 24 Jul 2007 08:50:37 -0400
On Tue, Jul 24, 2007 at 01:25:31PM +0100, Richard W.M. Jones wrote:
Daniel P. Berrange wrote:
I agree - the fact that the 'uri' to virConnectOpen doesn't technically
have to
always be a URI (eg, NULL, or Xen, or xen) is a major cause of pain
virt-manager
since we have to special case parsing of it, rather than just handing off
to a
generic URI parser module. We should mandate wellformed URIs for the
migrate
API, where wellformed is defined to be whatever libxml is able to parse :-)
I'd be a bit happier if libxml2 could parse a bare string like
"hostname" and "hostname:1234" more like my browser does. At the moment
this is what it does:
With my libxml2 maintainer hat on: no
What libxml2 provides is are URI handling
hostname ---> scheme=(null) server=(null) port=0 path=hostname
query_raw=(null)
makes no sense. From an URI perspective "hostname" is a valid URI
Reference, it could not be interpreted in any other way.
hostname:1234 ---> scheme=hostname server=(null) port=0 path=(null)
query_raw=(null)
no better, "hostname:1234" is also a valid URI Reference
I'm sure there's some smartypants standards reason for it, but it's
counterintuitive to me.
Your browser in general interprets "hostname" as an URI Reference
(hopefully), but the small window where an user can type something
relats more to guessing where you tried to direct him rather than
any sensible and predictable behaviour.
Again, sorry no, makes no sense from a libxml2 perspective.
Daniel
--
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard | virtualization library http://libvirt.org/
veillard redhat com | libxml GNOME XML XSLT toolkit http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]