~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/mutabletree.py

  • Committer: Robert Collins
  • Date: 2010-05-06 23:41:35 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506234135-yivbzczw1sejxnxc
Lock methods on ``Tree``, ``Branch`` and ``Repository`` are now
expected to return an object which can be used to unlock them. This reduces
duplicate code when using cleanups. The previous 'tokens's returned by
``Branch.lock_write`` and ``Repository.lock_write`` are now attributes
on the result of the lock_write. ``repository.RepositoryWriteLockResult``
and ``branch.BranchWriteLockResult`` document this. (Robert Collins)

``log._get_info_for_log_files`` now takes an add_cleanup callable.
(Robert Collins)

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006, 2007 Canonical Ltd
 
1
# Copyright (C) 2006-2010 Canonical Ltd
2
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
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
17
"""MutableTree object.
18
18
 
23
23
from bzrlib.lazy_import import lazy_import
24
24
lazy_import(globals(), """
25
25
import os
 
26
import re
26
27
 
27
28
from bzrlib import (
28
29
    add,
29
30
    bzrdir,
30
 
    )
31
 
from bzrlib.osutils import dirname
32
 
from bzrlib.revisiontree import RevisionTree
33
 
from bzrlib.trace import mutter, warning
34
 
""")
35
 
 
36
 
from bzrlib import (
37
31
    errors,
 
32
    hooks,
38
33
    osutils,
 
34
    revisiontree,
 
35
    symbol_versioning,
 
36
    trace,
39
37
    tree,
40
38
    )
 
39
""")
 
40
 
41
41
from bzrlib.decorators import needs_read_lock, needs_write_lock
42
 
from bzrlib.osutils import splitpath
43
 
from bzrlib.symbol_versioning import DEPRECATED_PARAMETER
44
42
 
45
43
 
46
44
def needs_tree_write_lock(unbound):
68
66
    entirely in memory.
69
67
 
70
68
    For now, we are not treating MutableTree as an interface to provide
71
 
    conformance tests for - rather we are testing MemoryTree specifically, and 
 
69
    conformance tests for - rather we are testing MemoryTree specifically, and
72
70
    interface testing implementations of WorkingTree.
73
71
 
74
72
    A mutable tree always has an associated Branch and BzrDir object - the
75
73
    branch and bzrdir attributes.
76
74
    """
 
75
    def __init__(self, *args, **kw):
 
76
        super(MutableTree, self).__init__(*args, **kw)
 
77
        # Is this tree on a case-insensitive or case-preserving file-system?
 
78
        # Sub-classes may initialize to False if they detect they are being
 
79
        # used on media which doesn't differentiate the case of names.
 
80
        self.case_sensitive = True
77
81
 
78
82
    @needs_tree_write_lock
79
83
    def add(self, files, ids=None, kinds=None):
95
99
        TODO: Perhaps callback with the ids and paths as they're added.
96
100
        """
97
101
        if isinstance(files, basestring):
98
 
            assert(ids is None or isinstance(ids, basestring))
99
 
            assert(kinds is None or isinstance(kinds, basestring))
 
102
            # XXX: Passing a single string is inconsistent and should be
 
103
            # deprecated.
 
104
            if not (ids is None or isinstance(ids, basestring)):
 
105
                raise AssertionError()
 
106
            if not (kinds is None or isinstance(kinds, basestring)):
 
107
                raise AssertionError()
100
108
            files = [files]
101
109
            if ids is not None:
102
110
                ids = [ids]
108
116
        if ids is None:
109
117
            ids = [None] * len(files)
110
118
        else:
111
 
            assert(len(ids) == len(files))
 
119
            if not (len(ids) == len(files)):
 
120
                raise AssertionError()
112
121
        if kinds is None:
113
122
            kinds = [None] * len(files)
114
 
        else:
115
 
            assert(len(kinds) == len(files))
 
123
        elif not len(kinds) == len(files):
 
124
            raise AssertionError()
116
125
        for f in files:
117
126
            # generic constraint checks:
118
127
            if self.is_control_filename(f):
119
128
                raise errors.ForbiddenControlFileError(filename=f)
120
 
            fp = splitpath(f)
121
 
        # fill out file kinds for all files [not needed when we stop 
 
