~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/commit.py

  • Committer: Robert Collins
  • Date: 2006-06-26 16:23:10 UTC
  • mfrom: (1780.2.1 misc-fixen)
  • mto: This revision was merged to the branch mainline in revision 1815.
  • Revision ID: robertc@robertcollins.net-20060626162310-98f5b55b8cc19d46
(robertc) Misc minor typos and the like.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005 Canonical Ltd
2
 
#
 
1
# Copyright (C) 2005, 2006 Canonical Ltd
 
2
 
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
5
5
# the Free Software Foundation; either version 2 of the License, or
56
56
# merges from, then it should still be reported as newly added
57
57
# relative to the basis revision.
58
58
 
 
59
# TODO: Do checks that the tree can be committed *before* running the 
 
60
# editor; this should include checks for a pointless commit and for 
 
61
# unknown or missing files.
 
62
 
 
63
# TODO: If commit fails, leave the message in a file somewhere.
 
64
 
59
65
 
60
66
import os
61
67
import re
62
68
import sys
63
69
import time
64
 
import pdb
 
70
import warnings
65
71
 
66
 
from binascii import hexlify
67
72
from cStringIO import StringIO
68
73
 
69
 
from bzrlib.osutils import (local_time_offset, username,
70
 
                            rand_bytes, compact_date, user_email,
71
 
                            kind_marker, is_inside_any, quotefn,
72
 
                            sha_string, sha_strings, sha_file, isdir, isfile,
73
 
                            split_lines)
74
 
from bzrlib.branch import gen_file_id
 
74
from bzrlib.atomicfile import AtomicFile
 
75
import bzrlib.config
 
76
import bzrlib.errors as errors
75
77
from bzrlib.errors import (BzrError, PointlessCommit,
76
 
                           HistoryMissing,
77
 
                           ConflictsInTree
 
78
                           ConflictsInTree,
 
79
                           StrictCommitFailed
78
80
                           )
79
 
from bzrlib.revision import Revision
 
81
from bzrlib.osutils import (kind_marker, isdir,isfile, is_inside_any, 
 
82
                            is_inside_or_parent_of_any,
 
83
                            quotefn, sha_file, split_lines)
 
84
from bzrlib.testament import Testament
80
85
from bzrlib.trace import mutter, note, warning
81
86
from bzrlib.xml5 import serializer_v5
82
 
from bzrlib.inventory import Inventory, ROOT_ID
83
 
from bzrlib.weave import Weave
84
 
from bzrlib.weavefile import read_weave, write_weave_v5
85
 
from bzrlib.atomicfile import AtomicFile
86
 
 
87
 
 
 
87
from bzrlib.inventory import Inventory, ROOT_ID, InventoryEntry
 
88
from bzrlib.symbol_versioning import (deprecated_passed,
 
89
        deprecated_function,
 
90
        zero_seven,
 
91
        DEPRECATED_PARAMETER)
 
92
from bzrlib.workingtree import WorkingTree
 
93
 
 
94
 
 
95
@deprecated_function(zero_seven)
88
96
def commit(*args, **kwargs):
89
97
    """Commit a new revision to a branch.
90
98
 
92
100
 
93
101
    New code should use the Commit class instead.
94
102
    """
95
 
    ## XXX: Remove this in favor of Branch.commit?
 
103
    ## XXX: Remove this in favor of WorkingTree.commit?
96
104
    Commit().commit(*args, **kwargs)
97
105
 
98
106
 
114
122
    def missing(self, path):
115
123
        pass
116
124
 
 
125
    def renamed(self, change, old_path, new_path):
 
126
        pass
 
127
 
 
128
 
117
129
class ReportCommitToLog(NullCommitReporter):
118
130
 
 
131
    # this may be more useful if 'note' was replaced by an overridable
 
132
    # method on self, which would allow more trivial subclassing.
 
133
    # alternative, a callable could be passed in, allowing really trivial
 
