On Mon, Aug 13, 2012 at 07:50:12PM +0400, Dmitry Guryanov wrote: > Let's change URI to parallels:///system. Parallels Server supports > creating VMs from non-privileged accounts, but it's not main usage > scenario and it may be forbidden in the future. > > Also containers, which will be supported by the driver, can be managed > only by root, so /system path is more suitable for this driver. [...] > diff --git a/src/parallels/parallels_driver.c b/src/parallels/parallels_driver.c > index 05db54f..53b5a50 100644 > --- a/src/parallels/parallels_driver.c > +++ b/src/parallels/parallels_driver.c > @@ -670,11 +670,11 @@ parallelsOpen(virConnectPtr conn, > (conn->uri->path[0] == '/' && conn->uri->path[1] == '\0')) { > virReportError(VIR_ERR_INVALID_ARG, "%s", > _("parallelsOpen: supply a path or use " > - "parallels:///session")); > + "parallels:///system")); > return VIR_DRV_OPEN_ERROR; > } > > - if (STREQ(conn->uri->path, "/session")) > + if (STREQ(conn->uri->path, "/system")) > ret = parallelsOpenDefault(conn); > else > return VIR_DRV_OPEN_DECLINED; Hum, Okay, the rc0 is not a release so we can still break things so better fix this early than late ! ACK, 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