129
            fp = osutils.splitpath(f)
 
130
        # fill out file kinds for all files [not needed when we stop
122
131
        # caring about the instantaneous file kind within a uncommmitted tree
123
132
        #
124
133
        self._gather_kinds(files, kinds)
175
184
        from bzrlib import commit
176
185
        if revprops is None:
177
186
            revprops = {}
 
187
        possible_master_transports=[]
178
188
        if not 'branch-nick' in revprops:
179
 
            revprops['branch-nick'] = self.branch.nick
 
189
            revprops['branch-nick'] = self.branch._get_nick(
 
190
                kwargs.get('local', False),
 
191
                possible_master_transports)
 
192
        authors = kwargs.pop('authors', None)
180
193
        author = kwargs.pop('author', None)
 
194
        if authors is not None:
 
195
            if author is not None:
 
196
                raise AssertionError('Specifying both author and authors '
 
197
                        'is not allowed. Specify just authors instead')
 
198
            if 'author' in revprops or 'authors' in revprops:
 
199
                # XXX: maybe we should just accept one of them?
 
200
                raise AssertionError('author property given twice')
 
201
            if authors:
 
202
                for individual in authors:
 
203
                    if '\n' in individual:
 
204
                        raise AssertionError('\\n is not a valid character '
 
205
                                'in an author identity')
 
206
                revprops['authors'] = '\n'.join(authors)
181
207
        if author is not None:
182
 
            assert 'author' not in revprops
183
 
            revprops['author'] = author
 
208
            symbol_versioning.warn('The parameter author was deprecated'
 
209
                   ' in version 1.13. Use authors instead',
 
210
                   DeprecationWarning)
 
211
            if 'author' in revprops or 'authors' in revprops:
 
212
                # XXX: maybe we should just accept one of them?
 
213
                raise AssertionError('author property given twice')
 
214
            if '\n' in author:
 
215
                raise AssertionError('\\n is not a valid character '
 
216
                        'in an author identity')
 
217
            revprops['authors'] = author
184
218
        # args for wt.commit start at message from the Commit.commit method,
185
219
        args = (message, ) + args
 
220
        for hook in MutableTree.hooks['start_commit']:
 
221
            hook(self)
186
222
        committed_id = commit.Commit().commit(working_tree=self,
187
 
            revprops=revprops, *args, **kwargs)
 
223
            revprops=revprops,
 
224
            possible_master_transports=possible_master_transports,
 
225
            *args, **kwargs)
 
226
        post_hook_params = PostCommitHookParams(self)
 
227
        for hook in MutableTree.hooks['post_commit']:
 
228
            hook(post_hook_params)
188
229
        return committed_id
189
230
 
190
231
    def _gather_kinds(self, files, kinds):
192
233
        raise NotImplementedError(self._gather_kinds)
193
234
 
194
235
    @needs_read_lock
 
236
    def has_changes(self, _from_tree=None):
 
237
        """Quickly check that the tree contains at least one commitable change.
 
238
 
 
239
        :param _from_tree: tree to compare against to find changes (default to
 
240
            the basis tree and is intended to be used by tests).
 
241
 
 
242
        :return: True if a change is found. False otherwise
 
243
        """
 
244
        # Check pending merges
 
245
        if len(self.get_parent_ids()) > 1:
 
246
            return True
 
247
        if _from_tree is None:
 
248
            _from_tree = self.basis_tree()
 
249
        changes = self.iter_changes(_from_tree)
 
250
        try:
 
251
            change = changes.next()
 
252
            # Exclude root (talk about black magic... --vila 20090629)
 
253
            if change[4] == (None, None):
 
254
                change = changes.next()
 
255
            return True
 
256
        except StopIteration:
 
257
            # No changes
 
258
            return False
 
259
 
 
260
    @needs_read_lock
 
261
    def check_changed_or_out_of_date(self, strict, opt_name,
 
262
                                     more_error, more_warning):
 
263
        """Check the tree for uncommitted changes and branch synchronization.
 
264
 
 
265
        If strict is None and not set in the config files, a warning is issued.
 
266
        If strict is True, an error is raised.
 
267
        If strict is False, no checks are done and no warning is issued.
 
268
 
 
269
        :param strict: True, False or None, searched in branch config if None.
 
270
 
 
271
        :param opt_name: strict option name to search in config file.
 
272
 
 
273
        :param more_error: Details about how to avoid the check.
 
274
 
 
275
        :param more_warning: Details about what is happening.
 
276
        """
 