134
    # reuse for some uis. RBC 20060511
 
135
 
119
136
    def snapshot_change(self, change, path):
 
137
        if change == 'unchanged':
 
138
            return
120
139
        note("%s %s", change, path)
121
140
 
122
141
    def completed(self, revno, rev_id):
123
 
        note('committed r%d {%s}', revno, rev_id)
 
142
        note('Committed revision %d.', revno)
124
143
    
125
144
    def deleted(self, file_id):
126
145
        note('deleted %s', file_id)
131
150
    def missing(self, path):
132
151
        note('missing %s', path)
133
152
 
 
153
    def renamed(self, change, old_path, new_path):
 
154
        note('%s %s => %s', change, old_path, new_path)
 
155
 
 
156
 
134
157
class Commit(object):
135
158
    """Task of committing a new revision.
136
159
 
144
167
            working inventory.
145
168
    """
146
169
    def __init__(self,
147
 
                 reporter=None):
 
170
                 reporter=None,
 
171
                 config=None):
148
172
        if reporter is not None:
149
173
            self.reporter = reporter
150
174
        else:
151
175
            self.reporter = NullCommitReporter()
152
 
 
 
176
        if config is not None:
 
177
            self.config = config
 
178
        else:
 
179
            self.config = None
153
180
        
154
181
    def commit(self,
155
 
               branch, message,
 
182
               branch=DEPRECATED_PARAMETER, message=None,
156
183
               timestamp=None,
157
184
               timezone=None,
158
185
               committer=None,
159
186
               specific_files=None,
160
187
               rev_id=None,
161
188
               allow_pointless=True,
162
 
               verbose=False):
 
189
               strict=False,
 
190
               verbose=False,
 
191
               revprops=None,
 
192
               working_tree=None,
 
193
               local=False,
 
194
               reporter=None,
 
195
               config=None):
163
196
        """Commit working copy as a new revision.
164
197
 
 
198
        branch -- the deprecated branch to commit to. New callers should pass in 
 
199
                  working_tree instead
 
200
 
 
201
        message -- the commit message, a mandatory parameter
 
202
 
165
203
        timestamp -- if not None, seconds-since-epoch for a
166
204
             postdated/predated commit.
167
205
 
175
213
 
176
214
        allow_pointless -- If true (default), commit even if nothing
177
215
            has changed and no merges are recorded.
 
216
 
 
217
        strict -- If true, don't allow a commit if the working tree
 
218
            contains unknown files.
 
219
 
 
220
        revprops -- Properties for new revision
 
221
        :param local: Perform a local only commit.
178
222
        """
179
223
        mutter('preparing to commit')
180
224
 
181
 
        self.branch = branch
182
 
        self.weave_store = branch.weave_store
183
 
        self.rev_id = rev_id
 
225
        if deprecated_passed(branch):
 
226
            warnings.warn("Commit.commit (branch, ...): The branch parameter is "
 
227
                 "deprecated as of bzr 0.8. Please use working_tree= instead.",
 
228
                 DeprecationWarning, stacklevel=2)
 
229
            self.branch = branch
 
230
            self.work_tree = self.branch.bzrdir.open_workingtree()
 
231
        elif working_tree is None:
 
232
            raise BzrError("One of branch and working_tree must be passed into commit().")
 
233
        else:
 
234
            self.work_tree = working_tree
 
235
            self.branch = self.work_tree.branch
 
236
        if message is None:
 
237
            raise BzrError("The message keyword parameter is required for commit().")
 
238
 
 
239
        self.bound_branch = None
 
240
        self.local = local
 
241
        self.master_branch = None
 
242
        self.master_locked = False
 
243
        self.rev_id = None
184
244
        self.specific_files = specific_files
185
245
        self.allow_pointless = allow_pointless
186
246
 
187
 
        if timestamp is None:
188
 
            self.timestamp = time.time()
189
 
        else:
190
 
            self.timestamp = long(timestamp)
