~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/commit.py

  • Committer: aaron.bentley at utoronto
  • Date: 2005-08-27 04:42:41 UTC
  • mfrom: (1092.1.43)
  • mto: (1185.3.4)
  • mto: This revision was merged to the branch mainline in revision 1178.
  • Revision ID: aaron.bentley@utoronto.ca-20050827044241-23d676133b9fc981
Merge of robertc@robertcollins.net-20050826013321-52eee1f1da679ee9

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2006, 2007 Canonical Ltd
2
 
#
 
1
# Copyright (C) 2005 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
6
6
# (at your option) any later version.
7
 
#
 
7
 
8
8
# This program is distributed in the hope that it will be useful,
9
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
11
# GNU General Public License for more details.
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
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
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
 
import os
53
 
import re
54
 
import sys
55
 
import time
56
 
 
57
 
from cStringIO import StringIO
58
 
 
59
 
from bzrlib import (
60
 
    debug,
61
 
    errors,
62
 
    revision,
63
 
    tree,
64
 
    )
65
 
from bzrlib.branch import Branch
66
 
import bzrlib.config
67
 
from bzrlib.errors import (BzrError, PointlessCommit,
68
 
                           ConflictsInTree,
69
 
                           StrictCommitFailed
70
 
                           )
71
 
from bzrlib.osutils import (get_user_encoding,
72
 
                            kind_marker, isdir,isfile, is_inside_any,
73
 
                            is_inside_or_parent_of_any,
74
 
                            minimum_path_selection,
75
 
                            quotefn, sha_file, split_lines,
76
 
                            splitpath,
77
 
                            )
78
 
from bzrlib.testament import Testament
79
 
from bzrlib.trace import mutter, note, warning, is_quiet
80
 
from bzrlib.inventory import InventoryEntry, make_entry
81
 
from bzrlib import symbol_versioning
82
 
from bzrlib.symbol_versioning import (deprecated_passed,
83
 
        deprecated_function,
84
 
        DEPRECATED_PARAMETER)
85
 
from bzrlib.workingtree import WorkingTree
86
 
from bzrlib.urlutils import unescape_for_display
87
 
import bzrlib.ui
88
 
 
89
 
 
90
 
class NullCommitReporter(object):
91
 
    """I report on progress of a commit."""
92
 
 
93
 
    def started(self, revno, revid, location=None):
94
 
        if location is None:
95
 
            symbol_versioning.warn("As of bzr 1.0 you must pass a location "
96
 
                                   "to started.", DeprecationWarning,
97
 
                                   stacklevel=2)
98
 
        pass
99
 
 
100
 
    def snapshot_change(self, change, path):
101
 
        pass
102
 
 
103
 
    def completed(self, revno, rev_id):
104
 
        pass
105
 
 
106
 
    def deleted(self, file_id):
107
 
        pass
108
 
 
109
 
    def escaped(self, escape_count, message):
110
 
        pass
111
 
 
112
 
    def missing(self, path):
113
 
        pass
114
 
 
115
 
    def renamed(self, change, old_path, new_path):
116
 
        pass
117
 
 
118
 
    def is_verbose(self):
119
 
        return False
120
 
 
121
 
 
122
 
class ReportCommitToLog(NullCommitReporter):
123
 
 
124
 
    def _note(self, format, *args):
125
 
        """Output a message.
126
 
 
127
 
        Subclasses may choose to override this method.
128
 
        """
129
 
        note(format, *args)
130
 
 
131
 
    def snapshot_change(self, change, path):
132
 
        if change == 'unchanged':
133
 
            return
134
 
        if change == 'added' and path == '':
135
 
            return
136
 
        self._note("%s %s", change, path)
137
 
 
138
 
    def started(self, revno, rev_id, location=None):
139
 
        if location is not None:
140
 
            location = ' to: ' + unescape_for_display(location, 'utf-8')
141
 
        else:
142
 
            # When started was added, location was only made optional by
143
 
            # accident.  Matt Nordhoff 20071129
144
 
            symbol_versioning.warn("As of bzr 1.0 you must pass a location "
145
 
                                   "to started.", DeprecationWarning,
146
 
                                   stacklevel=2)
147
 
            location = ''
148
 
        self._note('Committing%s', location)
149
 
 
150
 
    def completed(self, revno, rev_id):
151
 
        self._note('Committed revision %d.', revno)
152
 
 
153
 
    def deleted(self, file_id):
154
 
        self._note('deleted %s', file_id)
155
 
 
156
 
    def escaped(self, escape_count, message):
157
 
        self._note("replaced %d control characters in message", escape_count)
158
 
 
159
 
    def missing(self, path):
160
 
        self._note('missing %s', path)
161
 
 
162
 
    def renamed(self, change, old_path, new_path):
163
 
        self._note('%s %s => %s', change, old_path, new_path)
164
 
 
165
 
    def is_verbose(self):
166
 
        return True
167
 
 
168
 
 
169
 
class Commit(object):
170
 
    """Task of committing a new revision.
171
 
 
172
 
    This is a MethodObject: it accumulates state as the commit is
173
 
    prepared, and then it is discarded.  It doesn't represent
174
 
    historical revisions, just the act of recording a new one.
175
 
 
176
 
            missing_ids
177
 
            Modified to hold a list of files that have been deleted from
178
 
            the working directory; these should be removed from the
179
 
            working inventory.
 
18
 
 
19
# FIXME: "bzr commit doc/format" commits doc/format.txt!
 
20
 
 
21
def commit(branch, message,
 
22
           timestamp=None,
 
23
           timezone=None,
 
24
           committer=None,
 
25
           verbose=True,
 
26
           specific_files=None,
 
27
           rev_id=None,
 
28
           allow_pointless=True):
 
29
    """Commit working copy as a new revision.
 
30
 
 
31
    The basic approach is to add all the file texts into the
 
32
    store, then the inventory, then make a new revision pointing
 
33
    to that inventory and store that.
 
34
 
 
35
    This is not quite safe if the working copy changes during the
 
36
    commit; for the moment that is simply not allowed.  A better
 
37
    approach is to make a temporary copy of the files before
 
38
    computing their hashes, and then add those hashes in turn to
 
39
    the inventory.  This should mean at least that there are no
 
40
    broken hash pointers.  There is no way we can get a snapshot
 
41
    of the whole directory at an instant.  This would also have to
 
