~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/mutabletree.py

  • Committer: Andrew Bennetts
  • Date: 2007-10-08 02:22:17 UTC
  • mto: This revision was merged to the branch mainline in revision 2900.
  • Revision ID: andrew.bennetts@canonical.com-20071008022217-8gxkimbur9vzgbav
Fix bug 146715: bzr+ssh:// and sftp:// should not assume port-not-specified means port 22

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2006, 2007 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., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
"""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
27
26
 
28
27
from bzrlib import (
29
28
    add,
30
29
    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 (
31
37
    errors,
32
 
    hooks,
33
38
    osutils,
34
 
    revisiontree,
35
 
    symbol_versioning,
36
 
    trace,
37
39
    tree,
38
40
    )
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
42
44
 
43
45
 
44
46
def needs_tree_write_lock(unbound):
66
68
    entirely in memory.
67
69
 
68
70
    For now, we are not treating MutableTree as an interface to provide
69
 
    conformance tests for - rather we are testing MemoryTree specifically, and
 
71
    conformance tests for - rather we are testing MemoryTree specifically, and 
70
72
    interface testing implementations of WorkingTree.
71
73
 
72
74
    A mutable tree always has an associated Branch and BzrDir object - the
73
75
    branch and bzrdir attributes.
74
76
    """
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
81
77
 
82
78
    @needs_tree_write_lock
83
79
    def add(self, files, ids=None, kinds=None):
99
95
        TODO: Perhaps callback with the ids and paths as they're added.
100
96
        """
101
97
        if isinstance(files, 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()
 
98
            assert(ids is None or isinstance(ids, basestring))
 
99
            assert(kinds is None or isinstance(kinds, basestring))
108
100
            files = [files]
109
101
            if ids is not None:
110
102
                ids = [ids]
116
108
        if ids is None:
117
109
            ids = [None] * len(files)
118
110
        else:
119
 
            if not (len(ids) == len(files)):
120
 
                raise AssertionError()
 
111
            assert(len(ids) == len(files))
 
112
            ids = [osutils.safe_file_id(file_id) for file_id in ids]
 
113
 
121
114
        if kinds is None:
122
115
            kinds = [None] * len(files)
123
 
        elif not len(kinds) == len(files):
124
 
            raise AssertionError()
 
116
        else:
 
117
            assert(len(kinds) == len(files))
125
118
        for f in files:
126
119
            # generic constraint checks:
127
120
            if self.is_control_filename(f):
128
121
                raise errors.ForbiddenControlFileError(filename=f)
129
 
            fp = osutils.splitpath(f)
130
 
        # fill out file kinds for all files [not needed when we stop
 
122
            fp = splitpath(f)
 
123
        # fill out file kinds for all files [not needed when we stop 
131
124
        # caring about the instantaneous file kind within a uncommmitted tree
132
125
        #
133
126
        self._gather_kinds(files, kinds)
184
177
        from bzrlib import commit
185
178
        if revprops is None:
186
179
            revprops = {}
187
 
        possible_master_transports=[]
188
180
        if not 'branch-nick' in revprops:
189
 
            revprops['branch-nick'] = self.branch._get_nick(
190
 
                kwargs.get('local', False),
191
 
                possible_master_transports)
192
 
        authors = kwargs.pop('authors', None)
 
181
            revprops['branch-nick'] = self.branch.nick
193
182
        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)
207
183
        if author is not None:
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
            assert 'author' not in revprops
 
185
            revprops['author'] = author
218
186
        # args for wt.commit start at message from the Commit.commit method,
219
187
        args = (message, ) + args
220
 
        for hook in MutableTree.hooks['start_commit']:
221
 
            hook(self)
222
188
        committed_id = commit.Commit().commit(working_tree=self,
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)
 
189
            revprops=revprops, *args, **kwargs)
229
190
        return committed_id
230
191
 
231
192
    def _gather_kinds(self, files, kinds):
233
194
        raise NotImplementedError(self._gather_kinds)
234
195
 
235
196
    @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
299
197
    def last_revision(self):
300
198
        """Return the revision id of the last commit performed in this tree.
301
199
 
302
200
        In early tree formats the result of last_revision is the same as the
303
201
        branch last_revision, but that is no longer the case for modern tree
304
202
        formats.
305
 
 
 
203
        
306
204
        last_revision returns the left most parent id, or None if there are no
307
205
        parents.
308
206
 
325
223
    def lock_write(self):
326
224
        """Lock the tree and its branch. This allows mutating calls to be made.
327
225
 
328
 
        Some mutating methods will take out implicit write locks, but in
 
226
        Some mutating methods will take out implicit write locks, but in 
329
227
        general you should always obtain a write lock before calling mutating
330
228
        methods on a tree.
331
229
        """
