[Pkg-libvirt-maintainers] Bug#716672: Bug#716672: Patch attached and tested

[Guido Günther]

tag 716672 +pending
thanks

On Mon, Jul 15, 2013 at 10:30:01AM +0200, Roy Meulekamp wrote:
> Guido Günther schreef op 13.07.2013 10:30:
> >Hi Roy,
> >On Fri, Jul 12, 2013 at 03:31:36PM +0200, Roy Meulekamp wrote:
> >>Hi Guido,
> >>
> >>I have attached a patch for CloneManager.py and tested that on my
> >>installation. This will create valid clones of the original guest by
> >>keeping a qcow2 image as a qcow2 type in the XML of the guest.
> >
> >Thanks a lot! Did you cherry-pick this from upstream? If so, could you
> >just attach the URL so I can add it for reference?
> >Cheers,
> > -- Guido
> >
> Hi Guido,
> 
> I had meant to paste the URL's, but I see that I didn't. I have
> combined two upstream patches to fix this problem. They can be found
> here:
> https://git.fedorahosted.org/cgit/python-virtinst.git/commit/?id=f0195e95d57deda83daed5231582ca86bde519ae
> https://git.fedorahosted.org/cgit/python-virtinst.git/commit/?id=8447cee94b36da389a73d3d81ecbbbe543ac5d8a
> 
> I only used the CloneManager.py parts of the patches. Also there is
> a revert of part of the patch: https://git.fedorahosted.org/cgit/python-virtinst.git/commit/?id=9dda1ae02aa08c73c43dea1069eb2117a68fb3e5,
> and later this is redone. Therefore, I combined the patches into
> one. The patch I had attached, I have tested it and confirmed that
> it works.

Thanks a lot. I've pushed this to virtinst's wheezy branch:

    http://anonscm.debian.org/gitweb/?p=pkg-libvirt/virtinst.git;a=shortlog;h=refs/heads/debian/wheezy

I'll also prepare the uplaod to s-p-u.
Cheers,
 -- Guido

> 
> Cheers,
> 
> Roy
>