Code review comment for lp:~brightbox/ubuntu/maverick/qemu-kvm/qemu-kvm.fix-697197

Dustin,

I've been following the discussion on the qemu development list and
they are going for a complete rewrite of the associated functions to
get rid of the overloaded behaviour. There's an ongoing discussion
with the RedHat boys about it.

Additionally I think this patch needs to go into Lucid as well.

On 11 February 2011 15:39, Dustin Kirkland <email address hidden> wrote:
> Review: Approve
> Looks good, thanks for doing this, Neil.
>
> I'm going to update it just slightly, as this debdiff will need to go through the security queue, since there's an associated CVE.  I'll prep that upload and the security team will sponsor it into maverick-security.
>
> I'll get it uploaded to natty now.
>
> The last thing I need you to do is to email your patch to the qemu-devel mailing list.  The maintainers do not accept patches solely attached to bugs in Launchpad.  Their processes require that you email the patch to the mailing list.  Sorry for the run-around.  Cheers!
> --
> https://code.launchpad.net/~brightbox/ubuntu/maverick/qemu-kvm/qemu-kvm.fix-697197/+merge/47820
> Your team Brightbox is subscribed to branch lp:~brightbox/ubuntu/maverick/qemu-kvm/qemu-kvm.fix-697197.
>

--
Neil Wilson