/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/commit.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2011-02-09 08:51:29 UTC
  • mfrom: (5638.2.2 537442-annotate-removed)
  • Revision ID: pqm@pqm.ubuntu.com-20110209085129-0qspgtdaj4gfu2qr
(vila) Annotate deleted files if they can be found in the specified
 revision. (Andrew King)

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2005-2011 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
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
 
16
 
 
17
 
 
18
# The newly committed revision is going to have a shape corresponding
 
19
# to that of the working tree.  Files that are not in the
 
20
# working tree and that were in the predecessor are reported as
 
21
# removed --- this can include files that were either removed from the
 
22
# inventory or deleted in the working tree.  If they were only
 
23
# deleted from disk, they are removed from the working inventory.
 
24
 
 
25
# We then consider the remaining entries, which will be in the new
 
26
# version.  Directory entries are simply copied across.  File entries
 
27
# must be checked to see if a new version of the file should be
 
28
# recorded.  For each parent revision tree, we check to see what
 
29
# version of the file was present.  If the file was present in at
 
30
# least one tree, and if it was the same version in all the trees,
 
31
# then we can just refer to that version.  Otherwise, a new version
 
32
# representing the merger of the file versions must be added.
 
33
 
 
34
# TODO: Update hashcache before and after - or does the WorkingTree
 
35
# look after that?
 
36
 
 
37
# TODO: Rather than mashing together the ancestry and storing it back,
 
38
# perhaps the weave should have single method which does it all in one
 
39
# go, avoiding a lot of redundant work.
 
40
 
 
41
# TODO: Perhaps give a warning if one of the revisions marked as
 
42
# merged is already in the ancestry, and then don't record it as a
 
43
# distinct parent.
 
44
 
 
45
# TODO: If the file is newly merged but unchanged from the version it
 
46
# merges from, then it should still be reported as newly added
 
47
# relative to the basis revision.
 
48
 
 
49
# TODO: Change the parameter 'rev_id' to 'revision_id' to be consistent with
 
50
# the rest of the code; add a deprecation of the old name.
 
51
 
 
52
from bzrlib import (
 
53
    debug,
 
54
    errors,
 
55
    revision,
 
56
    trace,
 
57
    tree,
 
58
    ui,
 
59
    )
 
60
from bzrlib.branch import Branch
 
61
from bzrlib.cleanup import OperationWithCleanups
 
62
import bzrlib.config
 
63
from bzrlib.errors import (BzrError, PointlessCommit,
 
64
                           ConflictsInTree,
 
65
                           StrictCommitFailed
 
66
                           )
 
67
from bzrlib.osutils import (get_user_encoding,
 
68
                            is_inside_any,
 
69
                            minimum_path_selection,
 
70
                            splitpath,
 
71
                            )
 
72
from bzrlib.trace import mutter, note, is_quiet
 
73
from bzrlib.inventory import Inventory, InventoryEntry, make_entry
 
74
from bzrlib import symbol_versioning
 
75
from bzrlib.urlutils import unescape_for_display
 
76
 
 
77
 
 
78
class NullCommitReporter(object):
 
79
    """I report on progress of a commit."""
 
80
 
 
81
    def started(self, revno, revid, location=None):
 
82
        if location is None:
 
83
            symbol_versioning.warn("As of bzr 1.0 you must pass a location "
 
84
                                   "to started.", DeprecationWarning,
 
85
                                   stacklevel=2)
 
86
        pass
 
87
 
 
88
    def snapshot_change(self, change, path):
 
89
        pass
 
90
 
 
91
    def completed(self, revno, rev_id):
 
92
        pass
 
93
 
 
94
    def deleted(self, path):
 
95
        pass
 
96
 
 
97
    def missing(self, path):
 
98
        pass
 
99
 
 
100
    def renamed(self, change, old_path, new_path):
 
101
        pass
 
102
 
 
103
    def is_verbose(self):
 
104
        return False
 
105
 
 
106
 
 
107
class ReportCommitToLog(NullCommitReporter):
 
108
 
 
109
    def _note(self, format, *args):
 
110
        """Output a message.
 
111
 
 
112
        Subclasses may choose to override this method.
 
113
        """
 
114
        note(format, *args)
 
115
 
 
116
    def snapshot_change(self, change, path):
 
117
        if path == '' and change in ('added', 'modified'):
 
118
            return
 
119
        self._note("%s %s", change, path)
 
120
 
 
121
    def started(self, revno, rev_id, location=None):
 
122
        if location is not None:
 
123
            location = ' to: ' + unescape_for_display(location, 'utf-8')
 
124
        else:
 
125
            # When started was added, location was only made optional by
 
126
            # accident.  Matt Nordhoff 20071129
 
127
            symbol_versioning.warn("As of bzr 1.0 you must pass a location "
 
128
                                   "to started.", DeprecationWarning,
 
129
                                   stacklevel=2)
 
130
            location = ''
 
131
        self._note('Committing%s', location)
 
132
 
 
133
    def completed(self, revno, rev_id):
 
134
        self._note('Committed revision %d.', revno)
 
135
        # self._note goes to the console too; so while we want to log the
 
136
        # rev_id, we can't trivially only log it. (See bug 526425). Long
 
137
        # term we should rearrange the reporting structure, but for now
 
138
        # we just mutter seperately. We mutter the revid and revno together
 
139
        # so that concurrent bzr invocations won't lead to confusion.
 
140
        mutter('Committed revid %s as revno %d.', rev_id, revno)
 
141
 
 
142
    def deleted(self, path):
 
143
        self._note('deleted %s', path)
 