42
    be robust against files disappearing, moving, etc.  So the
 
43
    whole thing is a bit hard.
 
44
 
 
45
    This raises PointlessCommit if there are no changes, no new merges,
 
46
    and allow_pointless  is false.
 
47
 
 
48
    timestamp -- if not None, seconds-since-epoch for a
 
49
         postdated/predated commit.
 
50
 
 
51
    specific_files
 
52
        If true, commit only those files.
 
53
 
 
54
    rev_id
 
55
        If set, use this as the new revision id.
 
56
        Useful for test or import commands that need to tightly
 
57
        control what revisions are assigned.  If you duplicate
 
58
        a revision id that exists elsewhere it is your own fault.
 
59
        If null (default), a time/random revision id is generated.
180
60
    """
181
 
    def __init__(self,
182
 
                 reporter=None,
183
 
                 config=None):
184
 
        """Create a Commit object.
185
 
 
186
 
        :param reporter: the default reporter to use or None to decide later
187
 
        """
188
 
        self.reporter = reporter
189
 
        self.config = config
190
 
 
191
 
    def commit(self,
192
 
               message=None,
193
 
               timestamp=None,
194
 
               timezone=None,
195
 
               committer=None,
196
 
               specific_files=None,
197
 
               rev_id=None,
198
 
               allow_pointless=True,
199
 
               strict=False,
200
 
               verbose=False,
201
 
               revprops=None,
202
 
               working_tree=None,
203
 
               local=False,
204
 
               reporter=None,
205
 
               config=None,
206
 
               message_callback=None,
207
 
               recursive='down',
208
 
               exclude=None,
209
 
               possible_master_transports=None):
210
 
        """Commit working copy as a new revision.
211
 
 
212
 
        :param message: the commit message (it or message_callback is required)
213
 
 
214
 
        :param timestamp: if not None, seconds-since-epoch for a
215
 
            postdated/predated commit.
216
 
 
217
 
        :param specific_files: If true, commit only those files.
218
 
 
219
 
        :param rev_id: If set, use this as the new revision id.
220
 
            Useful for test or import commands that need to tightly
221
 
            control what revisions are assigned.  If you duplicate
222
 
            a revision id that exists elsewhere it is your own fault.
223
 
            If null (default), a time/random revision id is generated.
224
 
 
225
 
        :param allow_pointless: If true (default), commit even if nothing
226
 
            has changed and no merges are recorded.
227
 
 
228
 
        :param strict: If true, don't allow a commit if the working tree
229
 
            contains unknown files.
230
 
 
231
 
        :param revprops: Properties for new revision
232
 
        :param local: Perform a local only commit.
233
 
        :param reporter: the reporter to use or None for the default
234
 
        :param verbose: if True and the reporter is not None, report everything
235
 
        :param recursive: If set to 'down', commit in any subtrees that have
236
 
            pending changes of any sort during this commit.
237
 
        :param exclude: None or a list of relative paths to exclude from the
238
 
            commit. Pending changes to excluded files will be ignored by the
239
 
            commit. 
