Re: [libvirt] [PATCH] Fix virsh {net,pool}-edit
On Thu, Feb 12, 2009 at 04:58:50PM -0500, Cole Robinson wrote:
The dumpxml commands for networks and pools don't support any
flag
arguments, and in fact explictly fail if flags != 0. This is not the
case for vm dumpxml though, and flags were added to the base 'edit'
implementation in virsh recently.
The net and pool derivatives were not addressing this difference, so any
net-edit or pool-edit attempt currently gives an error like:
Network error : invalid argument in virNetworkGetXMLDesc
The attached patch is one way to fix this. Thanks to Charles Duffy for
the report.
Urgh, I rather dislike this code generation of the other edit
calls. It'd be much clearer if we justincluded the source for
them directly and had shared helper methods, rather than munging
source with perl.
Daniel
--
|: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :|
|: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|