191
 
            
192
 
        if rev_id is None:
193
 
            self.rev_id = _gen_revision_id(self.branch, self.timestamp)
194
 
        else:
195
 
            self.rev_id = rev_id
196
 
 
197
 
        if committer is None:
198
 
            self.committer = username(self.branch)
199
 
        else:
200
 
            assert isinstance(committer, basestring), type(committer)
201
 
            self.committer = committer
202
 
 
203
 
        if timezone is None:
204
 
            self.timezone = local_time_offset()
205
 
        else:
206
 
            self.timezone = int(timezone)
207
 
 
208
 
        assert isinstance(message, basestring), type(message)
209
 
        self.message = message
210
 
        self._escape_commit_message()
211
 
 
212
 
        self.branch.lock_write()
 
247
        if reporter is None and self.reporter is None:
 
248
            self.reporter = NullCommitReporter()
 
249
        elif reporter is not None:
 
250
            self.reporter = reporter
 
251
 
 
252
        self.work_tree.lock_write()
 
253
        self.pb = bzrlib.ui.ui_factory.nested_progress_bar()
213
254
        try:
214
 
            self.work_tree = self.branch.working_tree()
 
255
            # Cannot commit with conflicts present.
 
256
            if len(self.work_tree.conflicts())>0:
 
257
                raise ConflictsInTree
 
258
 
 
259
            # setup the bound branch variables as needed.
 
260
            self._check_bound_branch()
 
261
 
 
262
            # check for out of date working trees
 
263
            # if we are bound, then self.branch is the master branch and this
 
264
            # test is thus all we need.
 
265
            if self.work_tree.last_revision() != self.master_branch.last_revision():
 
266
                raise errors.OutOfDateTree(self.work_tree)
 
267
    
 
268
            if strict:
 
269
                # raise an exception as soon as we find a single unknown.
 
270
                for unknown in self.work_tree.unknowns():
 
271
                    raise StrictCommitFailed()
 
272
                   
 
273
            if self.config is None:
 
274
                self.config = self.branch.get_config()
 
275
      
 
276
            if isinstance(message, str):
 
277
                message = message.decode(bzrlib.user_encoding)
 
278
            assert isinstance(message, unicode), type(message)
 
279
            self.message = message
 
280
            self._escape_commit_message()
 
281
 
215
282
            self.work_inv = self.work_tree.inventory
216
 
            self.basis_tree = self.branch.basis_tree()
 
283
            self.basis_tree = self.work_tree.basis_tree()
217
284
            self.basis_inv = self.basis_tree.inventory
 
285
            # one to finish, one for rev and inventory, and one for each
 
286
            # inventory entry, and the same for the new inventory.
 
287
            # note that this estimate is too long when we do a partial tree
 
288
            # commit which excludes some new files from being considered.
 
289
            # The estimate is corrected when we populate the new inv.
 
290
            self.pb_total = len(self.work_inv) + 5
 
291
            self.pb_count = 0
218
292
 
219
293
            self._gather_parents()
220
294
            if len(self.parents) > 1 and self.specific_files:
221
 
                raise NotImplementedError('selected-file commit of merges is not supported yet')
 
295
                raise NotImplementedError('selected-file commit of merges is not supported yet: files %r',
 
296
                        self.specific_files)
222
297
            self._check_parents_present()
 
298
            self.builder = self.branch.get_commit_builder(self.parents, 
 
299
                self.config, timestamp, timezone, committer, revprops, rev_id)
223
300
            
224
301
            self._remove_deleted()
225
302
            self._populate_new_inv()
226
 
            self._store_snapshot()
227
303
            self._report_deletes()
228
304
 
229
305
            if not (self.allow_pointless
230
306
                    or len(self.parents) > 1
231
 
                    or self.new_inv != self.basis_inv):
 
307
                    or self.builder.new_inventory != self.basis_inv):
232
308
                raise PointlessCommit()
233
309
 