144
 
 
145
    def missing(self, path):
 
146
        self._note('missing %s', path)
 
147
 
 
148
    def renamed(self, change, old_path, new_path):
 
149
        self._note('%s %s => %s', change, old_path, new_path)
 
150
 
 
151
    def is_verbose(self):
 
152
        return True
 
153
 
 
154
 
 
155
class Commit(object):
 
156
    """Task of committing a new revision.
 
157
 
 
158
    This is a MethodObject: it accumulates state as the commit is
 
159
    prepared, and then it is discarded.  It doesn't represent
 
160
    historical revisions, just the act of recording a new one.
 
161
 
 
162
            missing_ids
 
163
            Modified to hold a list of files that have been deleted from
 
164
            the working directory; these should be removed from the
 
165
            working inventory.
 
166
    """
 
167
    def __init__(self,
 
168
                 reporter=None,
 
169
                 config=None):
 
170
        """Create a Commit object.
 
171
 
 
172
        :param reporter: the default reporter to use or None to decide later
 
173
        """
 
174
        self.reporter = reporter
 
175
        self.config = config
 
176
 
 
177
    @staticmethod
 
178
    def update_revprops(revprops, branch, authors=None, author=None,
 
179
                        local=False, possible_master_transports=None):
 
180
        if revprops is None:
 
181
            revprops = {}
 
182
        if possible_master_transports is None:
 
183
            possible_master_transports = []
 
184
        if not 'branch-nick' in revprops:
 
185
            revprops['branch-nick'] = branch._get_nick(
 
186
                local,
 
187
                possible_master_transports)
 
188
        if authors is not None:
 
189
            if author is not None:
 
190
                raise AssertionError('Specifying both author and authors '
 
191
                        'is not allowed. Specify just authors instead')
 
192
            if 'author' in revprops or 'authors' in revprops:
 
193
                # XXX: maybe we should just accept one of them?
 
194
                raise AssertionError('author property given twice')
 
195
            if authors:
 
196
                for individual in authors:
 
197
                    if '\n' in individual:
 
198
                        raise AssertionError('\\n is not a valid character '
 
199
                                'in an author identity')
 
200
                revprops['authors'] = '\n'.join(authors)
 
201
        if author is not None:
 
202
            symbol_versioning.warn('The parameter author was deprecated'
 
203
                   ' in version 1.13. Use authors instead',
 
204
                   DeprecationWarning)
 
205
            if 'author' in revprops or 'authors' in revprops:
 
206
                # XXX: maybe we should just accept one of them?
 
207
                raise AssertionError('author property given twice')
 
208
            if '\n' in author:
 
209
                raise AssertionError('\\n is not a valid character '
 
210
                        'in an author identity')
 
211
            revprops['authors'] = author
 
212
        return revprops
 
213
 
 
214
    def commit(self,
 
215
               message=None,
 
216
               timestamp=None,
 
217
               timezone=None,
 
218
               committer=None,
 
219
               specific_files=None,
 
220
               rev_id=None,
 
221
               allow_pointless=True,
 
222
               strict=False,
 
223
               verbose=False,
 
224
               revprops=None,
 
225
               working_tree=None,
 
226
               local=False,
 
227
               reporter=None,
 
228
               config=None,
 
229
               message_callback=None,
 
230
               recursive='down',
 
231
               exclude=None,
 
232
               possible_master_transports=None):
 
233
        """Commit working copy as a new revision.
 
234
 
 
235
        :param message: the commit message (it or message_callback is required)
 
236
        :param message_callback: A callback: message = message_callback(cmt_obj)
 
237
 
 
238
        :param timestamp: if not None, seconds-since-epoch for a
 
239
            postdated/predated commit.
 
240
 
 
241
        :param specific_files: If not None, commit only those files. An empty
 
242
            list means 'commit no files'.
 
243
 
 
244
        :param rev_id: If set, use this as the new revision id.
 
245
            Useful for test or import commands that need to tightly
 
246
            control what revisions are assigned.  If you duplicate
 
247
            a revision id that exists elsewhere it is your own fault.
 
248
            If null (default), a time/random revision id is generated.
 
249
 
 
250
        :param allow_pointless: If true (default), commit even if nothing
 
251
            has changed and no merges are recorded.
 
252
 
 
253
        :param strict: If true, don't allow a commit if the working tree
 
254
            contains unknown files.
 
255
 
 
256
        :param revprops: Properties for new revision
 
257
        :param local: Perform a local only commit.
 
258
        :param reporter: the reporter to use or None for the default
 
259
        :param verbose: if True and the reporter is not None, report everything
 
260
        :param recursive: If set to 'down', commit in any subtrees that have
 
261
            pending changes of any sort during this commit.
 
262
        :param exclude: None or a list of relative paths to exclude from the
 
263
            commit. Pending changes to excluded files will be ignored by the
 
264
            commit.
 
265
        """
 
266
        operation = OperationWithCleanups(self._commit)
 
267
        self.revprops = revprops or {}
 
268
        # XXX: Can be set on __init__ or passed in - this is a bit ugly.
 
269
        self.config = config or self.config
 
270
        return operation.run(
 
271
               message=message,
 
272
               timestamp=timestamp,
 
273
               timezone=timezone,
 
274
               committer=committer,
 
275
               specific_files=specific_files,
 
276
               rev_id=rev_id,
 
277
               allow_pointless=allow_pointless,
 
278
               strict=strict,
 
279
               verbose=verbose,
 
280
               working_tree=working_tree,
 
281
               local=local,
 
282
               reporter=reporter,
 
283
               message_callback=message_callback,
 
284
               recursive=recursive,
 
285
               exclude=exclude,
 
286
               possible_master_transports=possible_master_transports)
 
