[xen-tools] Re: Feature proposal

[Steve Kemp]

On Thu Jul 05, 2007 at 10:21:49 +0200, Sascha Kettler wrote:

> I've included my current version of the patch against the latest CVS
> version (not sending it to the list as it is rather large for a patch).

  You've included ".#xen-create-image" too!  Probably best to run:

    make clean
    make diff > /tmp/update.diff

  Then include/link to that.

> These are the things I've changed:

  I've given the patch a quick look and I *think* it looks fine.

  I'll need to study it in more depth before commiting it, but I do
 want to say right now Thank You very much for this work.

> I've done quite some testing with --dir and --lvm, but you may want to
> try some more with --evms and physical devices, but I think they should
> just work :)

  :)

> Hope to see this going into the next release of xen-tools.

  I'll apply it tonight to CVS and test it for a while.  Any problems
 I'll mail you/the list and let you know.

  Unfortunately  I don't have a working EVMS system to test against,
 but I'm sure we'll find out if it breaks fairly quickly.

> The xen-delete-image will not really work until we can parse the Xen
> configuration file to get all the used devices and container files.

  Noted.

Steve
--
#  Kink-Friendly Dating
http://ctrl-alt-date.com/