On Wed, Aug 06, 2008 at 12:20:50PM +0200, Chris Lalancette wrote:
> DanB pointed out that my sexpr xend_internal patch from yesterday broke one of
> the regression tests.  The problem is that the xenDaemonFormatSxpr{Disk,Net}
> functions are shared between domain creation time and attaching disk time.
> Unfortunately, though, Xend expects something different during these two 
> times.
>  During domain creation time, it wants the "(device" in front of the sexpr,
> while during attach time it does not.  To remedy this situation, I added a 
> flag
> to these two functions to differentiate between these two modes.  With this
> patch in place, all of the regression tests pass.

  Looks fine to me, +1

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
[EMAIL PROTECTED]  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/

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

Reply via email to