277
        if strict is None:
 
278
            strict = self.branch.get_config().get_user_option_as_bool(opt_name)
 
279
        if strict is not False:
 
280
            err_class = None
 
281
            if (self.has_changes()):
 
282
                err_class = errors.UncommittedChanges
 
283
            elif self.last_revision() != self.branch.last_revision():
 
284
                # The tree has lost sync with its branch, there is little
 
285
                # chance that the user is aware of it but he can still force
 
286
                # the action with --no-strict
 
287
                err_class = errors.OutOfDateTree
 
288
            if err_class is not None:
 
289
                if strict is None:
 
290
                    err = err_class(self, more=more_warning)
 
291
                    # We don't want to interrupt the user if he expressed no
 
292
                    # preference about strict.
 
293
                    trace.warning('%s', err._format())
 
294
                else:
 
295
                    err = err_class(self, more=more_error)
 
296
                    raise err
 
297
 
 
298
    @needs_read_lock
195
299
    def last_revision(self):
196
300
        """Return the revision id of the last commit performed in this tree.
197
301
 
198
302
        In early tree formats the result of last_revision is the same as the
199
303
        branch last_revision, but that is no longer the case for modern tree
200
304
        formats.
201
 
        
 
305
 
202
306
        last_revision returns the left most parent id, or None if there are no
203
307
        parents.
204
308
 
221
325
    def lock_write(self):
222
326
        """Lock the tree and its branch. This allows mutating calls to be made.
223
327
 
224
 
        Some mutating methods will take out implicit write locks, but in 
 
328
        Some mutating methods will take out implicit write locks, but in
225
329
        general you should always obtain a write lock before calling mutating
226
330
        methods on a tree.
227
331
        """
237
341
        """
238
342
        raise NotImplementedError(self.mkdir)
239
343
 
 
344
    def _observed_sha1(self, file_id, path, (sha1, stat_value)):
 
345
        """Tell the tree we have observed a paths sha1.
 
346
 
 
347
        The intent of this function is to allow trees that have a hashcache to
 
348
        update the hashcache during commit. If the observed file is too new
 
349
        (based on the stat_value) to be safely hash-cached the tree will ignore
 
350
        it.
 
351
 
 
352
        The default implementation does nothing.
 
353
 
 
354
        :param file_id: The file id
 
355
        :param path: The file path
 
356
        :param sha1: The sha 1 that was observed.
 
357
        :param stat_value: A stat result for the file the sha1 was read from.
 
358
        :return: None
 
359
        """
 
360
 
 
361
    def _fix_case_of_inventory_path(self, path):
 
362
        """If our tree isn't case sensitive, return the canonical path"""
 
363
        if not self.case_sensitive:
 
364
            path = self.get_canonical_inventory_path(path)
 
365
        return path
 
366
 
 
367
    @needs_write_lock
 
368
    def put_file_bytes_non_atomic(self, file_id, bytes):
 
369
        """Update the content of a file in the tree.
 
370
 
 
371
        Note that the file is written in-place rather than being
 
372
        written to a temporary location and renamed. As a consequence,
 
373
        readers can potentially see the file half-written.
 
374
 
 
375
        :param file_id: file-id of the file
 
376
        :param bytes: the new file contents
 
377
        """
 
378
        raise NotImplementedError(self.put_file_bytes_non_atomic)
 
379
 
240
380
    def set_parent_ids(self, revision_ids, allow_leftmost_as_ghost=False):
241
381
        """Set the parents ids of the working tree.
242
382
 
247
387
    def set_parent_trees(self, parents_list, allow_leftmost_as_ghost=False):
248
388
        """Set the parents of the working tree.
249
389
 
250
 
        :param parents_list: A list of (revision_id, tree) tuples. 
 
390
        :param parents_list: A list of (revision_id, tree) tuples.
251
391
            If tree is None, then that element is treated as an unreachable
252
392
            parent tree - i.e. a ghost.
