Re: [libvirt] [PATCH]: Fix sexpr generation again

[Daniel Veillard <veillard redhat com>]

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/
veillard redhat com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/