341
239
        """
342
240
        raise NotImplementedError(self.mkdir)
343
241
 
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
 
 
380
242
    def set_parent_ids(self, revision_ids, allow_leftmost_as_ghost=False):
381
243
        """Set the parents ids of the working tree.
382
244
 
387
249
    def set_parent_trees(self, parents_list, allow_leftmost_as_ghost=False):
388
250
        """Set the parents of the working tree.
389
251
 
390
 
        :param parents_list: A list of (revision_id, tree) tuples.
 
252
        :param parents_list: A list of (revision_id, tree) tuples. 
391
253
            If tree is None, then that element is treated as an unreachable
392
254
            parent tree - i.e. a ghost.
393
255
        """
401
263
        For the specific behaviour see the help for cmd_add().
402
264
 
403
265
        :param action: A reporter to be called with the inventory, parent_ie,
404
 
            path and kind of the path being added. It may return a file_id if
 
266
            path and kind of the path being added. It may return a file_id if 
405
267
            a specific one should be used.
406
268
        :param save: Save the inventory after completing the adds. If False
407
269
            this provides dry-run functionality by doing the add and not saving
413
275
        # not in an inner loop; and we want to remove direct use of this,
414
276
        # so here as a reminder for now. RBC 20070703
415
277
        from bzrlib.inventory import InventoryEntry
 
278
        assert isinstance(recurse, bool)
416
279
        if action is None:
417
280
            action = add.AddAction()
418
 
 
 
281
        
419
282
        if not file_list:
420
283
            # no paths supplied: add the entire tree.
421
 
            # FIXME: this assumes we are running in a working tree subdir :-/
422
 
            # -- vila 20100208
423
284
            file_list = [u'.']
424
285
        # mutter("smart add of %r")
425
286
        inv = self.inventory
427
288
        ignored = {}
428
289
        dirs_to_add = []
429
290
        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())
438
291
 
439
 
        # validate user file paths and convert all paths to tree
 
292
        # validate user file paths and convert all paths to tree 
440
293
        # relative : it's cheaper to make a tree relative path an abspath
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
 
294
        # than to convert an abspath to tree relative.
 
295
        for filepath in file_list:
 
296
            rf = _FastPath(self.relpath(filepath))
 
297
            # validate user parameters. Our recursive code avoids adding new files
 
298
            # that need such validation 
447
299
            if self.is_control_filename(rf.raw_path):
448
300
                raise errors.ForbiddenControlFileError(filename=rf.raw_path)
449
 
 
 
301
            
450
302
            abspath = self.abspath(rf.raw_path)
451
303
            kind = osutils.file_kind(abspath)
452
304
            if kind == 'directory':
455
307
            else:
456
308
                if not InventoryEntry.versionable_kind(kind):
457
309
                    raise errors.BadFileKindError(filename=abspath, kind=kind)
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
 
310
            # ensure the named path is added, so that ignore rules in the later directory
 
311
            # walk dont skip it.
 
312
            # we dont have a parent ie known yet.: use the relatively slower inventory 
 
313
            # probing method
462
314
            versioned = inv.has_filename(rf.raw_path)
463
315
            if versioned:
464
316
                continue
480
332
                dirs_to_add.append((path, None))
481
333
            prev_dir = path.raw_path
482
334
 
483
 
        illegalpath_re = re.compile(r'[\r\n]')
484
335
        # dirs_to_add is initialised to a list of directories, but as we scan
485
336
        # directories we append files to it.
486
337
        # XXX: We should determine kind of files when we scan them rather than
495
346
            kind = osutils.file_kind(abspath)
496
347
 
497
348
            if not InventoryEntry.versionable_kind(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)
 
349
                warning("skipping %s (can't add file of kind '%s')", abspath, kind)
510
350
                continue
511
351
 
512
352
            if parent_ie is not None:
513
353
                versioned = directory.base_path in parent_ie.children
514
354
            else:
515
 
                # without the parent ie, use the relatively slower inventory
 
355
                # without the parent ie, use the relatively slower inventory 
516
356
                # probing method
517
 
                versioned = inv.has_filename(
518
 
                        self._fix_case_of_inventory_path(directory.raw_path))
 
357
                versioned = inv.has_filename(directory.raw_path)
519
358
 
520
359
            if kind == 'directory':
521
360
                try:
535
374
                pass
536
375
                # mutter("%r is already versioned", abspath)
537
376
            elif sub_tree:
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)
 
377
                # XXX: This is wrong; people *might* reasonably be trying to add
 