253
393
        """
261
401
        For the specific behaviour see the help for cmd_add().
262
402
 
263
403
        :param action: A reporter to be called with the inventory, parent_ie,
264
 
            path and kind of the path being added. It may return a file_id if 
 
404
            path and kind of the path being added. It may return a file_id if
265
405
            a specific one should be used.
266
406
        :param save: Save the inventory after completing the adds. If False
267
407
            this provides dry-run functionality by doing the add and not saving
273
413
        # not in an inner loop; and we want to remove direct use of this,
274
414
        # so here as a reminder for now. RBC 20070703
275
415
        from bzrlib.inventory import InventoryEntry
276
 
        assert isinstance(recurse, bool)
277
416
        if action is None:
278
417
            action = add.AddAction()
279
 
        
 
418
 
280
419
        if not file_list:
281
420
            # no paths supplied: add the entire tree.
 
421
            # FIXME: this assumes we are running in a working tree subdir :-/
 
422
            # -- vila 20100208
282
423
            file_list = [u'.']
283
424
        # mutter("smart add of %r")
284
425
        inv = self.inventory
286
427
        ignored = {}
287
428
        dirs_to_add = []
288
429
        user_dirs = set()
 
430
        conflicts_related = set()
 
431
        # Not all mutable trees can have conflicts
 
432
        if getattr(self, 'conflicts', None) is not None:
 
433
            # Collect all related files without checking whether they exist or
 
434
            # are versioned. It's cheaper to do that once for all conflicts
 
435
            # than trying to find the relevant conflict for each added file.
 
436
            for c in self.conflicts():
 
437
                conflicts_related.update(c.associated_filenames())
289
438
 
290
 
        # validate user file paths and convert all paths to tree 
 
439
        # validate user file paths and convert all paths to tree
291
440
        # relative : it's cheaper to make a tree relative path an abspath
292
 
        # than to convert an abspath to tree relative.
293
 
        for filepath in file_list:
294
 
            rf = _FastPath(self.relpath(filepath))
295
 
            # validate user parameters. Our recursive code avoids adding new files
296
 
            # that need such validation 
 
441
        # than to convert an abspath to tree relative, and it's cheaper to
 
442
        # perform the canonicalization in bulk.
 
443
        for filepath in osutils.canonical_relpaths(self.basedir, file_list):
 
444
            rf = _FastPath(filepath)
 
445
            # validate user parameters. Our recursive code avoids adding new
 
446
            # files that need such validation
297
447
            if self.is_control_filename(rf.raw_path):
298
448
                raise errors.ForbiddenControlFileError(filename=rf.raw_path)
299
 
            
 
449
 
300
450
            abspath = self.abspath(rf.raw_path)
301
451
            kind = osutils.file_kind(abspath)
302
452
            if kind == 'directory':
305
455
            else:
306
456
                if not InventoryEntry.versionable_kind(kind):
307
457
                    raise errors.BadFileKindError(filename=abspath, kind=kind)
308
 
            # ensure the named path is added, so that ignore rules in the later directory
309
 
            # walk dont skip it.
310
 
            # we dont have a parent ie known yet.: use the relatively slower inventory 
311
 
            # probing method
 
458
            # ensure the named path is added, so that ignore rules in the later
 
459
            # directory walk dont skip it.
 
460
            # we dont have a parent ie known yet.: use the relatively slower
 
461
            # inventory probing method
312
462
            versioned = inv.has_filename(rf.raw_path)
313
463
            if versioned:
314
464
                continue
330
480
                dirs_to_add.append((path, None))
331
481
            prev_dir = path.raw_path
332
482
 
 
483
        illegalpath_re = re.compile(r'[\r\n]')
333
484
        # dirs_to_add is initialised to a list of directories, but as we scan
334
485
        # directories we append files to it.
335
486
        # XXX: We should determine kind of files when we scan them rather than
344
495
            kind = osutils.file_kind(abspath)
345
496
 
346
497
            if not InventoryEntry.versionable_kind(kind):
347
 
                warning("skipping %s (can't add file of kind '%s')", abspath, kind)
 
498
                trace.warning("skipping %s (can't add file of kind '%s')",
 
499
                              abspath, kind)
 
500
                continue
 
501
            if illegalpath_re.search(directory.raw_path):
 
502
                trace.warning("skipping %r (contains \\n or \\r)" % abspath)
 
503
                continue
 
504
            if directory.raw_path in conflicts_related:
 
505
                # If the file looks like one generated for a conflict, don't
 
506
                # add it.
 
507
                trace.warning(
 
508
                    'skipping %s (generated to help resolve conflicts)',
 
509
                    abspath)
348
510
                continue
349
511
 
350
512
            if parent_ie is not None:
351
513
                versioned = directory.base_path in parent_ie.children
352
514
            else:
353
 
                # without the parent ie, use the relatively slower inventory 
 
515
                # without the parent ie, use the relatively slower inventory
354
516
                # probing method
355
 
                versioned = inv.has_filename(directory.raw_path)
 
517
                versioned = inv.has_filename(
 
518
                        self._fix_case_of_inventory_path(directory.raw_path))
356
519
 
357
520
            if kind == 'directory':
358
521
                try:
372
535
                pass
373
536
                # mutter("%r is already versioned", abspath)
374
537
            elif sub_tree:
375
 
                # XXX: This is wrong; people *might* reasonably be trying to add
376
 
                # subtrees as subtrees.  This should probably only be done in formats 
377
 
                # which can represent subtrees, and even then perhaps only when
378
 
                # the user asked to add subtrees.  At the moment you can add them
379
 
                # specially through 'join --reference', which is perhaps
380
 
                # reasonable: adding a new reference is a special operation and
381
 
                # can have a special behaviour.  mbp 20070306
382
 
                mutter("%r is a nested bzr tree", abspath)
 
538
                # XXX: This is wrong; people *might* reasonably be trying to
 
539
                # add subtrees as subtrees.  This should probably only be done
 
540
                # in formats which can represent subtrees, and even then
 
541
                # perhaps only when the user asked to add subtrees.  At the
 
542
                # moment you can add them specially through 'join --reference',
 
543
                # which is perhaps reasonable: adding a new reference is a
 
544
                # special operation and can have a special behaviour.  mbp
 
545
                # 20070306
 
546
                trace.mutter("%r is a nested bzr tree", abspath)
383
547
            else:
384
548
                _add_one(self, inv, parent_ie, directory, kind, action)
385
549
                added.append(directory.raw_path)
389
553
                    # must be present:
390
554
                    this_ie = parent_ie.children[directory.base_path]
391
555
                else:
392
 
                    # without the parent ie, use the relatively slower inventory 
 
556
                    # without the parent ie, use the relatively slower inventory
393
557
                    # probing method
394
 
                    this_id = inv.path2id(directory.raw_path)
 
558
                    this_id = inv.path2id(
 
559
                        self._fix_case_of_inventory_path(directory.raw_path))
395
560
                    if this_id is None:
396
561
                        this_ie = None
397
562
                    else:
398
563
                        this_ie = inv[this_id]
399
564
 
400
565
                for subf in sorted(os.listdir(abspath)):
401
 
                    # here we could use TreeDirectory rather than 
 
566
                    # here we could use TreeDirectory rather than
402
567
                    # string concatenation.
403
568
                    subp = osutils.pathjoin(directory.raw_path, subf)
404
 
                    # TODO: is_control_filename is very slow. Make it faster. 
405
 
                    # TreeDirectory.is_control_filename could also make this 
406
 
                    # faster - its impossible for a non root dir to have a 
 
569
                    # TODO: is_control_filename is very slow. Make it faster.
 
570
                    # TreeDirectory.is_control_filename could also make this
 
571
                    # faster - its impossible for a non root dir to have a
407
572
                    # control file.
408
573
                    if self.is_control_filename(subp):
409
 
                        mutter("skip control directory %r", subp)
 
574
                        trace.mutter("skip control directory %r", subp)
410
575
                    elif subf in this_ie.children:
411
576
                        # recurse into this already versioned subdir.
412
577
                        dirs_to_add.append((_FastPath(subp, subf), this_ie))
437
602
        inventory for the parent new_revid, and all other parent trees are
438
603
        discarded.
439
604
 
 
605
        All the changes in the delta should be changes synchronising the basis
 
606
        tree with some or all of the working tree, with a change to a directory
 
607
        requiring that its contents have been recursively included. That is,
 
608
        this is not a general purpose tree modification routine, but a helper
 
609
        for commit which is not required to handle situations that do not arise
 
610
        outside of commit.
 
611
 
 
612
        See the inventory developers documentation for the theory behind
 
613
        inventory deltas.
 
614
 
440
615
        :param new_revid: The new revision id for the trees parent.
441
616
        :param delta: An inventory delta (see apply_inventory_delta) describing
442
617
            the changes from the current left most parent revision to new_revid.
452
627
        # WorkingTree classes for optimised versions for specific format trees.
453
628
        basis = self.basis_tree()
454
629
        basis.lock_read()
455
 
        inventory = basis.inventory
 
630
        # TODO: Consider re-evaluating the need for this with CHKInventory
 
631
        # we don't strictly need to mutate an inventory for this
 
632
        # it only makes sense when apply_delta is cheaper than get_inventory()
 
633
        inventory = basis.inventory._get_mutable_inventory()
456
634
        basis.unlock()
457
635
        inventory.apply_delta(delta)
458
 
        rev_tree = RevisionTree(self.branch.repository, inventory, new_revid)
 
636
        rev_tree = revisiontree.RevisionTree(self.branch.repository,
 
637
                                             inventory, new_revid)
459
638
        self.set_parent_trees([(new_revid, rev_tree)])
460
639
 
461
640
 
 
641
class MutableTreeHooks(hooks.Hooks):
 
642
    """A dictionary mapping a hook name to a list of callables for mutabletree
 
