[libvirt] [libvirt-perl PATCH] Fix several APIs

Daniel P. Berrange berrange at redhat.com
Tue Sep 4 09:20:57 UTC 2012


On Mon, Sep 03, 2012 at 06:29:47PM +0800, Osier Yang wrote:
> On 2012年08月29日 14:44, Daniel P. Berrange wrote:
> >On Wed, Aug 29, 2012 at 01:29:37AM -0400, Alex Jia wrote:
> >>ACK.
> >>
> >>--
> >>Regards,
> >>Alex
> >>
> >>
> >>----- Original Message -----
> >>From: "Osier Yang"<jyang at redhat.com>
> >>To: libvir-list at redhat.com
> >>Sent: Tuesday, August 28, 2012 11:52:24 PM
> >>Subject: [libvirt] [libvirt-perl PATCH] Fix several APIs
> >>
> >>These APIs accept one more argument (flags), which was ignored in
> >>the XS implementations.
> >>---
> >>  Virt.xs |   46 ++++++++++++++++++++++++++--------------------
> >>  1 files changed, 26 insertions(+), 20 deletions(-)
> >
> >ACK, if you also update the Domain.pm docs to mention the new flags
> >parameters
> >
> 
> Thanks, Pushed with the doc and nits pointed out by gren updated
> (btw, most of the flags for APIs are not documented, and even some
> constants, such as constants for virTypedParameterFlags, could be
> later patch).

Those constants should not be required, since the conversiuon is done
internally. There is a test suite which validates that all libvirt
public constants are present in the Perl binding, minus some blacklisted
constants like virTypedParameterFlags ones


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|




More information about the libvir-list mailing list