[Merge] lp:~mvo/python-apt/mvo into lp:~ubuntu-core-dev/python-apt/ubuntu

Michael Vogt michael.vogt at ubuntu.com
Wed Oct 17 12:58:20 UTC 2012


On Mon, Oct 15, 2012 at 05:08:22PM -0000, Jason Conti wrote:
> Review: Approve
> 
> Looks good to me overall. My only concern is that test_cache_delete_leasks_fds() may fail in certain circumstances, and it happened to fail in my test rebuild. Python does eventually get around to deleting the records instance.
[..]

Nevermind, its also failing for me now, not sure why I did not see
that before. I deleted the test again.

Cheers,
 Michael

-- 
https://code.launchpad.net/~mvo/python-apt/mvo/+merge/129630
Your team Ubuntu Core Development Team is requested to review the proposed merge of lp:~mvo/python-apt/mvo into lp:~ubuntu-core-dev/python-apt/ubuntu.



More information about the Ubuntu-reviews mailing list