643
    hooks.
 
644
    """
 
645
 
 
646
    def __init__(self):
 
647
        """Create the default hooks.
 
648
 
 
649
        """
 
650
        hooks.Hooks.__init__(self)
 
651
        self.create_hook(hooks.HookPoint('start_commit',
 
652
            "Called before a commit is performed on a tree. The start commit "
 
653
            "hook is able to change the tree before the commit takes place. "
 
654
            "start_commit is called with the bzrlib.mutabletree.MutableTree "
 
655
            "that the commit is being performed on.", (1, 4), None))
 
656
        self.create_hook(hooks.HookPoint('post_commit',
 
657
            "Called after a commit is performed on a tree. The hook is "
 
658
            "called with a bzrlib.mutabletree.PostCommitHookParams object. "
 
659
            "The mutable tree the commit was performed on is available via "
 
660
            "the mutable_tree attribute of that object.", (2, 0), None))
 
661
 
 
662
 
 
663
# install the default hooks into the MutableTree class.
 
664
MutableTree.hooks = MutableTreeHooks()
 
665
 
 
666
 
 
667
class PostCommitHookParams(object):
 
668
    """Parameters for the post_commit hook.
 
669
 
 
670
    To access the parameters, use the following attributes:
 
671
 
 
672
    * mutable_tree - the MutableTree object
 