240
 
        """
241
 
        mutter('preparing to commit')
242
 
 
243
 
        if working_tree is None:
244
 
            raise BzrError("working_tree must be passed into commit().")
245
 
        else:
246
 
            self.work_tree = working_tree
247
 
            self.branch = self.work_tree.branch
248
 
            if getattr(self.work_tree, 'requires_rich_root', lambda: False)():
249
 
                if not self.branch.repository.supports_rich_root():
250
 
                    raise errors.RootNotRich()
251
 
        if message_callback is None:
252
 
            if message is not None:
253
 
                if isinstance(message, str):
254
 
                    message = message.decode(get_user_encoding())
255
 
                message_callback = lambda x: message
256
 
            else:
257
 
                raise BzrError("The message or message_callback keyword"
258
 
                               " parameter is required for commit().")
259
 
 
260
 
        self.bound_branch = None
261
 
        self.any_entries_changed = False
262
 
        self.any_entries_deleted = False
263
 
        if exclude is not None:
264
 
            self.exclude = sorted(
265
 
                minimum_path_selection(exclude))
266
 
        else:
267
 
            self.exclude = []
268
 
        self.local = local
269
 
        self.master_branch = None
270
 
        self.master_locked = False
271
 
        self.recursive = recursive
272
 
        self.rev_id = None
273
 
        if specific_files is not None:
274
 
            self.specific_files = sorted(
275
 
                minimum_path_selection(specific_files))
276
 
        else:
277
 
            self.specific_files = None
278
 
        self.specific_file_ids = None
279
 
        self.allow_pointless = allow_pointless
280
 
        self.revprops = revprops
281
 
        self.message_callback = message_callback
282
 
        self.timestamp = timestamp
283
 
        self.timezone = timezone
284
 
        self.committer = committer
285
 
        self.strict = strict
286
 
        self.verbose = verbose
287
 
        # accumulates an inventory delta to the basis entry, so we can make
288
 
        # just the necessary updates to the workingtree's cached basis.
289
 
        self._basis_delta = []
290
 
 
291
 
        self.work_tree.lock_write()
292
 
        self.pb = bzrlib.ui.ui_factory.nested_progress_bar()
293
 
        self.basis_revid = self.work_tree.last_revision()
294
 
        self.basis_tree = self.work_tree.basis_tree()
295
 
        self.basis_tree.lock_read()
296
 
        try:
297
 
            # Cannot commit with conflicts present.
298
 
            if len(self.work_tree.conflicts()) > 0:
299
 
                raise ConflictsInTree
300
 
 
301
 
            # Setup the bound branch variables as needed.
302
 
            self._check_bound_branch(possible_master_transports)
303
 
 
304
 
            # Check that the working tree is up to date
305
 
            old_revno, new_revno = self._check_out_of_date_tree()
306
 
 
307
 
            # Complete configuration setup
308
 
            if reporter is not None:
309
 
                self.reporter = reporter
310
 
            elif self.reporter is None:
311
 
                self.reporter = self._select_reporter()
312
 
            if self.config is None:
313
 
                self.config = self.branch.get_config()
314
 
 
315
 
            # If provided, ensure the specified files are versioned
316
 
            if self.specific_files is not None:
317
 
                # Note: This routine is being called because it raises
318
 
                # PathNotVersionedError as a side effect of finding the IDs. We
319
 
                # later use the ids we found as input to the working tree
320
 
                # inventory iterator, so we only consider those ids rather than
321
 
                # examining the whole tree again.
322
 
                # XXX: Dont we have filter_unversioned to do this more
323
 
                # cheaply?
324
 
                self.specific_file_ids = tree.find_ids_across_trees(
325
 
                    specific_files, [self.basis_tree, self.work_tree])
326
 
 
327
 
            # Setup the progress bar. As the number of files that need to be
328
 
            # committed in unknown, progress is reported as stages.
329
 
            # We keep track of entries separately though and include that
330
 
            # information in the progress bar during the relevant stages.
331
 
            self.pb_stage_name = ""
332
 
            self.pb_stage_count = 0
333
 
            self.pb_stage_total = 5
334
 
            if self.bound_branch:
335
 
                self.pb_stage_total += 1
336
 
            self.pb.show_pct = False
337
 
            self.pb.show_spinner = False
338
 
            self.pb.show_eta = False
339
 
            self.pb.show_count = True
340
 
            self.pb.show_bar = True
341
 
 
342
 
            self.basis_inv = self.basis_tree.inventory
343
 
            self._gather_parents()
344
 
            # After a merge, a selected file commit is not supported.
345
 
            # See 'bzr help merge' for an explanation as to why.
346
 
            if len(self.parents) > 1 and self.specific_files:
347
 
                raise errors.CannotCommitSelectedFileMerge(self.specific_files)
348
 
            # Excludes are a form of selected file commit.
349
 
            if len(self.parents) > 1 and self.exclude:
350
 
                raise errors.CannotCommitSelectedFileMerge(self.exclude)
351
 
 
352
 
            # Collect the changes
353
 
            self._set_progress_stage("Collecting changes",
354
 
                    entries_title="Directory")
355
 
            self.builder = self.branch.get_commit_builder(self.parents,
356
 
                self.config, timestamp, timezone, committer, revprops, rev_id)
357
 
            
358
 
            try:
359
 
                # find the location being committed to
360
 
                if self.bound_branch:
361
 
                    master_location = self.master_branch.base
362
 
                else:
363
 
                    master_location = self.branch.base
364
 
 
365
 
                # report the start of the commit
366
 
                self.reporter.started(new_revno, self.rev_id, master_location)
367
 
 
368
 
                self._update_builder_with_changes()
369
 
                self._report_and_accumulate_deletes()
370
 
                self._check_pointless()
371
 
 
372
 
                # TODO: Now the new inventory is known, check for conflicts.
373
 
                # ADHB 2006-08-08: If this is done, populate_new_inv should not add
374
 
                # weave lines, because nothing should be recorded until it is known
375
 
                # that commit will succeed.
376
 
                self._set_progress_stage("Saving data locally")
377
 
                self.builder.finish_inventory()
378
 
 
379
 
                # Prompt the user for a commit message if none provided
380
 
                message = message_callback(self)
381
 
                self.message = message
382
 
                self._escape_commit_message()
383
 
 
384
 
                # Add revision data to the local branch
385
 
                self.rev_id = self.builder.commit(self.message)
386
 
 
387
 
            except:
388
 
                self.builder.abort()
389
 
                raise
390
 
 
391
 
            self._process_pre_hooks(old_revno, new_revno)
392
 
 
393
 
            # Upload revision data to the master.
394
 
            # this will propagate merged revisions too if needed.
395
 
            if self.bound_branch:
396
 
                if not self.master_branch.repository.has_same_location(
397
 
                        self.branch.repository):
398
 
                    self._set_progress_stage("Uploading data to master branch")
399
 
                    self.master_branch.repository.fetch(self.branch.repository,
400
 
                        revision_id=self.rev_id)
401
 
                # now the master has the revision data
402
 
                # 'commit' to the master first so a timeout here causes the
403
 
                # local branch to be out of date
404
 
                self.master_branch.set_last_revision_info(new_revno,
405
 
                                                          self.rev_id)
406
 
 
407
 
            # and now do the commit locally.
408
 
            self.branch.set_last_revision_info(new_revno, self.rev_id)
409
 
 
410
 
            # Make the working tree up to date with the branch
411
 
            self._set_progress_stage("Updating the working tree")
412
 
            self.work_tree.update_basis_by_delta(self.rev_id,
413
 
                 self._basis_delta)
414
 
            self.reporter.completed(new_revno, self.rev_id)
415
 
            self._process_post_hooks(old_revno, new_revno)
416
 
        finally:
417
 
            self._cleanup()
418
 
        return self.rev_id
419
 
 
420
 
    def _select_reporter(self):
421
 
        """Select the CommitReporter to use."""
422
 
        if is_quiet():
423
 
            return NullCommitReporter()
424
 
        return ReportCommitToLog()
425
 
 
426
 
    def _check_pointless(self):
427
 
        if self.allow_pointless:
428
 
            return
429
 
        # A merge with no effect on files
430
 
        if len(self.parents) > 1:
431
 
            return
432
 
        # TODO: we could simplify this by using self._basis_delta.
433
 
 
434
 
        # The initial commit adds a root directory, but this in itself is not
435
 
        # a worthwhile commit.
436
 
        if (self.basis_revid == revision.NULL_REVISION and
437
 
            len(self.builder.new_inventory) == 1):
 
61
 
 
62
    import time, tempfile
 
63
 
 
64
    from bzrlib.osutils import local_time_offset, username
 
65
    from bzrlib.branch import gen_file_id
 
66
    from bzrlib.errors import BzrError, PointlessCommit
 
67
    from bzrlib.revision import Revision, RevisionReference
 
68
    from bzrlib.trace import mutter, note
 
69
    from bzrlib.xml import pack_xml
 
70
 
 
71
    branch.lock_write()
 
72
 
 
73
    try:
 
74
        # First walk over the working inventory; and both update that
 
75
        # and also build a new revision inventory.  The revision
 
76
        # inventory needs to hold the text-id, sha1 and size of the
 
77
        # actual file versions committed in the revision.  (These are
 
78
        # not present in the working inventory.)  We also need to
 
79
        # detect missing/deleted files, and remove them from the
 
80
        # working inventory.
 
81
 
 
82
        work_tree = branch.working_tree()
 
83
        work_inv = work_tree.inventory
 
84
        basis = branch.basis_tree()
 
85
        basis_inv = basis.inventory
 
86
 
 
87
        if verbose:
 
88
            # note('looking for changes...')
 
89
            # print 'looking for changes...'
 
90
            # disabled; should be done at a higher level
 
91
            pass
 
92
 
 
93
        pending_merges = branch.pending_merges()
 
94
 
 
95
        missing_ids, new_inv, any_changes = \
 
96
                     _gather_commit(branch,
 
97
                                    work_tree,
 
98
                                    work_inv,
 
99
                                    basis_inv,
 
100
                                    specific_files,
 
101
                                    verbose)
 
102
 
 
103
        if not (any_changes or allow_pointless or pending_merges):
438
104
            raise PointlessCommit()
439
 
        # If length == 1, then we only have the root entry. Which means
440
 
        # that there is no real difference (only the root could be different)
441
 
        # unless deletes occured, in which case the length is irrelevant.
442
 
        if (self.any_entries_deleted or 
443
 
            (len(self.builder.new_inventory) != 1 and
444
 
             self.any_entries_changed)):
445
 
            return
446
 
        raise PointlessCommit()
447
 
 
448
 
    def _check_bound_branch(self, possible_master_transports=None):
449
 
        """Check to see if the local branch is bound.