287
 
 
288
    def _commit(self, operation, message, timestamp, timezone, committer,
 
289
            specific_files, rev_id, allow_pointless, strict, verbose,
 
290
            working_tree, local, reporter, message_callback, recursive,
 
291
            exclude, possible_master_transports):
 
292
        mutter('preparing to commit')
 
293
 
 
294
        if working_tree is None:
 
295
            raise BzrError("working_tree must be passed into commit().")
 
296
        else:
 
297
            self.work_tree = working_tree
 
298
            self.branch = self.work_tree.branch
 
299
            if getattr(self.work_tree, 'requires_rich_root', lambda: False)():
 
300
                if not self.branch.repository.supports_rich_root():
 
301
                    raise errors.RootNotRich()
 
302
        if message_callback is None:
 
303
            if message is not None:
 
304
                if isinstance(message, str):
 
305
                    message = message.decode(get_user_encoding())
 
306
                message_callback = lambda x: message
 
307
            else:
 
308
                raise BzrError("The message or message_callback keyword"
 
309
                               " parameter is required for commit().")
 
310
 
 
311
        self.bound_branch = None
 
312
        self.any_entries_deleted = False
 
313
        if exclude is not None:
 
314
            self.exclude = sorted(
 
315
                minimum_path_selection(exclude))
 
316
        else:
 
317
            self.exclude = []
 
318
        self.local = local
 
319
        self.master_branch = None
 
320
        self.recursive = recursive
 
321
        self.rev_id = None
 
322
        # self.specific_files is None to indicate no filter, or any iterable to
 
323
        # indicate a filter - [] means no files at all, as per iter_changes.
 
324
        if specific_files is not None:
 
325
            self.specific_files = sorted(
 
326
                minimum_path_selection(specific_files))
 
327
        else:
 
328
            self.specific_files = None
 
329
            
 
330
        self.allow_pointless = allow_pointless
 
331
        self.message_callback = message_callback
 
332
        self.timestamp = timestamp
 
333
        self.timezone = timezone
 
334
        self.committer = committer
 
335
        self.strict = strict
 
336
        self.verbose = verbose
 
337
 
 
338
        self.work_tree.lock_write()
 
339
        operation.add_cleanup(self.work_tree.unlock)
 
340
        self.parents = self.work_tree.get_parent_ids()
 
341
        # We can use record_iter_changes IFF iter_changes is compatible with
 
342
        # the command line parameters, and the repository has fast delta
 
343
        # generation. See bug 347649.
 
344
        self.use_record_iter_changes = (
 
345
            not self.exclude and 
 
346
            not self.branch.repository._format.supports_tree_reference and
 
347
            (self.branch.repository._format.fast_deltas or
 
348
             len(self.parents) < 2))
 
349
        self.pb = ui.ui_factory.nested_progress_bar()
 
350
        operation.add_cleanup(self.pb.finished)
 
351
        self.basis_revid = self.work_tree.last_revision()
 
352
        self.basis_tree = self.work_tree.basis_tree()
 
353
        self.basis_tree.lock_read()
 
354
        operation.add_cleanup(self.basis_tree.unlock)
 
355
        # Cannot commit with conflicts present.
 
356
        if len(self.work_tree.conflicts()) > 0:
 
357
            raise ConflictsInTree
 
358
 
 
359
        # Setup the bound branch variables as needed.
 
360
        self._check_bound_branch(operation, possible_master_transports)
 
361
 
 
362
        # Check that the working tree is up to date
 
363
        old_revno, new_revno = self._check_out_of_date_tree()
 
364
 
 
365
        # Complete configuration setup
 
366
        if reporter is not None:
 
367
            self.reporter = reporter
 
368
        elif self.reporter is None:
 
369
            self.reporter = self._select_reporter()
 
370
        if self.config is None:
 
371
            self.config = self.branch.get_config()
 
372
 
 
373
        self._set_specific_file_ids()
 
374
 
 
375
        # Setup the progress bar. As the number of files that need to be
 
376
        # committed in unknown, progress is reported as stages.
 
377
        # We keep track of entries separately though and include that
 
378
        # information in the progress bar during the relevant stages.
 
379
        self.pb_stage_name = ""
 
380
        self.pb_stage_count = 0
 
381
        self.pb_stage_total = 5
 
382
        if self.bound_branch:
 
383
            # 2 extra stages: "Uploading data to master branch" and "Merging
 
384
            # tags to master branch"
 
385
            self.pb_stage_total += 2
 
386
        self.pb.show_pct = False
 
387
        self.pb.show_spinner = False
 
388
        self.pb.show_eta = False
 
389
        self.pb.show_count = True
 
390
        self.pb.show_bar = True
 
391
 
 
392
        self._gather_parents()
 
393
        # After a merge, a selected file commit is not supported.
 
394
        # See 'bzr help merge' for an explanation as to why.
 
395
        if len(self.parents) > 1 and self.specific_files is not None:
 
396
            raise errors.CannotCommitSelectedFileMerge(self.specific_files)
 
397
        # Excludes are a form of selected file commit.
 
398
        if len(self.parents) > 1 and self.exclude:
 
399
            raise errors.CannotCommitSelectedFileMerge(self.exclude)
 
400
 
 
401
        # Collect the changes
 
402
        self._set_progress_stage("Collecting changes", counter=True)
 