234
 
            if len(list(self.work_tree.iter_conflicts()))>0:
235
 
                raise ConflictsInTree
 
310
            self._emit_progress_update()
 
311
            # TODO: Now the new inventory is known, check for conflicts and prompt the 
 
312
            # user for a commit message.
 
313
            self.builder.finish_inventory()
 
314
            self._emit_progress_update()
 
315
            self.rev_id = self.builder.commit(self.message)
 
316
            self._emit_progress_update()
 
317
            # revision data is in the local branch now.
 
318
            
 
319
            # upload revision data to the master.
 
320
            # this will propagate merged revisions too if needed.
 
321
            if self.bound_branch:
 
322
                self.master_branch.repository.fetch(self.branch.repository,
 
323
                                                    revision_id=self.rev_id)
 
324
                # now the master has the revision data
 
325
                # 'commit' to the master first so a timeout here causes the local
 
326
                # branch to be out of date
 
327
                self.master_branch.append_revision(self.rev_id)
236
328
 
237
 
            self._record_inventory()
238
 
            self._make_revision()
239
 
            self.reporter.completed(self.branch.revno()+1, self.rev_id)
 
329
            # and now do the commit locally.
240
330
            self.branch.append_revision(self.rev_id)
241
 
            self.branch.set_pending_merges([])
 
331
 
 
332
            self.work_tree.set_pending_merges([])
 
333
            self.work_tree.set_last_revision(self.rev_id)
 
334
            # now the work tree is up to date with the branch
 
335
            
 
336
            self.reporter.completed(self.branch.revno(), self.rev_id)
 
337
            if self.config.post_commit() is not None:
 
338
                hooks = self.config.post_commit().split(' ')
 
339
                # this would be nicer with twisted.python.reflect.namedAny
 
340
                for hook in hooks:
 
341
                    result = eval(hook + '(branch, rev_id)',
 
342
                                  {'branch':self.branch,
 
343
                                   'bzrlib':bzrlib,
 
344
                                   'rev_id':self.rev_id})
 
345
            self._emit_progress_update()
242
346
        finally:
243
 
            self.branch.unlock()
244
 
 
245
 
    def _record_inventory(self):
246
 
        """Store the inventory for the new revision."""
247
 
        inv_text = serializer_v5.write_inventory_to_string(self.new_inv)
248
 
        self.inv_sha1 = sha_string(inv_text)
249
 
        s = self.branch.control_weaves
250
 
        s.add_text('inventory', self.rev_id,
251
 
                   split_lines(inv_text), self.present_parents)
 
347
            self._cleanup()
 
348
        return self.rev_id
 
349
 
 
350
    def _check_bound_branch(self):
 
351
        """Check to see if the local branch is bound.
 
352
 
 
353
        If it is bound, then most of the commit will actually be
 
354
        done using the remote branch as the target branch.
 
355
        Only at the end will the local branch be updated.
 
356
        """
 
357
        if self.local and not self.branch.get_bound_location():
 
358
            raise errors.LocalRequiresBoundBranch()
 
359
 
 
360
        if not self.local:
 
361
            self.master_branch = self.branch.get_master_branch()
 
362
 
 
363
        if not self.master_branch:
 
364
            # make this branch the reference branch for out of date checks.
 
365
            self.master_branch = self.branch
 
366
            return
 
367
 
 
368
        # If the master branch is bound, we must fail
 
369
        master_bound_location = self.master_branch.get_bound_location()
 
370
        if master_bound_location:
 
371
            raise errors.CommitToDoubleBoundBranch(self.branch,
 
372
                    self.master_branch, master_bound_location)
 
373
 
 
374
        # TODO: jam 20051230 We could automatically push local
 
375
        #       commits to the remote branch if they would fit.
 
376
        #       But for now, just require remote to be identical
 
377
        #       to local.
 
378
        
 
379
        # Make sure the local branch is identical to the master
 
380
        master_rh = self.master_branch.revision_history()
 