450
 
 
451
 
        If it is bound, then most of the commit will actually be
452
 
        done using the remote branch as the target branch.
453
 
        Only at the end will the local branch be updated.
454
 
        """
455
 
        if self.local and not self.branch.get_bound_location():
456
 
            raise errors.LocalRequiresBoundBranch()
457
 
 
458
 
        if not self.local:
459
 
            self.master_branch = self.branch.get_master_branch(
460
 
                possible_master_transports)
461
 
 
462
 
        if not self.master_branch:
463
 
            # make this branch the reference branch for out of date checks.
464
 
            self.master_branch = self.branch
465
 
            return
466
 
 
467
 
        # If the master branch is bound, we must fail
468
 
        master_bound_location = self.master_branch.get_bound_location()
469
 
        if master_bound_location:
470
 
            raise errors.CommitToDoubleBoundBranch(self.branch,
471
 
                    self.master_branch, master_bound_location)
472
 
 
473
 
        # TODO: jam 20051230 We could automatically push local
474
 
        #       commits to the remote branch if they would fit.
475
 
        #       But for now, just require remote to be identical
476
 
        #       to local.
477
 
        
478
 
        # Make sure the local branch is identical to the master
479
 
        master_info = self.master_branch.last_revision_info()
480
 
        local_info = self.branch.last_revision_info()
481
 
        if local_info != master_info:
482
 
            raise errors.BoundBranchOutOfDate(self.branch,
483
 
                    self.master_branch)
484
 
 
485
 
        # Now things are ready to change the master branch
486
 
        # so grab the lock
487
 
        self.bound_branch = self.branch
488
 
        self.master_branch.lock_write()
489
 
        self.master_locked = True
490
 
 
491
 
    def _check_out_of_date_tree(self):
492
 
        """Check that the working tree is up to date.
493
 
 
494
 
        :return: old_revision_number,new_revision_number tuple
495
 
        """
496
 
        try:
497
 
            first_tree_parent = self.work_tree.get_parent_ids()[0]
498
 
        except IndexError:
499
 
            # if there are no parents, treat our parent as 'None'
500
 
            # this is so that we still consider the master branch
501
 
            # - in a checkout scenario the tree may have no
502
 
            # parents but the branch may do.
503
 
            first_tree_parent = bzrlib.revision.NULL_REVISION
504
 
        old_revno, master_last = self.master_branch.last_revision_info()
505
 
        if master_last != first_tree_parent:
506
 
            if master_last != bzrlib.revision.NULL_REVISION:
507
 
                raise errors.OutOfDateTree(self.work_tree)
508
 
        if self.branch.repository.has_revision(first_tree_parent):
509
 
            new_revno = old_revno + 1
510
 
        else:
511
 
            # ghost parents never appear in revision history.
512
 
            new_revno = 1
513
 
        return old_revno,new_revno
514
 
 
515
 
    def _process_pre_hooks(self, old_revno, new_revno):
516
 
        """Process any registered pre commit hooks."""
517
 
        self._set_progress_stage("Running pre_commit hooks")
518
 
        self._process_hooks("pre_commit", old_revno, new_revno)
519
 
 
520
 
    def _process_post_hooks(self, old_revno, new_revno):
521
 
        """Process any registered post commit hooks."""
522
 
        # Process the post commit hooks, if any
523
 
        self._set_progress_stage("Running post_commit hooks")
524
 
        # old style commit hooks - should be deprecated ? (obsoleted in
525
 
        # 0.15)
526
 
        if self.config.post_commit() is not None:
527
 
            hooks = self.config.post_commit().split(' ')
528
 
            # this would be nicer with twisted.python.reflect.namedAny
529
 
            for hook in hooks:
530
 
                result = eval(hook + '(branch, rev_id)',
531
 
                              {'branch':self.branch,
532
 
                               'bzrlib':bzrlib,
533
 
                               'rev_id':self.rev_id})
534
 
        # process new style post commit hooks
535
 
        self._process_hooks("post_commit", old_revno, new_revno)
536
 
 
537
 
    def _process_hooks(self, hook_name, old_revno, new_revno):
538
 
        if not Branch.hooks[hook_name]:
539
 
            return
540
 
        
541
 
        # new style commit hooks:
542
 
        if not self.bound_branch:
543
 
            hook_master = self.branch
544
 
            hook_local = None
545
 
        else:
546
 
            hook_master = self.master_branch
547
 
            hook_local = self.branch
548
 
        # With bound branches, when the master is behind the local branch,
549
 
        # the 'old_revno' and old_revid values here are incorrect.
550
 
        # XXX: FIXME ^. RBC 20060206
551
 
        if self.parents:
552
 
            old_revid = self.parents[0]
553
 
        else:
554
 
            old_revid = bzrlib.revision.NULL_REVISION
555
 
        
556
 
        if hook_name == "pre_commit":
557
 
            future_tree = self.builder.revision_tree()
558
 
            tree_delta = future_tree.changes_from(self.basis_tree,
559
 
                                             include_root=True)
560
 
        
561
 
        for hook in Branch.hooks[hook_name]:
562
 
            # show the running hook in the progress bar. As hooks may
563
 
            # end up doing nothing (e.g. because they are not configured by
564
 
            # the user) this is still showing progress, not showing overall
565
 
            # actions - its up to each plugin to show a UI if it want's to
566
 
            # (such as 'Emailing diff to foo@example.com').
567
 
            self.pb_stage_name = "Running %s hooks [%s]" % \
568
 
                (hook_name, Branch.hooks.get_hook_name(hook))
569
 
            self._emit_progress()
570
 
            if 'hooks' in debug.debug_flags:
571
 
                mutter("Invoking commit hook: %r", hook)
572
 
            if hook_name == "post_commit":
573
 
                hook(hook_local, hook_master, old_revno, old_revid, new_revno,
574
 
                     self.rev_id)
575
 
            elif hook_name == "pre_commit":
576
 
                hook(hook_local, hook_master,
577
 
                     old_revno, old_revid, new_revno, self.rev_id,
578
 
                     tree_delta, future_tree)
579
 
 
580
 
    def _cleanup(self):
581
 
        """Cleanup any open locks, progress bars etc."""
582
 
        cleanups = [self._cleanup_bound_branch,
583
 
                    self.basis_tree.unlock,
584
 
                    self.work_tree.unlock,
585
 
                    self.pb.finished]
586
 
        found_exception = None
587
 
        for cleanup in cleanups:
588
 
            try:
589
 
                cleanup()
590
 
            # we want every cleanup to run no matter what.
591
 
            # so we have a catchall here, but we will raise the
592
 
            # last encountered exception up the stack: and
593
 
            # typically this will be useful enough.
594
 
            except Exception, e:
595
 
                found_exception = e
596
 
        if found_exception is not None: 
597
 
            # don't do a plan raise, because the last exception may have been
598
 
            # trashed, e is our sure-to-work exception even though it loses the
599
 
            # full traceback. XXX: RBC 20060421 perhaps we could check the
600
 
            # exc_info and if its the same one do a plain raise otherwise 
601
 
            # 'raise e' as we do now.
602
 
            raise e
603
 
 
604
 
    def _cleanup_bound_branch(self):
605
 
        """Executed at the end of a try/finally to cleanup a bound branch.
