From: Avery Pennarun <apenwarr@gmail.com>
Date: Sun, 14 Feb 2010 08:35:45 +0000 (-0500)
Subject: Another suspicious fix for CatPipe parallelism.
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=fe5ecf61a144e620ac436ab70884cc0d469e7317;p=packages%2Fb%2Fbup.git

Another suspicious fix for CatPipe parallelism.

This really shouldn't be necessary: it's clear to me that the 'it' object
should be going out of scope right away, and thus getting cleaned up by the
garbage collector.

But on one of my Linux PCs (with python 2.4.4) it fails the unit tests
unless I add this patch.  Oh well, let's do it then.
---

diff --git a/git.py b/git.py
index dbd61c1..87ccf8f 100644
--- a/git.py
+++ b/git.py
@@ -643,6 +643,7 @@ class CatPipe:
         yield type
         for blob in it:
             yield blob
+        del it
 
     def _slow_get(self, id):
         assert(id.find('\n') < 0)