378
                # subtrees as subtrees.  This should probably only be done in formats 
 
379
                # which can represent subtrees, and even then perhaps only when
 
380
                # the user asked to add subtrees.  At the moment you can add them
 
381
                # specially through 'join --reference', which is perhaps
 
382
                # reasonable: adding a new reference is a special operation and
 
383
                # can have a special behaviour.  mbp 20070306
 
384
                mutter("%r is a nested bzr tree", abspath)
547
385
            else:
548
386
                _add_one(self, inv, parent_ie, directory, kind, action)
549
387
                added.append(directory.raw_path)
553
391
                    # must be present:
554
392
                    this_ie = parent_ie.children[directory.base_path]
555
393
                else:
556
 
                    # without the parent ie, use the relatively slower inventory
 
394
                    # without the parent ie, use the relatively slower inventory 
557
395
                    # probing method
558
 
                    this_id = inv.path2id(
559
 
                        self._fix_case_of_inventory_path(directory.raw_path))
 
396
                    this_id = inv.path2id(directory.raw_path)
560
397
                    if this_id is None:
561
398
                        this_ie = None
562
399
                    else:
563
400
                        this_ie = inv[this_id]
564
401
 
565
402
                for subf in sorted(os.listdir(abspath)):
566
 
                    # here we could use TreeDirectory rather than
 
403
                    # here we could use TreeDirectory rather than 
567
404
                    # string concatenation.
568
405
                    subp = osutils.pathjoin(directory.raw_path, subf)
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
 
406
                    # TODO: is_control_filename is very slow. Make it faster. 
 
407
                    # TreeDirectory.is_control_filename could also make this 
 
408
                    # faster - its impossible for a non root dir to have a 
572
409
                    # control file.
573
410
                    if self.is_control_filename(subp):
574
 
                        trace.mutter("skip control directory %r", subp)
 
411
                        mutter("skip control directory %r", subp)
575
412
                    elif subf in this_ie.children:
576
413
                        # recurse into this already versioned subdir.
577
414
                        dirs_to_add.append((_FastPath(subp, subf), this_ie))
594
431
                self.read_working_inventory()
595
432
        return added, ignored
596
433
 
597
 
    def update_basis_by_delta(self, new_revid, delta):
 
434
    def update_to_one_parent_via_delta(self, new_revid, delta):
598
435
        """Update the parents of this tree after a commit.
599
436
 
600
437
        This gives the tree one parent, with revision id new_revid. The
601
 
        inventory delta is applied to the current basis tree to generate the
 
438
        inventory delta is applied ot the current basis tree to generate the
602
439
        inventory for the parent new_revid, and all other parent trees are
603
440
        discarded.
604
441
 
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
 
 
615
442
        :param new_revid: The new revision id for the trees parent.
616
443
        :param delta: An inventory delta (see apply_inventory_delta) describing
617
444
            the changes from the current left most parent revision to new_revid.
627
454
        # WorkingTree classes for optimised versions for specific format trees.
628
455
        basis = self.basis_tree()
629
456
        basis.lock_read()
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()
 
457
        inventory = basis.inventory
634
458
        basis.unlock()
635
459
        inventory.apply_delta(delta)
636
 
        rev_tree = revisiontree.RevisionTree(self.branch.repository,
637
 
                                             inventory, new_revid)
 
460
        rev_tree = RevisionTree(self.branch.repository, inventory, new_revid)
638
461
        self.set_parent_trees([(new_revid, rev_tree)])
639
462
 
640
463
 
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
 
 
680
464
class _FastPath(object):
681
465
    """A path object with fast accessors for things like basename."""
682
466
 
716
500
        added = []
717
501
    else:
718
502
        # slower but does not need parent_ie
719
 
        if inv.has_filename(tree._fix_case_of_inventory_path(path.raw_path)):
 
503
        if inv.has_filename(path.raw_path):
720
504
            return []
721
505
        # its really not there : add the parent
722
506
        # note that the dirname use leads to some extra str copying etc but as
723
507
        # there are a limited number of dirs we can be nested under, it should
724
508
        # generally find it very fast and not recurse after that.
725
509
        added = _add_one_and_parent(tree, inv, None,
726
 
            _FastPath(osutils.dirname(path.raw_path)), 'directory', action)
727
 
        parent_id = inv.path2id(osutils.dirname(path.raw_path))
 
510
            _FastPath(dirname(path.raw_path)), 'directory', action)
 
511
        parent_id = inv.path2id(dirname(path.raw_path))
728
512
        parent_ie = inv[parent_id]
729
513
    _add_one(tree, inv, parent_ie, path, kind, action)
730
514
    return added + [path.raw_path]