673
    """
 
674
 
 
675
    def __init__(self, mutable_tree):
 
676
        """Create the parameters for the post_commit hook."""
 
677
        self.mutable_tree = mutable_tree
 
678
 
 
679
 
462
680
class _FastPath(object):
463
681
    """A path object with fast accessors for things like basename."""
464
682
 
498
716
        added = []
499
717
    else:
500
718
        # slower but does not need parent_ie
501
 
        if inv.has_filename(path.raw_path):
 
719
        if inv.has_filename(tree._fix_case_of_inventory_path(path.raw_path)):
502
720
            return []
503
721
        # its really not there : add the parent
504
722
        # note that the dirname use leads to some extra str copying etc but as
505
723
        # there are a limited number of dirs we can be nested under, it should
506
724
        # generally find it very fast and not recurse after that.
507
725
        added = _add_one_and_parent(tree, inv, None,
508
 
            _FastPath(dirname(path.raw_path)), 'directory', action)
509
 
        parent_id = inv.path2id(dirname(path.raw_path))
 
726
            _FastPath(osutils.dirname(path.raw_path)), 'directory', action)
 
727
        parent_id = inv.path2id(osutils.dirname(path.raw_path))
510
728
        parent_ie = inv[parent_id]
511
729
    _add_one(tree, inv, parent_ie, path, kind, action)
512
730
    return added + [path.raw_path]