← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/generate_ids.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/generate_ids.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
"""Common code for generating file or revision ids."""
18
18
 
32
32
    lazy_regex,
33
33
    )
34
34
 
35
 
# the regex removes any weird characters; we don't escape them
 
35
# the regex removes any weird characters; we don't escape them 
36
36
# but rather just pull them out
37
37
_file_id_chars_re = lazy_regex.lazy_compile(r'[^\w.]')
38
38
_rev_id_chars_re = lazy_regex.lazy_compile(r'[^-\w.+@]')
42
42
 
43
43
def _next_id_suffix():
44
44
    """Create a new file id suffix that is reasonably unique.
45
 
 
 
45
    
46
46
    On the first call we combine the current time with 64 bits of randomness to
47
47
    give a highly probably globally unique number. Then each call in the same
48
48
    process adds 1 to a serial number we append to that unique value.
49
49
    """
50
 
    # XXX TODO: change bzrlib.add.smart_add_tree to call workingtree.add() rather
 
50
    # XXX TODO: change bzrlib.add.smart_add_tree to call workingtree.add() rather 
51
51
    # than having to move the id randomness out of the inner loop like this.
52
52
    # XXX TODO: for the global randomness this uses we should add the thread-id
53
53
    # before the serial #.