On 02/01/2013 12:55 PM, Serge Hallyn wrote: > Quoting Eric Blake (eblake@xxxxxxxxxx): >> On 01/30/2013 08:05 PM, Serge Hallyn wrote: >>> Without these two string changes in generator.py, the >>> virGetLastError wrapper does not get created in >>> /usr/share/pyshared/libvirt.py. Noticed when running >>> tests with virt-install. >>> >>> Signed-off-by: Serge Hallyn <serge.hallyn@xxxxxxxxxx> >>> --- >>> python/generator.py | 4 ++-- >>> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> ACK. >> >> Bummer - that means the python bindings are broken in the 1.0.2 release. > > D'oh, with my patch, Chuck was also getting the following error: > > chuck@homer:~/work/server/libvirt/libvirt-1.0.2/python/tests$ python error.py > Traceback (most recent call last): > File "error.py", line 35, in <module> > elif errno[0] == libvirt.VIR_ERR_NO_CONNECT or \ > AttributeError: 'module' object has no attribute 'VIR_ERR_NO_CONNECT' > > presumably because include/libvirt/virterror.h still exists, and my > patch was over-zealous. Does that mean we need a followup patch that looks for both virerror AND virterror? -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list