403
        self.builder = self.branch.get_commit_builder(self.parents,
 
404
            self.config, timestamp, timezone, committer, self.revprops, rev_id)
 
405
 
 
406
        try:
 
407
            self.builder.will_record_deletes()
 
408
            # find the location being committed to
 
409
            if self.bound_branch:
 
410
                master_location = self.master_branch.base
 
411
            else:
 
412
                master_location = self.branch.base
 
413
 
 
414
            # report the start of the commit
 
415
            self.reporter.started(new_revno, self.rev_id, master_location)
 
416
 
 
417
            self._update_builder_with_changes()
 
418
            self._check_pointless()
 
419
 
 
420
            # TODO: Now the new inventory is known, check for conflicts.
 
421
            # ADHB 2006-08-08: If this is done, populate_new_inv should not add
 
422
            # weave lines, because nothing should be recorded until it is known
 
423
            # that commit will succeed.
 
424
            self._set_progress_stage("Saving data locally")
 
425
            self.builder.finish_inventory()
 
426
 
 
427
            # Prompt the user for a commit message if none provided
 
428
            message = message_callback(self)
 
429
            self.message = message
 
430
 
 
431
            # Add revision data to the local branch
 
432
            self.rev_id = self.builder.commit(self.message)
 
433
 
 
434
        except Exception, e:
 
435
            mutter("aborting commit write group because of exception:")
 
436
            trace.log_exception_quietly()
 
437
            note("aborting commit write group: %r" % (e,))
 
438
            self.builder.abort()
 
439
            raise
 
440
 
 
441
        self._process_pre_hooks(old_revno, new_revno)
 
442
 
 
443
        # Upload revision data to the master.
 
444
        # this will propagate merged revisions too if needed.
 
445
        if self.bound_branch:
 
446
            self._set_progress_stage("Uploading data to master branch")
 
447
            # 'commit' to the master first so a timeout here causes the
 
448
            # local branch to be out of date
 
449
            self.master_branch.import_last_revision_info_and_tags(
 
450
                self.branch, new_revno, self.rev_id)
 
451
 
 
452
        # and now do the commit locally.
 
453
        self.branch.set_last_revision_info(new_revno, self.rev_id)
 
454
 
 
455
        # Merge local tags to remote
 
456
        if self.bound_branch:
 
457
            self._set_progress_stage("Merging tags to master branch")
 
458
            tag_conflicts = self.branch.tags.merge_to(self.master_branch.tags)
 
459
            if tag_conflicts:
 
460
                warning_lines = ['    ' + name for name, _, _ in tag_conflicts]
 
461
                note("Conflicting tags in bound branch:\n" +
 
462
                    "\n".join(warning_lines))
 
463
 
 
464
        # Make the working tree be up to date with the branch. This
 
465
        # includes automatic changes scheduled to be made to the tree, such
 
466
        # as updating its basis and unversioning paths that were missing.
 
467
        self.work_tree.unversion(self.deleted_ids)
 
468
        self._set_progress_stage("Updating the working tree")
 
469
        self.work_tree.update_basis_by_delta(self.rev_id,
 
470
             self.builder.get_basis_delta())
 
471
        self.reporter.completed(new_revno, self.rev_id)
 
472
        self._process_post_hooks(old_revno, new_revno)
 
473
        return self.rev_id
 
474
 
 
475
    def _select_reporter(self):
 
476
        """Select the CommitReporter to use."""
 
477
        if is_quiet():
 
478
            return NullCommitReporter()
 
479
        return ReportCommitToLog()
 
480
 
 
481
    def _check_pointless(self):
 
482
        if self.allow_pointless:
 
483
            return
 
484
        # A merge with no effect on files
 
485
        if len(self.parents) > 1:
 
486
            return
 
487
        # TODO: we could simplify this by using self.builder.basis_delta.
 
488
 
 
489
        # The initial commit adds a root directory, but this in itself is not
 
490
        # a worthwhile commit.
 
491
        if (self.basis_revid == revision.NULL_REVISION and
 
492
            ((self.builder.new_inventory is not None and
 
493
             len(self.builder.new_inventory) == 1) or
 
494
            len(self.builder._basis_delta) == 1)):
 
495
            raise PointlessCommit()
 
496
        if self.builder.any_changes():
 
497
            return
 
498
        raise PointlessCommit()
 
499
 
 
500
    def _check_bound_branch(self, operation, possible_master_transports=None):
 
501
        """Check to see if the local branch is bound.
 
502
 
 
503
        If it is bound, then most of the commit will actually be
 
504
        done using the remote branch as the target branch.
 
505
        Only at the end will the local branch be updated.
 
506
        """
 
507
        if self.local and not self.branch.get_bound_location():
 
508
            raise errors.LocalRequiresBoundBranch()
 
509
 
 
510
        if not self.local:
 
511
            self.master_branch = self.branch.get_master_branch(
 
512
                possible_master_transports)
 
513
 
 
514
        if not self.master_branch:
 
515
            # make this branch the reference branch for out of date checks.
 
516
            self.master_branch = self.branch
 
517
            return
 
518
 
 
519
        # If the master branch is bound, we must fail
 
520
        master_bound_location = self.master_branch.get_bound_location()
 
521
        if master_bound_location:
 
522
            raise errors.CommitToDoubleBoundBranch(self.branch,
 
523
                    self.master_branch, master_bound_location)
 
524
 
 
525
        # TODO: jam 20051230 We could automatically push local
 
526
        #       commits to the remote branch if they would fit.
 
527
        #       But for now, just require remote to be identical
 
528
        #       to local.
 