381
        local_rh = self.branch.revision_history()
 
382
        if local_rh != master_rh:
 
383
            raise errors.BoundBranchOutOfDate(self.branch,
 
384
                    self.master_branch)
 
385
 
 
386
        # Now things are ready to change the master branch
 
387
        # so grab the lock
 
388
        self.bound_branch = self.branch
 
389
        self.master_branch.lock_write()
 
390
        self.master_locked = True
 
391
 
 
392
    def _cleanup(self):
 
393
        """Cleanup any open locks, progress bars etc."""
 
394
        cleanups = [self._cleanup_bound_branch,
 
395
                    self.work_tree.unlock,
 
396
                    self.pb.finished]
 
397
        found_exception = None
 
398
        for cleanup in cleanups:
 
399
            try:
 
400
                cleanup()
 
401
            # we want every cleanup to run no matter what.
 
402
            # so we have a catchall here, but we will raise the
 
403
            # last encountered exception up the stack: and
 
404
            # typically this will be useful enough.
 
405
            except Exception, e:
 
406
                found_exception = e
 
407
        if found_exception is not None: 
 
408
            # don't do a plan raise, because the last exception may have been
 
409
            # trashed, e is our sure-to-work exception even though it loses the
 
410
            # full traceback. XXX: RBC 20060421 perhaps we could check the
 
411
            # exc_info and if its the same one do a plain raise otherwise 
 
412
            # 'raise e' as we do now.
 
413
            raise e
 
414
 
 
415
    def _cleanup_bound_branch(self):
 
416
        """Executed at the end of a try/finally to cleanup a bound branch.
 
417
 
 
418
        If the branch wasn't bound, this is a no-op.
 
419
        If it was, it resents self.branch to the local branch, instead
 
420
        of being the master.
 
421
        """
 
422
        if not self.bound_branch:
 
423
            return
 
424
        if self.master_locked:
 
425
            self.master_branch.unlock()
252
426
 
253
427
    def _escape_commit_message(self):
254
428
        """Replace xml-incompatible control characters."""
 
429
        # FIXME: RBC 20060419 this should be done by the revision
 
430
        # serialiser not by commit. Then we can also add an unescaper
 
431
        # in the deserializer and start roundtripping revision messages
 
432
        # precisely. See repository_implementations/test_repository.py
 
433
        
255
434
        # Python strings can include characters that can't be
256
435
        # represented in well-formed XML; escape characters that
257
436
        # aren't listed in the XML specification
258
437
        # (http://www.w3.org/TR/REC-xml/#NT-Char).
259
 
        if isinstance(self.message, unicode):
260
 
            char_pattern = u'[^\x09\x0A\x0D\u0020-\uD7FF\uE000-\uFFFD]'
261
 
        else:
262
 
            # Use a regular 'str' as pattern to avoid having re.subn
263
 
            # return 'unicode' results.
264
 
            char_pattern = '[^x09\x0A\x0D\x20-\xFF]'
265
438
        self.message, escape_count = re.subn(
266
 
            char_pattern,
 
439
            u'[^\x09\x0A\x0D\u0020-\uD7FF\uE000-\uFFFD]+',
267
440
            lambda match: match.group(0).encode('unicode_escape'),
268
441
            self.message)
269
442
        if escape_count:
271
444
 
272
445
    def _gather_parents(self):
273
446
        """Record the parents of a merge for merge detection."""
274
 
        pending_merges = self.branch.pending_merges()
275
 
        self.parents = []
 
447
        # TODO: Make sure that this list doesn't contain duplicate 
 
448
        # entries and the order is preserved when doing this.
 
449
        self.parents = self.work_tree.get_parent_ids()
276
450
        self.parent_invs = []
277
 
        self.present_parents = []
278
 
        precursor_id = self.branch.last_revision()
279
 
        if precursor_id:
280
 
            self.parents.append(precursor_id)
