← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/inter.py

  • Committer: Vincent Ladeuil
  • Date: 2008-01-29 15:16:31 UTC
  • mto: (3206.1.1 trunk)
  • mto: This revision was merged to the branch mainline in revision 3207.
  • Revision ID: v.ladeuil+lp@free.fr-20080129151631-vqjd13tb405mobx6
Fix two more leaking tmp dirs, by reworking TransformPreview lock handling.

* bzrlib/tests/test_transform.py:
(TestTransformMerge): Revert previous patch and cleanly call
preview.finalize now that we can.

* bzrlib/tests/test_merge.py:
(TestMerge.test_make_preview_transform): Catch TransformPreview
leak.

* bzrlib/builtins.py:
(cmd_merge._do_preview): Finalize the TransformPreview or the
limbodir will stay in /tmp.

* bzrlib/transform.py:
(TreeTransformBase.__init__): Create the _deletiondir since it's
reffered to by finalize.
(TreeTransformBase.finalize): Delete the dir only if _deletiondir
is set.
(TreeTransform.__init__): Use a temp var for deletiondir and set
the attribute after the base class __init__ has been called.
(TransformPreview.__init__): Read locks the tree since finalize
wants to unlock it (as suggested by Aaron).

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/inter.py
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
 
18
18
"""Inter-object utility class."""
22
22
    """This class represents operations taking place between two objects.
23
23
 
24
24
    Its instances have methods like join or copy_content or fetch, and contain
25
 
    references to the source and target objects these operations can be
 
25
    references to the source and target objects these operations can be 
26
26
    carried out between.
27
27
 
28
28
    Often we will provide convenience methods on the objects which carry out
29
29
    operations with another of similar type - they will always forward to
30
 
    a subclass of InterObject - i.e.
 
30
    a subclass of InterObject - i.e. 
31
31
    InterVersionedFile.get(other).method_name(parameters).
32
32
 
33
 
    If the source and target objects implement the locking protocol -
 
33
    If the source and target objects implement the locking protocol - 
34
34
    lock_read, lock_write, unlock, then the InterObject's lock_read,
35
35
    lock_write and unlock methods may be used (optionally in conjunction with
36
36
    the needs_read_lock and needs_write_lock decorators.)
45
45
 
46
46
    def __init__(self, source, target):
47
47
        """Construct a default InterObject instance. Please use 'get'.
48
 
 
49
 
        Only subclasses of InterObject should call
 
48
        
 
49
        Only subclasses of InterObject should call 
50
50
        InterObject.__init__ - clients should call InterFOO.get where FOO
51
51
        is the base type of the objects they are interacting between. I.e.
52
52
        InterVersionedFile or InterRepository.