529
 
 
530
        # Make sure the local branch is identical to the master
 
531
        master_info = self.master_branch.last_revision_info()
 
532
        local_info = self.branch.last_revision_info()
 
533
        if local_info != master_info:
 
534
            raise errors.BoundBranchOutOfDate(self.branch,
 
535
                    self.master_branch)
 
536
 
 
537
        # Now things are ready to change the master branch
 
538
        # so grab the lock
 
539
        self.bound_branch = self.branch
 
540
        self.master_branch.lock_write()
 
541
        operation.add_cleanup(self.master_branch.unlock)
 
542
 
 
543
    def _check_out_of_date_tree(self):
 
544
        """Check that the working tree is up to date.
 
545
 
 
546
        :return: old_revision_number,new_revision_number tuple
 
547
        """
 
548
        try:
 
549
            first_tree_parent = self.work_tree.get_parent_ids()[0]
 
550
        except IndexError:
 
551
            # if there are no parents, treat our parent as 'None'
 
552
            # this is so that we still consider the master branch
 
553
            # - in a checkout scenario the tree may have no
 
554
            # parents but the branch may do.
 
555
            first_tree_parent = bzrlib.revision.NULL_REVISION
 
556
        old_revno, master_last = self.master_branch.last_revision_info()
 
557
        if master_last != first_tree_parent:
 
558
            if master_last != bzrlib.revision.NULL_REVISION:
 
559
                raise errors.OutOfDateTree(self.work_tree)
 
560
        if self.branch.repository.has_revision(first_tree_parent):
 
561
            new_revno = old_revno + 1
 
562
        else:
 
563
            # ghost parents never appear in revision history.
 
564
            new_revno = 1
 
565
        return old_revno,new_revno
 
566
 
 
567
    def _process_pre_hooks(self, old_revno, new_revno):
 
568
        """Process any registered pre commit hooks."""
 
569
        self._set_progress_stage("Running pre_commit hooks")
 
570
        self._process_hooks("pre_commit", old_revno, new_revno)
 
571
 
 
572
    def _process_post_hooks(self, old_revno, new_revno):
 
573
        """Process any registered post commit hooks."""
 
574
        # Process the post commit hooks, if any
 
575
        self._set_progress_stage("Running post_commit hooks")
 
576
        # old style commit hooks - should be deprecated ? (obsoleted in
 
577
        # 0.15)
 
578
        if self.config.post_commit() is not None:
 
579
            hooks = self.config.post_commit().split(' ')
 
580
            # this would be nicer with twisted.python.reflect.namedAny
 
581
            for hook in hooks:
 
582
                result = eval(hook + '(branch, rev_id)',
 
583
                              {'branch':self.branch,
 
584
                               'bzrlib':bzrlib,
 
585
                               'rev_id':self.rev_id})
 
586
        # process new style post commit hooks
 
587
        self._process_hooks("post_commit", old_revno, new_revno)
 
588
 
 
589
    def _process_hooks(self, hook_name, old_revno, new_revno):
 
590
        if not Branch.hooks[hook_name]:
 
591
            return
 
592
 
 
593
        # new style commit hooks:
 
594
        if not self.bound_branch:
 
595
            hook_master = self.branch
 
596
            hook_local = None
 
597
        else:
 
598
            hook_master = self.master_branch
 
599
            hook_local = self.branch
 
600
        # With bound branches, when the master is behind the local branch,
 
601
        # the 'old_revno' and old_revid values here are incorrect.
 
602
        # XXX: FIXME ^. RBC 20060206
 
603
        if self.parents:
 
604
            old_revid = self.parents[0]
 
605
        else:
 
606
            old_revid = bzrlib.revision.NULL_REVISION
 
607
 
 
608
        if hook_name == "pre_commit":
 
609
            future_tree = self.builder.revision_tree()
 
610
            tree_delta = future_tree.changes_from(self.basis_tree,
 
611
                                             include_root=True)
 
612
 
 
613
        for hook in Branch.hooks[hook_name]:
 
614
            # show the running hook in the progress bar. As hooks may
 
615
            # end up doing nothing (e.g. because they are not configured by
 
616
            # the user) this is still showing progress, not showing overall
 
617
            # actions - its up to each plugin to show a UI if it want's to
 
618
            # (such as 'Emailing diff to foo@example.com').
 
619
            self.pb_stage_name = "Running %s hooks [%s]" % \
 
620
                (hook_name, Branch.hooks.get_hook_name(hook))
 
621
            self._emit_progress()
 
622
            if 'hooks' in debug.debug_flags:
 
623
                mutter("Invoking commit hook: %r", hook)
 
624
            if hook_name == "post_commit":
 
625
                hook(hook_local, hook_master, old_revno, old_revid, new_revno,
 
626
                     self.rev_id)
 
627
            elif hook_name == "pre_commit":
 
628
                hook(hook_local, hook_master,
 
629
                     old_revno, old_revid, new_revno, self.rev_id,
 
630
                     tree_delta, future_tree)
 
631
 
 
632
    def _gather_parents(self):
 
633
        """Record the parents of a merge for merge detection."""
 
634
        # TODO: Make sure that this list doesn't contain duplicate
 
635
        # entries and the order is preserved when doing this.
 
636
        if self.use_record_iter_changes:
 
637
            return
 
638
        self.basis_inv = self.basis_tree.inventory
 
639
        self.parent_invs = [self.basis_inv]
 
640
        for revision in self.parents[1:]:
 
641
            if self.branch.repository.has_revision(revision):
 
642
                mutter('commit parent revision {%s}', revision)
 