281
 
        self.parents += pending_merges
282
451
        for revision in self.parents:
283
 
            if self.branch.has_revision(revision):
284
 
                self.parent_invs.append(self.branch.get_inventory(revision))
285
 
                self.present_parents.append(revision)
 
452
            if self.branch.repository.has_revision(revision):
 
453
                inventory = self.branch.repository.get_inventory(revision)
 
454
                self.parent_invs.append(inventory)
286
455
 
287
456
    def _check_parents_present(self):
288
457
        for parent_id in self.parents:
289
458
            mutter('commit parent revision {%s}', parent_id)
290
 
            if not self.branch.has_revision(parent_id):
 
459
            if not self.branch.repository.has_revision(parent_id):
291
460
                if parent_id == self.branch.last_revision():
292
461
                    warning("parent is missing %r", parent_id)
293
 
                    raise HistoryMissing(self.branch, 'revision', parent_id)
294
 
                else:
295
 
                    mutter("commit will ghost revision %r", parent_id)
 
462
                    raise BzrCheckError("branch %s is missing revision {%s}"
 
463
                            % (self.branch, parent_id))
296
464
            
297
 
    def _make_revision(self):
298
 
        """Record a new revision object for this commit."""
299
 
        self.rev = Revision(timestamp=self.timestamp,
300
 
                            timezone=self.timezone,
301
 
                            committer=self.committer,
302
 
                            message=self.message,
303
 
                            inventory_sha1=self.inv_sha1,
304
 
                            revision_id=self.rev_id)
305
 
        self.rev.parent_ids = self.parents
306
 
        rev_tmp = StringIO()
307
 
        serializer_v5.write_revision(self.rev, rev_tmp)
308
 
        rev_tmp.seek(0)
309
 
        self.branch.revision_store.add(rev_tmp, self.rev_id)
310
 
        mutter('new revision_id is {%s}', self.rev_id)
311
 
 
312
465
    def _remove_deleted(self):
313
466
        """Remove deleted files from the working inventories.
314
467
 
332
485
            deleted_ids.sort(reverse=True)
333
486
            for path, file_id in deleted_ids:
334
487
                del self.work_inv[file_id]
335
 
            self.branch._write_inventory(self.work_inv)
336
 
 
337
 
    def _store_snapshot(self):
338
 
        """Pass over inventory and record a snapshot.
339
 
 
340
 
        Entries get a new revision when they are modified in 
341
 
        any way, which includes a merge with a new set of
342
 
        parents that have the same entry. 
343
 
        """
344
 
        # XXX: Need to think more here about when the user has
345
 
        # made a specific decision on a particular value -- c.f.
346
 
        # mark-merge.  
347
 
        for path, ie in self.new_inv.iter_entries():
348
 
            previous_entries = ie.find_previous_heads(
349
 
                self.parent_invs, 
350
 
                self.weave_store.get_weave_or_empty(ie.file_id))
351
 
            if ie.revision is None:
352
 
                change = ie.snapshot(self.rev_id, path, previous_entries,
353
 
                                     self.work_tree, self.weave_store)
354
 
            else:
355
 
                change = "unchanged"
356
 
            self.reporter.snapshot_change(change, path)
 
488
            self.work_tree._write_inventory(self.work_inv)
357
489
 
358
490
    def _populate_new_inv(self):
359
491
        """Build revision inventory.
366
498
        revision set to their prior value.