606
 
 
607
 
        If the branch wasn't bound, this is a no-op.
608
 
        If it was, it resents self.branch to the local branch, instead
609
 
        of being the master.
610
 
        """
611
 
        if not self.bound_branch:
612
 
            return
613
 
        if self.master_locked:
614
 
            self.master_branch.unlock()
615
 
 
616
 
    def _escape_commit_message(self):
617
 
        """Replace xml-incompatible control characters."""
618
 
        # FIXME: RBC 20060419 this should be done by the revision
619
 
        # serialiser not by commit. Then we can also add an unescaper
620
 
        # in the deserializer and start roundtripping revision messages
621
 
        # precisely. See repository_implementations/test_repository.py
622
 
        
623
 
        # Python strings can include characters that can't be
624
 
        # represented in well-formed XML; escape characters that
625
 
        # aren't listed in the XML specification
626
 
        # (http://www.w3.org/TR/REC-xml/#NT-Char).
627
 
        self.message, escape_count = re.subn(
628
 
            u'[^\x09\x0A\x0D\u0020-\uD7FF\uE000-\uFFFD]+',
629
 
            lambda match: match.group(0).encode('unicode_escape'),
630
 
            self.message)
631
 
        if escape_count:
632
 
            self.reporter.escaped(escape_count, self.message)
633
 
 
634
 
    def _gather_parents(self):
635
 
        """Record the parents of a merge for merge detection."""
636
 
        # TODO: Make sure that this list doesn't contain duplicate 
637
 
        # entries and the order is preserved when doing this.
638
 
        self.parents = self.work_tree.get_parent_ids()
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
 
        # Build the revision inventory.
652
 
        #
653
 
        # This starts by creating a new empty inventory. Depending on
654
 
        # which files are selected for commit, and what is present in the
655
 
        # current tree, the new inventory is populated. inventory entries 
656
 
        # which are candidates for modification have their revision set to
657
 
        # None; inventory entries that are carried over untouched have their
658
 
        # revision set to their prior value.
659
 
        #
660
 
        # ESEPARATIONOFCONCERNS: this function is diffing and using the diff
661
 
        # results to create a new inventory at the same time, which results
662
 
        # in bugs like #46635.  Any reason not to use/enhance Tree.changes_from?
663
 
        # ADHB 11-07-2006
664
 
 
665
 
        exclude = self.exclude
666
 
        specific_files = self.specific_files or []
667
 
        mutter("Selecting files for commit with filter %s", specific_files)
668
 
 
669
 
        # Build the new inventory
670
 
        self._populate_from_inventory()
671
 
 
672
 
        # If specific files are selected, then all un-selected files must be
673
 
        # recorded in their previous state. For more details, see
674
 
        # https://lists.ubuntu.com/archives/bazaar/2007q3/028476.html.
675
 
        if specific_files or exclude:
676
 
            for path, old_ie in self.basis_inv.iter_entries():
677
 
                if old_ie.file_id in self.builder.new_inventory:
678
 
                    # already added - skip.
679
 
                    continue
680
 
                if (is_inside_any(specific_files, path)
681
 
                    and not is_inside_any(exclude, path)):
682
 
                    # was inside the selected path, and not excluded - if not
683
 
                    # present it has been deleted so skip.
684
 
                    continue
685
 
                # From here down it was either not selected, or was excluded:
686
 
                if old_ie.kind == 'directory':
687
 
                    self._next_progress_entry()
688
 
                # We preserve the entry unaltered.
689
 
                ie = old_ie.copy()
690
 
                # Note: specific file commits after a merge are currently
691
 
                # prohibited. This test is for sanity/safety in case it's
692
 
                # required after that changes.
693
 
                if len(self.parents) > 1:
694
 
                    ie.revision = None
695
 
                delta, version_recorded, _ = self.builder.record_entry_contents(
696
 
                    ie, self.parent_invs, path, self.basis_tree, None)
697
 
                if version_recorded:
698
 
                    self.any_entries_changed = True
699
 
                if delta:
700
 
                    self._basis_delta.append(delta)
701
 
 
702
 
    def _report_and_accumulate_deletes(self):
703
 
        # XXX: Could the list of deleted paths and ids be instead taken from
704
 
        # _populate_from_inventory?
705
 
        deleted_ids = set(self.basis_inv._byid.keys()) - \
706
 
            set(self.builder.new_inventory._byid.keys())
707
 
        if deleted_ids:
708
 
            self.any_entries_deleted = True
709
 
            deleted = [(self.basis_tree.id2path(file_id), file_id)
710
 
                for file_id in deleted_ids]
711
 
            deleted.sort()
712
 
            # XXX: this is not quite directory-order sorting
713
 
            for path, file_id in deleted:
714
 
                self._basis_delta.append((path, None, file_id, None))
715
 
                self.reporter.deleted(path)
716
 
 
717
 
    def _populate_from_inventory(self):
718
 
        """Populate the CommitBuilder by walking the working tree inventory."""
719
 
        if self.strict:
720
 
            # raise an exception as soon as we find a single unknown.
721
 
            for unknown in self.work_tree.unknowns():
722
 
                raise StrictCommitFailed()
723
 
        
724
 
        specific_files = self.specific_files
725
 
        exclude = self.exclude
726
 
        report_changes = self.reporter.is_verbose()
727
 
        deleted_ids = []
728
 
        # A tree of paths that have been deleted. E.g. if foo/bar has been
729
 
        # deleted, then we have {'foo':{'bar':{}}}
730
 
        deleted_paths = {}
731
 
        # XXX: Note that entries may have the wrong kind because the entry does
732
 
        # not reflect the status on disk.
733
 
        work_inv = self.work_tree.inventory
734
 
        # NB: entries will include entries within the excluded ids/paths
735
 
        # because iter_entries_by_dir has no 'exclude' facility today.
736
 
        entries = work_inv.iter_entries_by_dir(
737
 
            specific_file_ids=self.specific_file_ids, yield_parents=True)
738
 
        for path, existing_ie in entries:
739
 
            file_id = existing_ie.file_id
740
 
            name = existing_ie.name
741
 
            parent_id = existing_ie.parent_id
742
 
            kind = existing_ie.kind
743
 
            if kind == 'directory':
744
 
                self._next_progress_entry()
745
 
            # Skip files that have been deleted from the working tree.
746
 
            # The deleted path ids are also recorded so they can be explicitly
747
 
            # unversioned later.
748
 
            if deleted_paths:
749
 
                path_segments = splitpath(path)
750
 
                deleted_dict = deleted_paths
751
 
                for segment in path_segments:
752
 
                    deleted_dict = deleted_dict.get(segment, None)
753
 
                    if not deleted_dict:
754
 
                        # We either took a path not present in the dict
755
 
                        # (deleted_dict was None), or we've reached an empty
756
 
                        # child dir in the dict, so are now a sub-path.
757
 
                        break
758
 
                else:
759
 
                    deleted_dict = None
760
 
                if deleted_dict is not None:
761
 
                    # the path has a deleted parent, do not add it.
762
 
                    continue
763
 
            if exclude and is_inside_any(exclude, path):
764
 
                # Skip excluded paths. Excluded paths are processed by
765
 
                # _update_builder_with_changes.
766
 
                continue
767
 
            content_summary = self.work_tree.path_content_summary(path)
768
 
            # Note that when a filter of specific files is given, we must only
769
 
            # skip/record deleted files matching that filter.
770
 
            if not specific_files or is_inside_any(specific_files, path):
771
 
                if content_summary[0] == 'missing':
772
 
                    if not deleted_paths:
773
 
                        # path won't have been split yet.
774
 
                        path_segments = splitpath(path)
775
 
                    deleted_dict = deleted_paths
776
 
                    for segment in path_segments:
777
 
                        deleted_dict = deleted_dict.setdefault(segment, {})
778
 
                    self.reporter.missing(path)
779
 
                    deleted_ids.append(file_id)
780
 
                    continue
781
 
            # TODO: have the builder do the nested commit just-in-time IF and
782
 
            # only if needed.
783
 
            if content_summary[0] == 'tree-reference':
784
 
                # enforce repository nested tree policy.
785
 
                if (not self.work_tree.supports_tree_reference() or
786
 
                    # repository does not support it either.
787
 
                    not self.branch.repository._format.supports_tree_reference):
788
 
                    content_summary = ('directory',) + content_summary[1:]
789
 
            kind = content_summary[0]
790
 
            # TODO: specific_files filtering before nested tree processing
791
 
            if kind == 'tree-reference':
792
 
                if self.recursive == 'down':
793
 
                    nested_revision_id = self._commit_nested_tree(
794
 
                        file_id, path)
795
 
                    content_summary = content_summary[:3] + (
796
 
                        nested_revision_id,)
797
 
                else:
798
 
                    content_summary = content_summary[:3] + (
799
 
                        self.work_tree.get_reference_revision(file_id),)
800
 
 
801
 
            # Record an entry for this item
802
 
            # Note: I don't particularly want to have the existing_ie
803
 
            # parameter but the test suite currently (28-Jun-07) breaks
804
 
            # without it thanks to a unicode normalisation issue. :-(
805
 
            definitely_changed = kind != existing_ie.kind
806
 
            self._record_entry(path, file_id, specific_files, kind, name,
807
 
                parent_id, definitely_changed, existing_ie, report_changes,
808
 
                content_summary)
809
 
 
810
 
        # Unversion IDs that were found to be deleted
811
 
        self.work_tree.unversion(deleted_ids)
812
 
 
813
 
    def _commit_nested_tree(self, file_id, path):
814
 
        "Commit a nested tree."
815
 
        sub_tree = self.work_tree.get_nested_tree(file_id, path)
816
 
        # FIXME: be more comprehensive here:
817
 
        # this works when both trees are in --trees repository,
818
 
        # but when both are bound to a different repository,
819
 
        # it fails; a better way of approaching this is to 
820
 
        # finally implement the explicit-caches approach design
821
 
        # a while back - RBC 20070306.
822
 
        if sub_tree.branch.repository.has_same_location(
823
 
            self.work_tree.branch.repository):
824
 
            sub_tree.branch.repository = \
825
 
                self.work_tree.branch.repository
826
 
        try:
827
 
            return sub_tree.commit(message=None, revprops=self.revprops,
828
 
                recursive=self.recursive,
829
 
                message_callback=self.message_callback,
830
 
                timestamp=self.timestamp, timezone=self.timezone,
831
 
                committer=self.committer,
832
 
                allow_pointless=self.allow_pointless,
833
 
                strict=self.strict, verbose=self.verbose,
834
 
                local=self.local, reporter=self.reporter)
835
 
        except errors.PointlessCommit:
836
 
            return self.work_tree.get_reference_revision(file_id)
837
 
 
838
 
    def _record_entry(self, path, file_id, specific_files, kind, name,
839
 
        parent_id, definitely_changed, existing_ie, report_changes,
840
 
        content_summary):
841
 
        "Record the new inventory entry for a path if any."
842
 
        # mutter('check %s {%s}', path, file_id)
843
 
        # mutter('%s selected for commit', path)
844
 
        if definitely_changed or existing_ie is None:
845
 
            ie = make_entry(kind, name, parent_id, file_id)
846
 
        else:
847
 
            ie = existing_ie.copy()
848
 
            ie.revision = None
849
 
        # For carried over entries we don't care about the fs hash - the repo
850
 
        # isn't generating a sha, so we're not saving computation time.
851
 
        delta, version_recorded, fs_hash = self.builder.record_entry_contents(
852
 
            ie, self.parent_invs, path, self.work_tree, content_summary)
853
 
        if delta:
854
 
            self._basis_delta.append(delta)
855
 
        if version_recorded:
856
 
            self.any_entries_changed = True
857
 
        if report_changes:
858
 
            self._report_change(ie, path)
859
 
        if fs_hash:
860
 
            self.work_tree._observed_sha1(ie.file_id, path, fs_hash)
861
 
        return ie
862
 
 
863
 
    def _report_change(self, ie, path):
864
 
        """Report a change to the user.