643
                inventory = self.branch.repository.get_inventory(revision)
 
644
                self.parent_invs.append(inventory)
 
645
            else:
 
646
                mutter('commit parent ghost revision {%s}', revision)
 
647
 
 
648
    def _update_builder_with_changes(self):
 
649
        """Update the commit builder with the data about what has changed.
 
650
        """
 
651
        exclude = self.exclude
 
652
        specific_files = self.specific_files
 
653
        mutter("Selecting files for commit with filter %s", specific_files)
 
654
 
 
655
        self._check_strict()
 
656
        if self.use_record_iter_changes:
 
657
            iter_changes = self.work_tree.iter_changes(self.basis_tree,
 
658
                specific_files=specific_files)
 
659
            iter_changes = self._filter_iter_changes(iter_changes)
 
660
            for file_id, path, fs_hash in self.builder.record_iter_changes(
 
661
                self.work_tree, self.basis_revid, iter_changes):
 
662
                self.work_tree._observed_sha1(file_id, path, fs_hash)
 
663
        else:
 
664
            # Build the new inventory
 
665
            self._populate_from_inventory()
 
666
            self._record_unselected()
 
667
            self._report_and_accumulate_deletes()
 
668
 
 
669
    def _filter_iter_changes(self, iter_changes):
 
670
        """Process iter_changes.
 
671
 
 
672
        This method reports on the changes in iter_changes to the user, and 
 
673
        converts 'missing' entries in the iter_changes iterator to 'deleted'
 
674
        entries. 'missing' entries have their
 
675
 
 
676
        :param iter_changes: An iter_changes to process.
 
677
        :return: A generator of changes.
 
678
        """
 
679
        reporter = self.reporter
 
680
        report_changes = reporter.is_verbose()
 
681
        deleted_ids = []
 
682
        for change in iter_changes:
 
683
            if report_changes:
 
684
                old_path = change[1][0]
 
685
                new_path = change[1][1]
 
686
                versioned = change[3][1]
 
687
            kind = change[6][1]
 
688
            versioned = change[3][1]
 
689
            if kind is None and versioned:
 
690
                # 'missing' path
 
691
                if report_changes:
 
692
                    reporter.missing(new_path)
 
693
                deleted_ids.append(change[0])
 
694
                # Reset the new path (None) and new versioned flag (False)
 
695
                change = (change[0], (change[1][0], None), change[2],
 
696
                    (change[3][0], False)) + change[4:]
 
697
            elif kind == 'tree-reference':
 
698
                if self.recursive == 'down':
 
699
                    self._commit_nested_tree(change[0], change[1][1])
 
700
            if change[3][0] or change[3][1]:
 
701
                yield change
 
702
                if report_changes:
 
703
                    if new_path is None:
 
704
                        reporter.deleted(old_path)
 
705
                    elif old_path is None:
 
706
                        reporter.snapshot_change('added', new_path)
 
707
                    elif old_path != new_path:
 
708
                        reporter.renamed('renamed', old_path, new_path)
 
709
                    else:
 
710
                        if (new_path or 
 
711
                            self.work_tree.branch.repository._format.rich_root_data):
 
712
                            # Don't report on changes to '' in non rich root
 
713
                            # repositories.
 
714
                            reporter.snapshot_change('modified', new_path)
 
715
            self._next_progress_entry()
 
716
        # Unversion IDs that were found to be deleted
 
717
        self.deleted_ids = deleted_ids
 
718
 
 
719
    def _record_unselected(self):
 
720
        # If specific files are selected, then all un-selected files must be
 
721
        # recorded in their previous state. For more details, see
 
722
        # https://lists.ubuntu.com/archives/bazaar/2007q3/028476.html.
 
723
        if self.specific_files or self.exclude:
 
724
            specific_files = self.specific_files or []
 
725
            for path, old_ie in self.basis_inv.iter_entries():
 
726
                if old_ie.file_id in self.builder.new_inventory:
 
727
                    # already added - skip.
 
728
                    continue
 
729
                if (is_inside_any(specific_files, path)
 
730
                    and not is_inside_any(self.exclude, path)):
 
731
                    # was inside the selected path, and not excluded - if not
 
732
                    # present it has been deleted so skip.
 
733
                    continue
 
734
                # From here down it was either not selected, or was excluded:
 
735
                # We preserve the entry unaltered.
 
736
                ie = old_ie.copy()
 
737
                # Note: specific file commits after a merge are currently
 
738
                # prohibited. This test is for sanity/safety in case it's
 
739
                # required after that changes.
 
740
                if len(self.parents) > 1:
 
741
                    ie.revision = None
 
742
                self.builder.record_entry_contents(ie, self.parent_invs, path,
 
743
                    self.basis_tree, None)
 
744
 
 
745
    def _report_and_accumulate_deletes(self):
 
746
        if (isinstance(self.basis_inv, Inventory)
 
747
            and isinstance(self.builder.new_inventory, Inventory)):
 
748
            # the older Inventory classes provide a _byid dict, and building a
 
749
            # set from the keys of this dict is substantially faster than even
 
750
            # getting a set of ids from the inventory
 
751
            #
 
752
            # <lifeless> set(dict) is roughly the same speed as
 
753
            # set(iter(dict)) and both are significantly slower than
 
754
            # set(dict.keys())
 
755
            deleted_ids = set(self.basis_inv._byid.keys()) - \
 
756
               set(self.builder.new_inventory._byid.keys())
 
757
        else:
 
758
            deleted_ids = set(self.basis_inv) - set(self.builder.new_inventory)
 
759
        if deleted_ids:
 