367
499
        """
368
500
        mutter("Selecting files for commit with filter %s", self.specific_files)
369
 
        self.new_inv = Inventory()
 
501
        # iter_entries does not visit the ROOT_ID node so we need to call
 
502
        # self._emit_progress_update once by hand.
 
503
        self._emit_progress_update()
370
504
        for path, new_ie in self.work_inv.iter_entries():
 
505
            self._emit_progress_update()
371
506
            file_id = new_ie.file_id
372
 
            mutter('check %s {%s}', path, new_ie.file_id)
373
 
            if self.specific_files:
374
 
                if not is_inside_any(self.specific_files, path):
375
 
                    mutter('%s not selected for commit', path)
376
 
                    self._carry_entry(file_id)
 
507
            mutter('check %s {%s}', path, file_id)
 
508
            if (not self.specific_files or 
 
509
                is_inside_or_parent_of_any(self.specific_files, path)):
 
510
                    mutter('%s selected for commit', path)
 
511
                    ie = new_ie.copy()
 
512
                    ie.revision = None
 
513
            else:
 
514
                mutter('%s not selected for commit', path)
 
515
                if self.basis_inv.has_id(file_id):
 
516
                    ie = self.basis_inv[file_id].copy()
 
517
                else:
 
518
                    # this entry is new and not being committed
377
519
                    continue
378
 
                else:
379
 
                    # this is selected, ensure its parents are too.
380
 
                    parent_id = new_ie.parent_id
381
 
                    while parent_id != ROOT_ID:
382
 
                        if not self.new_inv.has_id(parent_id):
383
 
                            ie = self._select_entry(self.work_inv[parent_id])
384
 
                            mutter('%s selected for commit because of %s',
385
 
                                   self.new_inv.id2path(parent_id), path)
386
 
 
387
 
                        ie = self.new_inv[parent_id]
388
 
                        if ie.revision is not None:
389
 
                            ie.revision = None
390
 
                            mutter('%s selected for commit because of %s',
391
 
                                   self.new_inv.id2path(parent_id), path)
392
 
                        parent_id = ie.parent_id
393
 
            mutter('%s selected for commit', path)
394
 
            self._select_entry(new_ie)
395
 
 
396
 
    def _select_entry(self, new_ie):
397
 
        """Make new_ie be considered for committing."""
398
 
        ie = new_ie.copy()
399
 
        ie.revision = None
400
 
        self.new_inv.add(ie)
401
 
        return ie
402
 
 
403
 
    def _carry_entry(self, file_id):
404
 
        """Carry the file unchanged from the basis revision."""
405
 
        if self.basis_inv.has_id(file_id):
406
 
            self.new_inv.add(self.basis_inv[file_id].copy())
 
520
 
 
521
            self.builder.record_entry_contents(ie, self.parent_invs, 
 
522
                path, self.work_tree)
 
523
            # describe the nature of the change that has occurred relative to
 
524
            # the basis inventory.
 
525
            if (self.basis_inv.has_id(ie.file_id)):
 
526
                basis_ie = self.basis_inv[ie.file_id]
 
527
            else:
 
528
                basis_ie = None
 
529
            change = ie.describe_change(basis_ie, ie)
 
530
            if change in (InventoryEntry.RENAMED, 
 
531
                InventoryEntry.MODIFIED_AND_RENAMED):
 
532
                old_path = self.basis_inv.id2path(ie.file_id)
 
533
                self.reporter.renamed(change, old_path, path)
 
534
            else:
 
535
                self.reporter.snapshot_change(change, path)
 
536
 
 
537
    def _emit_progress_update(self):
 
538
        """Emit an update to the progress bar."""
 
539
        self.pb.update("Committing", self.pb_count, self.pb_total)
 
540
        self.pb_count += 1
407
541
 
408
542
    def _report_deletes(self):
409
 
        for file_id in self.basis_inv:
410
 
            if file_id not in self.new_inv:
411
 
                self.reporter.deleted(self.basis_inv.id2path(file_id))
412
 
 
413
 
def _gen_revision_id(branch, when):
414
 
    """Return new revision-id."""
415
 
    s = '%s-%s-' % (user_email(branch), compact_date(when))
416
 
    s += hexlify(rand_bytes(8))
417
 
    return s
 
543
        for path, ie in self.basis_inv.iter_entries():
 
544
            if ie.file_id not in self.builder.new_inventory:
 
545
                self.reporter.deleted(path)
 
546
 
 
547