865
 
 
866
 
        The change that has occurred is described relative to the basis
867
 
        inventory.
868
 
        """
869
 
        if (self.basis_inv.has_id(ie.file_id)):
870
 
            basis_ie = self.basis_inv[ie.file_id]
871
 
        else:
872
 
            basis_ie = None
873
 
        change = ie.describe_change(basis_ie, ie)
874
 
        if change in (InventoryEntry.RENAMED, 
875
 
            InventoryEntry.MODIFIED_AND_RENAMED):
876
 
            old_path = self.basis_inv.id2path(ie.file_id)
877
 
            self.reporter.renamed(change, old_path, path)
878
 
        else:
879
 
            self.reporter.snapshot_change(change, path)
880
 
 
881
 
    def _set_progress_stage(self, name, entries_title=None):
882
 
        """Set the progress stage and emit an update to the progress bar."""
883
 
        self.pb_stage_name = name
884
 
        self.pb_stage_count += 1
885
 
        self.pb_entries_title = entries_title
886
 
        if entries_title is not None:
887
 
            self.pb_entries_count = 0
888
 
            self.pb_entries_total = '?'
889
 
        self._emit_progress()
890
 
 
891
 
    def _next_progress_entry(self):
892
 
        """Emit an update to the progress bar and increment the entry count."""
893
 
        self.pb_entries_count += 1
894
 
        self._emit_progress()
895
 
 
896
 
    def _emit_progress(self):
897
 
        if self.pb_entries_title:
898
 
            if self.pb_entries_total == '?':
899
 
                text = "%s [%s %d] - Stage" % (self.pb_stage_name,
900
 
                    self.pb_entries_title, self.pb_entries_count)
901
 
            else:
902
 
                text = "%s [%s %d/%s] - Stage" % (self.pb_stage_name,
903
 
                    self.pb_entries_title, self.pb_entries_count,
904
 
                    str(self.pb_entries_total))
905
 
        else:
906
 
            text = "%s - Stage" % (self.pb_stage_name)
907
 
        self.pb.update(text, self.pb_stage_count, self.pb_stage_total)
 
105
 
 
106
        for file_id in missing_ids:
 
107
            # Any files that have been deleted are now removed from the
 
108
            # working inventory.  Files that were not selected for commit
 
109
            # are left as they were in the working inventory and ommitted
 
110
            # from the revision inventory.
 
111
 
 
112
            # have to do this later so we don't mess up the iterator.
 
113
            # since parents may be removed before their children we
 
114
            # have to test.
 
115
 
 
116
            # FIXME: There's probably a better way to do this; perhaps
 
117
            # the workingtree should know how to filter itbranch.
 
118
            if work_inv.has_id(file_id):
 
119
                del work_inv[file_id]
 
120
 
 
121
        if rev_id is None:
 
122
            rev_id = _gen_revision_id(branch, time.time())
 
123
        inv_id = rev_id
 
124
 
 
125
        inv_tmp = tempfile.TemporaryFile()
 
126
        pack_xml(new_inv, inv_tmp)
 
127
        inv_tmp.seek(0)
 
128
        branch.inventory_store.add(inv_tmp, inv_id)
 
129
        mutter('new inventory_id is {%s}' % inv_id)
 
130
 
 
131
        # We could also just sha hash the inv_tmp file
 
132
        # however, in the case that branch.inventory_store.add()
 
133
        # ever actually does anything special
 
134
        inv_sha1 = branch.get_inventory_sha1(inv_id)
 
135
 
 
136
        branch._write_inventory(work_inv)
 
137
 
 
138
        if timestamp == None:
 
139
            timestamp = time.time()
 
140
 
 
141
        if committer == None:
 
142
            committer = username(branch)
 
143
 
 
144
        if timezone == None:
 
145
            timezone = local_time_offset()
 
146
 
 
147
        mutter("building commit log message")
 
148
        rev = Revision(timestamp=timestamp,
 
149
                       timezone=timezone,
 
150
                       committer=committer,
 
151
                       message = message,
 
152
                       inventory_id=inv_id,
 
153
                       inventory_sha1=inv_sha1,
 
154
                       revision_id=rev_id)
 
155
 
 
156
        rev.parents = []
 
157
        precursor_id = branch.last_patch()
 
158
        if precursor_id:
 
159
            precursor_sha1 = branch.get_revision_sha1(precursor_id)
 
160
            rev.parents.append(RevisionReference(precursor_id, precursor_sha1))
 
161
        for merge_rev in pending_merges:
 
162
            rev.parents.append(RevisionReference(merge_rev))            
 
163
 
 
164
        rev_tmp = tempfile.TemporaryFile()
 
165
        pack_xml(rev, rev_tmp)
 
166
        rev_tmp.seek(0)
 
167
        branch.revision_store.add(rev_tmp, rev_id)
 
168
        mutter("new revision_id is {%s}" % rev_id)
 
169
 
 
170
        ## XXX: Everything up to here can simply be orphaned if we abort
 
171
        ## the commit; it will leave junk files behind but that doesn't
 
172
        ## matter.
 
173
 
 
174
        ## TODO: Read back the just-generated changeset, and make sure it
 
175
        ## applies and recreates the right state.
 
176
 
 
177
        ## TODO: Also calculate and store the inventory SHA1
 
178
        mutter("committing patch r%d" % (branch.revno() + 1))
 
179
 
 
180
        branch.append_revision(rev_id)
 
181
 
 
182
        branch.set_pending_merges([])
 
183
 
 
184
        if verbose:
 
185
            # disabled; should go through logging
 
186
            # note("commited r%d" % branch.revno())
 
187
            # print ("commited r%d" % branch.revno())
 
188
            pass
 
189
    finally:
 
190
        branch.unlock()
 
191
 
 
192
 
 
193
 
 
194
def _gen_revision_id(branch, when):
 
195
    """Return new revision-id."""
 
196
    from binascii import hexlify
 
197
    from bzrlib.osutils import rand_bytes, compact_date, user_email
 
198
 
 
199
    s = '%s-%s-' % (user_email(branch), compact_date(when))
 
200
    s += hexlify(rand_bytes(8))
 
201
    return s
 
202
 
 
203
 
 
204
def _gather_commit(branch, work_tree, work_inv, basis_inv, specific_files,
 
205
                   verbose):
 
206
    """Build inventory preparatory to commit.
 
