[libvirt] [PATCH] introducing <source> <name> (for logical storage pools)
David Lively
dlively at virtualiron.com
Tue Sep 2 15:34:46 UTC 2008
Thanks Daniel. I just merged in your changes. You seem to be missing a
small incremental change (checking the strdup return value for NULL),
attached.
Dave
On Tue, 2008-09-02 at 16:17 +0200, Daniel Veillard wrote:
> On Fri, Aug 29, 2008 at 03:49:27PM -0400, David Lively wrote:
> > Hi Jim -
> > I've attached a (very) small incremental patch (i.e., to be applied
> > after the one you've already merged) that addresses a couple things I
> > noticed missing:
> > (a) documents the new <source> <name> element in formatstorage.html.in
> > (b) adds --source-name to the (optional) args for virsh pool-define-as
> >
> > I've also attached a new version of the full patch containing this
> > change, in case that's easier.
>
> Okidoc, I finally added this in CVS, i just had to do a bit of porting
> since the XPath lookup function have an extra argument, but nothing
> hard. I also changed some of the error message to provide more context
> because as Jim pointed out they were a bit too generic.
>
> thanks a lot !
>
> Daniel
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: storage-source-name.patch
Type: text/x-patch
Size: 494 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20080902/72b809ba/attachment-0001.bin>
More information about the libvir-list
mailing list