<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=iso-8859-1">
<META NAME="Generator" CONTENT="MS Exchange Server version 6.5.7653.38">
<TITLE>RE: [libvirt] Domain Events Python Bindings (Round 2)</TITLE>
</HEAD>
<BODY>
<!-- Converted from text/plain format -->
<BR>

<P><FONT SIZE=2>I just took a look over the commit here:<BR>
<A HREF="http://git.et.redhat.com/?p=libvirt.git;a=commit;h=471b765ae0fc3efd721d6dbd3c82539880933c0b">http://git.et.redhat.com/?p=libvirt.git;a=commit;h=471b765ae0fc3efd721d6dbd3c82539880933c0b</A><BR>
<BR>
It appears that you may have missed adding the new file python/virConnect.py<BR>
This file is critical to this patch, as it controls proper callback registration, and cleanup coordination with the C code<BR>
<BR>
Ben<BR>
<BR>
-----Original Message-----<BR>
From: Daniel Veillard [<A HREF="mailto:veillard@redhat.com">mailto:veillard@redhat.com</A>]<BR>
Sent: Fri 10/31/2008 7:22 AM<BR>
To: Ben Guthro<BR>
Cc: libvir-list@redhat.com<BR>
Subject: Re: [libvirt] Domain Events Python Bindings (Round 2)<BR>
<BR>
On Fri, Oct 31, 2008 at 07:04:09AM -0400, Ben Guthro wrote:<BR>
><BR>
> Hmmm... I also realized that I never added the example to the top level Makefile.am EXTRA_DIST.<BR>
> I imagine that would need to be resolved to address adding it to the RPM<BR>
><BR>
> Would you like me to take a look at this, or will this be something you'll resolve as part of the RPM multlib cleanup?<BR>
<BR>
  I will handle this :-) don't worry !<BR>
<BR>
Daniel<BR>
<BR>
--<BR>
Daniel Veillard      | libxml Gnome XML XSLT toolkit  <A HREF="http://xmlsoft.org/">http://xmlsoft.org/</A><BR>
daniel@veillard.com  | Rpmfind RPM search engine <A HREF="http://rpmfind.net/">http://rpmfind.net/</A><BR>
<A HREF="http://veillard.com/">http://veillard.com/</A> | virtualization library  <A HREF="http://libvirt.org/">http://libvirt.org/</A><BR>
<BR>
</FONT>
</P>

</BODY>
</HTML>