760
            self.any_entries_deleted = True
 
761
            deleted = [(self.basis_tree.id2path(file_id), file_id)
 
762
                for file_id in deleted_ids]
 
763
            deleted.sort()
 
764
            # XXX: this is not quite directory-order sorting
 
765
            for path, file_id in deleted:
 
766
                self.builder.record_delete(path, file_id)
 
767
                self.reporter.deleted(path)
 
768
 
 
769
    def _check_strict(self):
 
770
        # XXX: when we use iter_changes this would likely be faster if
 
771
        # iter_changes would check for us (even in the presence of
 
772
        # selected_files).
 
773
        if self.strict:
 
774
            # raise an exception as soon as we find a single unknown.
 
775
            for unknown in self.work_tree.unknowns():
 
776
                raise StrictCommitFailed()
 
777
 
 
778
    def _populate_from_inventory(self):
 
779
        """Populate the CommitBuilder by walking the working tree inventory."""
 
780
        # Build the revision inventory.
 
781
        #
 
782
        # This starts by creating a new empty inventory. Depending on
 
783
        # which files are selected for commit, and what is present in the
 
784
        # current tree, the new inventory is populated. inventory entries
 
785
        # which are candidates for modification have their revision set to
 
786
        # None; inventory entries that are carried over untouched have their
 
787
        # revision set to their prior value.
 
788
        #
 
789
        # ESEPARATIONOFCONCERNS: this function is diffing and using the diff
 
790
        # results to create a new inventory at the same time, which results
 
791
        # in bugs like #46635.  Any reason not to use/enhance Tree.changes_from?
 
792
        # ADHB 11-07-2006
 
793
 
 
794
        specific_files = self.specific_files
 
795
        exclude = self.exclude
 
796
        report_changes = self.reporter.is_verbose()
 
797
        deleted_ids = []
 
798
        # A tree of paths that have been deleted. E.g. if foo/bar has been
 
799
        # deleted, then we have {'foo':{'bar':{}}}
 
800
        deleted_paths = {}
 
801
        # XXX: Note that entries may have the wrong kind because the entry does
 
802
        # not reflect the status on disk.
 
803
        work_inv = self.work_tree.inventory
 
804
        # NB: entries will include entries within the excluded ids/paths
 
805
        # because iter_entries_by_dir has no 'exclude' facility today.
 
806
        entries = work_inv.iter_entries_by_dir(
 
807
            specific_file_ids=self.specific_file_ids, yield_parents=True)
 
808
        for path, existing_ie in entries:
 
809
            file_id = existing_ie.file_id
 
810
            name = existing_ie.name
 
811
            parent_id = existing_ie.parent_id
 
812
            kind = existing_ie.kind
 
813
            # Skip files that have been deleted from the working tree.
 
814
            # The deleted path ids are also recorded so they can be explicitly
 
815
            # unversioned later.
 
816
            if deleted_paths:
 
817
                path_segments = splitpath(path)
 
818
                deleted_dict = deleted_paths
 
819
                for segment in path_segments:
 
820
                    deleted_dict = deleted_dict.get(segment, None)
 
821
                    if not deleted_dict:
 
822
                        # We either took a path not present in the dict
 
823
                        # (deleted_dict was None), or we've reached an empty
 
824
                        # child dir in the dict, so are now a sub-path.
 
825
                        break
 
826
                else:
 
827
                    deleted_dict = None
 
828
                if deleted_dict is not None:
 
829
                    # the path has a deleted parent, do not add it.
 
830
                    continue
 
831
            if exclude and is_inside_any(exclude, path):
 
832
                # Skip excluded paths. Excluded paths are processed by
 
833
                # _update_builder_with_changes.
 
834
                continue
 
835
            content_summary = self.work_tree.path_content_summary(path)
 
836
            kind = content_summary[0]
 
837
            # Note that when a filter of specific files is given, we must only
 
838
            # skip/record deleted files matching that filter.
 
839
            if not specific_files or is_inside_any(specific_files, path):
 
840
                if kind == 'missing':
 
841
                    if not deleted_paths:
 
842
                        # path won't have been split yet.
 
843
                        path_segments = splitpath(path)
 
844
                    deleted_dict = deleted_paths
 
845
                    for segment in path_segments:
 
846
                        deleted_dict = deleted_dict.setdefault(segment, {})
 
847
                    self.reporter.missing(path)
 
848
                    self._next_progress_entry()
 
849
                    deleted_ids.append(file_id)
 
850
                    continue
 
851
            # TODO: have the builder do the nested commit just-in-time IF and
 
852
            # only if needed.
 
853
            if kind == 'tree-reference':
 
854
                # enforce repository nested tree policy.
 
855
                if (not self.work_tree.supports_tree_reference() or
 
856
                    # repository does not support it either.
 
857
                    not self.branch.repository._format.supports_tree_reference):
 
858
                    kind = 'directory'
 
859
                    content_summary = (kind, None, None, None)
 
860
                elif self.recursive == 'down':
 
861
                    nested_revision_id = self._commit_nested_tree(
 
862
                        file_id, path)
 
863
                    content_summary = (kind, None, None, nested_revision_id)
 
864
                else:
 
865
                    nested_revision_id = self.work_tree.get_reference_revision(file_id)
 
866
                    content_summary = (kind, None, None, nested_revision_id)
 
867
 
 
868
            # Record an entry for this item
 
869
            # Note: I don't particularly want to have the existing_ie
 
870
            # parameter but the test suite currently (28-Jun-07) breaks
 
871
            # without it thanks to a unicode normalisation issue. :-(
 