207
 
 
208
    Returns missing_ids, new_inv, any_changes.
 
209
 
 
210
    This adds any changed files into the text store, and sets their
 
211
    test-id, sha and size in the returned inventory appropriately.
 
212
 
 
213
    missing_ids
 
214
        Modified to hold a list of files that have been deleted from
 
215
        the working directory; these should be removed from the
 
216
        working inventory.
 
217
    """
 
218
    from bzrlib.inventory import Inventory
 
219
    from bzrlib.osutils import isdir, isfile, sha_string, quotefn, \
 
220
         local_time_offset, username, kind_marker, is_inside_any
 
221
    
 
222
    from bzrlib.branch import gen_file_id
 
223
    from bzrlib.errors import BzrError
 
224
    from bzrlib.revision import Revision
 
225
    from bzrlib.trace import mutter, note
 
226
 
 
227
    any_changes = False
 
228
    inv = Inventory(work_inv.root.file_id)
 
229
    missing_ids = []
 
230
    
 
231
    for path, entry in work_inv.iter_entries():
 
232
        ## TODO: Check that the file kind has not changed from the previous
 
233
        ## revision of this file (if any).
 
234
 
 
235
        p = branch.abspath(path)
 
236
        file_id = entry.file_id
 
237
        mutter('commit prep file %s, id %r ' % (p, file_id))
 
238
 
 
239
        if specific_files and not is_inside_any(specific_files, path):
 
240
            mutter('  skipping file excluded from commit')
 
241
            if basis_inv.has_id(file_id):
 
242
                # carry over with previous state
 
243
                inv.add(basis_inv[file_id].copy())
 
244
            else:
 
245
                # omit this from committed inventory
 
246
                pass
 
247
            continue
 
248
 
 
249
        if not work_tree.has_id(file_id):
 
250
            if verbose:
 
251
                print('deleted %s%s' % (path, kind_marker(entry.kind)))
 
252
                any_changes = True
 
253
            mutter("    file is missing, removing from inventory")
 
254
            missing_ids.append(file_id)
 
255
            continue
 
256
 
 
257
        # this is present in the new inventory; may be new, modified or
 
258
        # unchanged.
 
259
        old_ie = basis_inv.has_id(file_id) and basis_inv[file_id]
 
260
        
 
261
        entry = entry.copy()
 
262
        inv.add(entry)
 
263
 
 
264
        if old_ie:
 
265
            old_kind = old_ie.kind
 
266
            if old_kind != entry.kind:
 
267
                raise BzrError("entry %r changed kind from %r to %r"
 
268
                        % (file_id, old_kind, entry.kind))
 
269
 
 
270
        if entry.kind == 'directory':
 
271
            if not isdir(p):
 
272
                raise BzrError("%s is entered as directory but not a directory"
 
273
                               % quotefn(p))
 
274
        elif entry.kind == 'file':
 
275
            if not isfile(p):
 
276
                raise BzrError("%s is entered as file but is not a file" % quotefn(p))
 
277
 
 
278
            new_sha1 = work_tree.get_file_sha1(file_id)
 
279
 
 
280
            if (old_ie
 
281
                and old_ie.text_sha1 == new_sha1):
 
282
                ## assert content == basis.get_file(file_id).read()
 
283
                entry.text_id = old_ie.text_id
 
284
                entry.text_sha1 = new_sha1
 
285
                entry.text_size = old_ie.text_size
 
286
                mutter('    unchanged from previous text_id {%s}' %
 
287
                       entry.text_id)
 
288
            else:
 
289
                content = file(p, 'rb').read()
 
290
 
 
291
                # calculate the sha again, just in case the file contents
 
292
                # changed since we updated the cache
 
293
                entry.text_sha1 = sha_string(content)
 
294
                entry.text_size = len(content)
 
295
 
 
296
                entry.text_id = gen_file_id(entry.name)
 
297
                branch.text_store.add(content, entry.text_id)
 
298
                mutter('    stored with text_id {%s}' % entry.text_id)
 
299
 
 
300
        if verbose:
 
301
            marked = path + kind_marker(entry.kind)
 
302
            if not old_ie:
 
303
                print 'added', marked
 
304
                any_changes = True
 
305
            elif old_ie == entry:
 
306
                pass                    # unchanged
 
307
            elif (old_ie.name == entry.name
 
308
                  and old_ie.parent_id == entry.parent_id):
 
309
                print 'modified', marked
 
310
                any_changes = True
 
311
            else:
 
312
                print 'renamed', marked
 
313
                any_changes = True
 
314
                        
 
315
    return missing_ids, inv, any_changes
 
316
 
908
317