872
            definitely_changed = kind != existing_ie.kind
 
873
            self._record_entry(path, file_id, specific_files, kind, name,
 
874
                parent_id, definitely_changed, existing_ie, report_changes,
 
875
                content_summary)
 
876
 
 
877
        # Unversion IDs that were found to be deleted
 
878
        self.deleted_ids = deleted_ids
 
879
 
 
880
    def _commit_nested_tree(self, file_id, path):
 
881
        "Commit a nested tree."
 
882
        sub_tree = self.work_tree.get_nested_tree(file_id, path)
 
883
        # FIXME: be more comprehensive here:
 
884
        # this works when both trees are in --trees repository,
 
885
        # but when both are bound to a different repository,
 
886
        # it fails; a better way of approaching this is to
 
887
        # finally implement the explicit-caches approach design
 
888
        # a while back - RBC 20070306.
 
889
        if sub_tree.branch.repository.has_same_location(
 
890
            self.work_tree.branch.repository):
 
891
            sub_tree.branch.repository = \
 
892
                self.work_tree.branch.repository
 
893
        try:
 
894
            return sub_tree.commit(message=None, revprops=self.revprops,
 
895
                recursive=self.recursive,
 
896
                message_callback=self.message_callback,
 
897
                timestamp=self.timestamp, timezone=self.timezone,
 
898
                committer=self.committer,
 
899
                allow_pointless=self.allow_pointless,
 
900
                strict=self.strict, verbose=self.verbose,
 
901
                local=self.local, reporter=self.reporter)
 
902
        except errors.PointlessCommit:
 
903
            return self.work_tree.get_reference_revision(file_id)
 
904
 
 
905
    def _record_entry(self, path, file_id, specific_files, kind, name,
 
906
        parent_id, definitely_changed, existing_ie, report_changes,
 
907
        content_summary):
 
908
        "Record the new inventory entry for a path if any."
 
909
        # mutter('check %s {%s}', path, file_id)
 
910
        # mutter('%s selected for commit', path)
 
911
        if definitely_changed or existing_ie is None:
 
912
            ie = make_entry(kind, name, parent_id, file_id)
 
913
        else:
 
914
            ie = existing_ie.copy()
 
915
            ie.revision = None
 
916
        # For carried over entries we don't care about the fs hash - the repo
 
917
        # isn't generating a sha, so we're not saving computation time.
 
918
        _, _, fs_hash = self.builder.record_entry_contents(
 
919
            ie, self.parent_invs, path, self.work_tree, content_summary)
 
920
        if report_changes:
 
921
            self._report_change(ie, path)
 
922
        if fs_hash:
 
923
            self.work_tree._observed_sha1(ie.file_id, path, fs_hash)
 
924
        return ie
 
925
 
 
926
    def _report_change(self, ie, path):
 
927
        """Report a change to the user.
 
928
 
 
929
        The change that has occurred is described relative to the basis
 
930
        inventory.
 
931
        """
 
932
        if (self.basis_inv.has_id(ie.file_id)):
 
933
            basis_ie = self.basis_inv[ie.file_id]
 
934
        else:
 
935
            basis_ie = None
 
936
        change = ie.describe_change(basis_ie, ie)
 
937
        if change in (InventoryEntry.RENAMED,
 
938
            InventoryEntry.MODIFIED_AND_RENAMED):
 
939
            old_path = self.basis_inv.id2path(ie.file_id)
 
940
            self.reporter.renamed(change, old_path, path)
 
941
            self._next_progress_entry()
 
942
        else:
 
943
            if change == 'unchanged':
 
944
                return
 
945
            self.reporter.snapshot_change(change, path)
 
946
            self._next_progress_entry()
 
947
 
 
948
    def _set_progress_stage(self, name, counter=False):
 
949
        """Set the progress stage and emit an update to the progress bar."""
 
950
        self.pb_stage_name = name
 
951
        self.pb_stage_count += 1
 
952
        if counter:
 
953
            self.pb_entries_count = 0
 
954
        else:
 
955
            self.pb_entries_count = None
 
956
        self._emit_progress()
 
957
 
 
958
    def _next_progress_entry(self):
 
959
        """Emit an update to the progress bar and increment the entry count."""
 
960
        self.pb_entries_count += 1
 
961
        self._emit_progress()
 
962
 
 
963
    def _emit_progress(self):
 
964
        if self.pb_entries_count is not None:
 
965
            text = "%s [%d] - Stage" % (self.pb_stage_name,
 
966
                self.pb_entries_count)
 
967
        else:
 
968
            text = "%s - Stage" % (self.pb_stage_name, )
 
969
        self.pb.update(text, self.pb_stage_count, self.pb_stage_total)
 
970
 
 
971
    def _set_specific_file_ids(self):
 
972
        """populate self.specific_file_ids if we will use it."""
 
973
        if not self.use_record_iter_changes:
 
974
            # If provided, ensure the specified files are versioned
 
975
            if self.specific_files is not None:
 
976
                # Note: This routine is being called because it raises
 
977
                # PathNotVersionedError as a side effect of finding the IDs. We
 
978
                # later use the ids we found as input to the working tree
 
979
                # inventory iterator, so we only consider those ids rather than
 
980
                # examining the whole tree again.
 
981
                # XXX: Dont we have filter_unversioned to do this more
 
982
                # cheaply?
 
983
                self.specific_file_ids = tree.find_ids_across_trees(
 
984
                    self.specific_files, [self.basis_tree, self.work_tree])
 
985
            else:
 
986
                self.specific_file_ids = None