~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/mutabletree.py

  • Committer: Andrew Bennetts
  • Date: 2010-07-29 11:17:57 UTC
  • mfrom: (5050.3.17 2.2)
  • mto: This revision was merged to the branch mainline in revision 5365.
  • Revision ID: andrew.bennetts@canonical.com-20100729111757-018h3pcefo7z0dnq
Merge lp:bzr/2.2 into lp:bzr.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2011 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
22
22
 
23
23
from bzrlib.lazy_import import lazy_import
24
24
lazy_import(globals(), """
25
 
import operator
26
25
import os
27
26
import re
28
27
 
29
28
from bzrlib import (
30
29
    add,
31
 
    controldir,
 
30
    bzrdir,
32
31
    errors,
33
32
    hooks,
34
 
    inventory as _mod_inventory,
35
33
    osutils,
36
34
    revisiontree,
 
35
    symbol_versioning,
37
36
    trace,
38
 
    transport as _mod_transport,
39
37
    tree,
40
38
    )
41
39
""")
71
69
    conformance tests for - rather we are testing MemoryTree specifically, and
72
70
    interface testing implementations of WorkingTree.
73
71
 
74
 
    A mutable tree always has an associated Branch and ControlDir object - the
 
72
    A mutable tree always has an associated Branch and BzrDir object - the
75
73
    branch and bzrdir attributes.
76
74
    """
77
75
    def __init__(self, *args, **kw):
81
79
        # used on media which doesn't differentiate the case of names.
82
80
        self.case_sensitive = True
83
81
 
84
 
    def is_control_filename(self, filename):
85
 
        """True if filename is the name of a control file in this tree.
86
 
 
87
 
        :param filename: A filename within the tree. This is a relative path
88
 
            from the root of this tree.
89
 
 
90
 
        This is true IF and ONLY IF the filename is part of the meta data
91
 
        that bzr controls in this tree. I.E. a random .bzr directory placed
92
 
        on disk will not be a control file for this tree.
93
 
        """
94
 
        raise NotImplementedError(self.is_control_filename)
95
 
 
96
82
    @needs_tree_write_lock
97
83
    def add(self, files, ids=None, kinds=None):
98
84
        """Add paths to the set of versioned paths.
162
148
        if sub_tree_id == self.get_root_id():
163
149
            raise errors.BadReferenceTarget(self, sub_tree,
164
150
                                     'Trees have the same root id.')
165
 
        if self.has_id(sub_tree_id):
 
151
        if sub_tree_id in self.inventory:
166
152
            raise errors.BadReferenceTarget(self, sub_tree,
167
153
                                            'Root id already present in tree')
168
154
        self._add([sub_tree_path], [sub_tree_id], ['tree-reference'])
177
163
        """
178
164
        raise NotImplementedError(self._add)
179
165
 
 
166
    @needs_tree_write_lock
180
167
    def apply_inventory_delta(self, changes):
181
168
        """Apply changes to the inventory as an atomic operation.
182
169
 
185
172
        :return None:
186
173
        :seealso Inventory.apply_delta: For details on the changes parameter.
187
174
        """
188
 
        raise NotImplementedError(self.apply_inventory_delta)
 
175
        self.flush()
 
176
        inv = self.inventory
 
177
        inv.apply_delta(changes)
 
178
        self._write_inventory(inv)
189
179
 
190
180
    @needs_write_lock
191
181
    def commit(self, message=None, revprops=None, *args,
260
250
        :param more_warning: Details about what is happening.
261
251
        """
262
252
        if strict is None:
263
 
            strict = self.branch.get_config_stack().get(opt_name)
 
253
            strict = self.branch.get_config().get_user_option_as_bool(opt_name)
264
254
        if strict is not False:
265
255
            err_class = None
266
256
            if (self.has_changes()):
343
333
        :return: None
344
334
        """
345
335
 
 
336
    def _fix_case_of_inventory_path(self, path):
 
337
        """If our tree isn't case sensitive, return the canonical path"""
 
338
        if not self.case_sensitive:
 
339
            path = self.get_canonical_inventory_path(path)
 
340
        return path
 
341
 
346
342
    @needs_write_lock
347
343
    def put_file_bytes_non_atomic(self, file_id, bytes):
348
344
        """Update the content of a file in the tree.
372
368
        """
373
369
        raise NotImplementedError(self.set_parent_trees)
374
370
 
 
371
    @needs_tree_write_lock
375
372
    def smart_add(self, file_list, recurse=True, action=None, save=True):
376
373
        """Version file_list, optionally recursing into directories.
377
374
 
392
389
            of added files, and ignored_files is a dict mapping files that were
393
390
            ignored to the rule that caused them to be ignored.
394
391
        """
395
 
        raise NotImplementedError(self.smart_add)
396
 
 
397
 
 
398
 
class MutableInventoryTree(MutableTree, tree.InventoryTree):
399
 
 
400
 
    @needs_tree_write_lock
401
 
    def apply_inventory_delta(self, changes):
402
 
        """Apply changes to the inventory as an atomic operation.
403
 
 
404
 
        :param changes: An inventory delta to apply to the working tree's
405
 
            inventory.
406
 
        :return None:
407
 
        :seealso Inventory.apply_delta: For details on the changes parameter.
408
 
        """
409
 
        self.flush()
 
392
        # not in an inner loop; and we want to remove direct use of this,
 
393
        # so here as a reminder for now. RBC 20070703
 
394
        from bzrlib.inventory import InventoryEntry
 
395
        if action is None:
 
396
            action = add.AddAction()
 
397
 
 
398
        if not file_list:
 
399
            # no paths supplied: add the entire tree.
 
400
            # FIXME: this assumes we are running in a working tree subdir :-/
 
401
            # -- vila 20100208
 
402
            file_list = [u'.']
 
403
        # mutter("smart add of %r")
410
404
        inv = self.inventory
411
 
        inv.apply_delta(changes)
412
 
        self._write_inventory(inv)
413
 
 
414
 
    def _fix_case_of_inventory_path(self, path):
415
 
        """If our tree isn't case sensitive, return the canonical path"""
416
 
        if not self.case_sensitive:
417
 
            path = self.get_canonical_inventory_path(path)
418
 
        return path
419
 
 
420
 
    @needs_tree_write_lock
421
 
    def smart_add(self, file_list, recurse=True, action=None, save=True):
422
 
        """Version file_list, optionally recursing into directories.
423
 
 
424
 
        This is designed more towards DWIM for humans than API clarity.
425
 
        For the specific behaviour see the help for cmd_add().
426
 
 
427
 
        :param file_list: List of zero or more paths.  *NB: these are 
428
 
            interpreted relative to the process cwd, not relative to the 
429
 
            tree.*  (Add and most other tree methods use tree-relative
430
 
            paths.)
431
 
        :param action: A reporter to be called with the inventory, parent_ie,
432
 
            path and kind of the path being added. It may return a file_id if
433
 
            a specific one should be used.
434
 
        :param save: Save the inventory after completing the adds. If False
435
 
            this provides dry-run functionality by doing the add and not saving
436
 
            the inventory.
437
 
        :return: A tuple - files_added, ignored_files. files_added is the count
438
 
            of added files, and ignored_files is a dict mapping files that were
439
 
            ignored to the rule that caused them to be ignored.
440
 
        """
 
405
        added = []
 
406
        ignored = {}
 
407
        dirs_to_add = []
 
408
        user_dirs = set()
 
409
        conflicts_related = set()
441
410
        # Not all mutable trees can have conflicts
442
411
        if getattr(self, 'conflicts', None) is not None:
443
412
            # Collect all related files without checking whether they exist or
444
413
            # are versioned. It's cheaper to do that once for all conflicts
445
414
            # than trying to find the relevant conflict for each added file.
446
 
            conflicts_related = set()
447
415
            for c in self.conflicts():
448
416
                conflicts_related.update(c.associated_filenames())
449
 
        else:
450
 
            conflicts_related = None
451
 
        adder = _SmartAddHelper(self, action, conflicts_related)
452
 
        adder.add(file_list, recurse=recurse)
453
 
        if save:
454
 
            invdelta = adder.get_inventory_delta()
455
 
            self.apply_inventory_delta(invdelta)
456
 
        return adder.added, adder.ignored
 
417
 
 
418
        # validate user file paths and convert all paths to tree
 
419
        # relative : it's cheaper to make a tree relative path an abspath
 
420
        # than to convert an abspath to tree relative, and it's cheaper to
 
421
        # perform the canonicalization in bulk.
 
422
        for filepath in osutils.canonical_relpaths(self.basedir, file_list):
 
423
            rf = _FastPath(filepath)
 
424
            # validate user parameters. Our recursive code avoids adding new
 
425
            # files that need such validation
 
426
            if self.is_control_filename(rf.raw_path):
 
427
                raise errors.ForbiddenControlFileError(filename=rf.raw_path)
 
428
 
 
429
            abspath = self.abspath(rf.raw_path)
 
430
            kind = osutils.file_kind(abspath)
 
431
            if kind == 'directory':
 
432
                # schedule the dir for scanning
 
433
                user_dirs.add(rf)
 
434
            else:
 
435
                if not InventoryEntry.versionable_kind(kind):
 
436
                    raise errors.BadFileKindError(filename=abspath, kind=kind)
 
437
            # ensure the named path is added, so that ignore rules in the later
 
438
            # directory walk dont skip it.
 
439
            # we dont have a parent ie known yet.: use the relatively slower
 
440
            # inventory probing method
 
441
            versioned = inv.has_filename(rf.raw_path)
 
442
            if versioned:
 
443
                continue
 
444
            added.extend(_add_one_and_parent(self, inv, None, rf, kind, action))
 
445
 
 
446
        if not recurse:
 
447
            # no need to walk any directories at all.
 
448
            if len(added) > 0 and save:
 
449
                self._write_inventory(inv)
 
450
            return added, ignored
 
451
 
 
452
        # only walk the minimal parents needed: we have user_dirs to override
 
453
        # ignores.
 
454
        prev_dir = None
 
455
 
 
456
        is_inside = osutils.is_inside_or_parent_of_any
 
457
        for path in sorted(user_dirs):
 
458
            if (prev_dir is None or not is_inside([prev_dir], path.raw_path)):
 
459
                dirs_to_add.append((path, None))
 
460
            prev_dir = path.raw_path
 
461
 
 
462
        illegalpath_re = re.compile(r'[\r\n]')
 
463
        # dirs_to_add is initialised to a list of directories, but as we scan
 
464
        # directories we append files to it.
 
465
        # XXX: We should determine kind of files when we scan them rather than
 
466
        # adding to this list. RBC 20070703
 
467
        for directory, parent_ie in dirs_to_add:
 
468
            # directory is tree-relative
 
469
            abspath = self.abspath(directory.raw_path)
 
470
 
 
471
            # get the contents of this directory.
 
472
 
 
473
            # find the kind of the path being added.
 
474
            kind = osutils.file_kind(abspath)
 
475
 
 
476
            if not InventoryEntry.versionable_kind(kind):
 
477
                trace.warning("skipping %s (can't add file of kind '%s')",
 
478
                              abspath, kind)
 
479
                continue
 
480
            if illegalpath_re.search(directory.raw_path):
 
481
                trace.warning("skipping %r (contains \\n or \\r)" % abspath)
 
482
                continue
 
483
            if directory.raw_path in conflicts_related:
 
484
                # If the file looks like one generated for a conflict, don't
 
485
                # add it.
 
486
                trace.warning(
 
487
                    'skipping %s (generated to help resolve conflicts)',
 
488
                    abspath)
 
489
                continue
 
490
 
 
491
            if parent_ie is not None:
 
492
                versioned = directory.base_path in parent_ie.children
 
493
            else:
 
494
                # without the parent ie, use the relatively slower inventory
 
495
                # probing method
 
496
                versioned = inv.has_filename(
 
497
                        self._fix_case_of_inventory_path(directory.raw_path))
 
498
 
 
499
            if kind == 'directory':
 
500
                try:
 
501
                    sub_branch = bzrdir.BzrDir.open(abspath)
 
502
                    sub_tree = True
 
503
                except errors.NotBranchError:
 
504
                    sub_tree = False
 
505
                except errors.UnsupportedFormatError:
 
506
                    sub_tree = True
 
507
            else:
 
508
                sub_tree = False
 
509
 
 
510
            if directory.raw_path == '':
 
511
                # mutter("tree root doesn't need to be added")
 
512
                sub_tree = False
 
513
            elif versioned:
 
514
                pass
 
515
                # mutter("%r is already versioned", abspath)
 
516
            elif sub_tree:
 
517
                # XXX: This is wrong; people *might* reasonably be trying to
 
518
                # add subtrees as subtrees.  This should probably only be done
 
519
                # in formats which can represent subtrees, and even then
 
520
                # perhaps only when the user asked to add subtrees.  At the
 
521
                # moment you can add them specially through 'join --reference',
 
522
                # which is perhaps reasonable: adding a new reference is a
 
523
                # special operation and can have a special behaviour.  mbp
 
524
                # 20070306
 
525
                trace.mutter("%r is a nested bzr tree", abspath)
 
526
            else:
 
527
                _add_one(self, inv, parent_ie, directory, kind, action)
 
528
                added.append(directory.raw_path)
 
529
 
 
530
            if kind == 'directory' and not sub_tree:
 
531
                if parent_ie is not None:
 
532
                    # must be present:
 
533
                    this_ie = parent_ie.children[directory.base_path]
 
534
                else:
 
535
                    # without the parent ie, use the relatively slower inventory
 
536
                    # probing method
 
537
                    this_id = inv.path2id(
 
538
                        self._fix_case_of_inventory_path(directory.raw_path))
 
539
                    if this_id is None:
 
540
                        this_ie = None
 
541
                    else:
 
542
                        this_ie = inv[this_id]
 
543
 
 
544
                for subf in sorted(os.listdir(abspath)):
 
545
                    # here we could use TreeDirectory rather than
 
546
                    # string concatenation.
 
547
                    subp = osutils.pathjoin(directory.raw_path, subf)
 
548
                    # TODO: is_control_filename is very slow. Make it faster.
 
549
                    # TreeDirectory.is_control_filename could also make this
 
550
                    # faster - its impossible for a non root dir to have a
 
551
                    # control file.
 
552
                    if self.is_control_filename(subp):
 
553
                        trace.mutter("skip control directory %r", subp)
 
554
                    elif subf in this_ie.children:
 
555
                        # recurse into this already versioned subdir.
 
556
                        dirs_to_add.append((_FastPath(subp, subf), this_ie))
 
557
                    else:
 
558
                        # user selection overrides ignoes
 
559
                        # ignore while selecting files - if we globbed in the
 
560
                        # outer loop we would ignore user files.
 
561
                        ignore_glob = self.is_ignored(subp)
 
562
                        if ignore_glob is not None:
 
563
                            # mutter("skip ignored sub-file %r", subp)
 
564
                            ignored.setdefault(ignore_glob, []).append(subp)
 
565
                        else:
 
566
                            #mutter("queue to add sub-file %r", subp)
 
567
                            dirs_to_add.append((_FastPath(subp, subf), this_ie))
 
568
 
 
569
        if len(added) > 0:
 
570
            if save:
 
571
                self._write_inventory(inv)
 
572
            else:
 
573
                self.read_working_inventory()
 
574
        return added, ignored
457
575
 
458
576
    def update_basis_by_delta(self, new_revid, delta):
459
577
        """Update the parents of this tree after a commit.
491
609
        # TODO: Consider re-evaluating the need for this with CHKInventory
492
610
        # we don't strictly need to mutate an inventory for this
493
611
        # it only makes sense when apply_delta is cheaper than get_inventory()
494
 
        inventory = _mod_inventory.mutable_inventory_from_tree(basis)
 
612
        inventory = basis.inventory._get_mutable_inventory()
495
613
        basis.unlock()
496
614
        inventory.apply_delta(delta)
497
 
        rev_tree = revisiontree.InventoryRevisionTree(self.branch.repository,
 
615
        rev_tree = revisiontree.RevisionTree(self.branch.repository,
498
616
                                             inventory, new_revid)
499
617
        self.set_parent_trees([(new_revid, rev_tree)])
500
618
 
508
626
        """Create the default hooks.
509
627
 
510
628
        """
511
 
        hooks.Hooks.__init__(self, "bzrlib.mutabletree", "MutableTree.hooks")
512
 
        self.add_hook('start_commit',
 
629
        hooks.Hooks.__init__(self)
 
630
        self.create_hook(hooks.HookPoint('start_commit',
513
631
            "Called before a commit is performed on a tree. The start commit "
514
632
            "hook is able to change the tree before the commit takes place. "
515
633
            "start_commit is called with the bzrlib.mutabletree.MutableTree "
516
 
            "that the commit is being performed on.", (1, 4))
517
 
        self.add_hook('post_commit',
 
634
            "that the commit is being performed on.", (1, 4), None))
 
635
        self.create_hook(hooks.HookPoint('post_commit',
518
636
            "Called after a commit is performed on a tree. The hook is "
519
637
            "called with a bzrlib.mutabletree.PostCommitHookParams object. "
520
638
            "The mutable tree the commit was performed on is available via "
521
 
            "the mutable_tree attribute of that object.", (2, 0))
 
639
            "the mutable_tree attribute of that object.", (2, 0), None))
522
640
 
523
641
 
524
642
# install the default hooks into the MutableTree class.
538
656
        self.mutable_tree = mutable_tree
539
657
 
540
658
 
541
 
class _SmartAddHelper(object):
542
 
    """Helper for MutableTree.smart_add."""
543
 
 
544
 
    def get_inventory_delta(self):
545
 
        return self._invdelta.values()
546
 
 
547
 
    def _get_ie(self, inv_path):
548
 
        """Retrieve the most up to date inventory entry for a path.
549
 
 
550
 
        :param inv_path: Normalized inventory path
551
 
        :return: Inventory entry (with possibly invalid .children for
552
 
            directories)
553
 
        """
554
 
        entry = self._invdelta.get(inv_path)
555
 
        if entry is not None:
556
 
            return entry[3]
557
 
        # Find a 'best fit' match if the filesystem is case-insensitive
558
 
        inv_path = self.tree._fix_case_of_inventory_path(inv_path)
559
 
        file_id = self.tree.path2id(inv_path)
560
 
        if file_id is not None:
561
 
            return self.tree.iter_entries_by_dir([file_id]).next()[1]
562
 
        return None
563
 
 
564
 
    def _convert_to_directory(self, this_ie, inv_path):
565
 
        """Convert an entry to a directory.
566
 
 
567
 
        :param this_ie: Inventory entry
568
 
        :param inv_path: Normalized path for the inventory entry
569
 
        :return: The new inventory entry
570
 
        """
571
 
        # Same as in _add_one below, if the inventory doesn't
572
 
        # think this is a directory, update the inventory
573
 
        this_ie = _mod_inventory.InventoryDirectory(
574
 
            this_ie.file_id, this_ie.name, this_ie.parent_id)
575
 
        self._invdelta[inv_path] = (inv_path, inv_path, this_ie.file_id,
576
 
            this_ie)
577
 
        return this_ie
578
 
 
579
 
    def _add_one_and_parent(self, parent_ie, path, kind, inv_path):
580
 
        """Add a new entry to the inventory and automatically add unversioned parents.
581
 
 
582
 
        :param parent_ie: Parent inventory entry if known, or None.  If
583
 
            None, the parent is looked up by name and used if present, otherwise it
584
 
            is recursively added.
585
 
        :param path: 
586
 
        :param kind: Kind of new entry (file, directory, etc)
587
 
        :param inv_path:
588
 
        :return: Inventory entry for path and a list of paths which have been added.
589
 
        """
590
 
        # Nothing to do if path is already versioned.
591
 
        # This is safe from infinite recursion because the tree root is
592
 
        # always versioned.
593
 
        inv_dirname = osutils.dirname(inv_path)
594
 
        dirname, basename = osutils.split(path)
595
 
        if parent_ie is None:
596
 
            # slower but does not need parent_ie
597
 
            this_ie = self._get_ie(inv_path)
598
 
            if this_ie is not None:
599
 
                return this_ie
600
 
            # its really not there : add the parent
601
 
            # note that the dirname use leads to some extra str copying etc but as
602
 
            # there are a limited number of dirs we can be nested under, it should
603
 
            # generally find it very fast and not recurse after that.
604
 
            parent_ie = self._add_one_and_parent(None,
605
 
                dirname, 'directory', 
606
 
                inv_dirname)
607
 
        # if the parent exists, but isn't a directory, we have to do the
608
 
        # kind change now -- really the inventory shouldn't pretend to know
609
 
        # the kind of wt files, but it does.
610
 
        if parent_ie.kind != 'directory':
611
 
            # nb: this relies on someone else checking that the path we're using
612
 
            # doesn't contain symlinks.
613
 
            parent_ie = self._convert_to_directory(parent_ie, inv_dirname)
614
 
        file_id = self.action(self.tree.inventory, parent_ie, path, kind)
615
 
        entry = _mod_inventory.make_entry(kind, basename, parent_ie.file_id,
616
 
            file_id=file_id)
617
 
        self._invdelta[inv_path] = (None, inv_path, entry.file_id, entry)
618
 
        self.added.append(inv_path)
619
 
        return entry
620
 
 
621
 
    def _gather_dirs_to_add(self, user_dirs):
622
 
        # only walk the minimal parents needed: we have user_dirs to override
623
 
        # ignores.
624
 
        prev_dir = None
625
 
 
626
 
        is_inside = osutils.is_inside_or_parent_of_any
627
 
        for path, (inv_path, this_ie) in sorted(
628
 
                user_dirs.iteritems(), key=operator.itemgetter(0)):
629
 
            if (prev_dir is None or not is_inside([prev_dir], path)):
630
 
                yield (path, inv_path, this_ie, None)
631
 
            prev_dir = path
632
 
        
633
 
    def __init__(self, tree, action, conflicts_related=None):
634
 
        self.tree = tree
635
 
        if action is None:
636
 
            self.action = add.AddAction()
637
 
        else:
638
 
            self.action = action
639
 
        self._invdelta = {}
640
 
        self.added = []
641
 
        self.ignored = {}
642
 
        if conflicts_related is None:
643
 
            self.conflicts_related = frozenset()
644
 
        else:
645
 
            self.conflicts_related = conflicts_related
646
 
 
647
 
    def add(self, file_list, recurse=True):
648
 
        from bzrlib.inventory import InventoryEntry
649
 
        if not file_list:
650
 
            # no paths supplied: add the entire tree.
651
 
            # FIXME: this assumes we are running in a working tree subdir :-/
652
 
            # -- vila 20100208
653
 
            file_list = [u'.']
654
 
 
655
 
        # expand any symlinks in the directory part, while leaving the
656
 
        # filename alone
657
 
        # only expanding if symlinks are supported avoids windows path bugs
658
 
        if osutils.has_symlinks():
659
 
            file_list = map(osutils.normalizepath, file_list)
660
 
 
661
 
        user_dirs = {}
662
 
        # validate user file paths and convert all paths to tree
663
 
        # relative : it's cheaper to make a tree relative path an abspath
664
 
        # than to convert an abspath to tree relative, and it's cheaper to
665
 
        # perform the canonicalization in bulk.
666
 
        for filepath in osutils.canonical_relpaths(self.tree.basedir, file_list):
667
 
            # validate user parameters. Our recursive code avoids adding new
668
 
            # files that need such validation
669
 
            if self.tree.is_control_filename(filepath):
670
 
                raise errors.ForbiddenControlFileError(filename=filepath)
671
 
 
672
 
            abspath = self.tree.abspath(filepath)
673
 
            kind = osutils.file_kind(abspath)
674
 
            # ensure the named path is added, so that ignore rules in the later
675
 
            # directory walk dont skip it.
676
 
            # we dont have a parent ie known yet.: use the relatively slower
677
 
            # inventory probing method
678
 
            inv_path, _ = osutils.normalized_filename(filepath)
679
 
            this_ie = self._get_ie(inv_path)
680
 
            if this_ie is None:
681
 
                this_ie = self._add_one_and_parent(None, filepath, kind, inv_path)
682
 
            if kind == 'directory':
683
 
                # schedule the dir for scanning
684
 
                user_dirs[filepath] = (inv_path, this_ie)
685
 
 
686
 
        if not recurse:
687
 
            # no need to walk any directories at all.
688
 
            return
689
 
 
690
 
        things_to_add = list(self._gather_dirs_to_add(user_dirs))
691
 
 
692
 
        illegalpath_re = re.compile(r'[\r\n]')
693
 
        for directory, inv_path, this_ie, parent_ie in things_to_add:
694
 
            # directory is tree-relative
695
 
            abspath = self.tree.abspath(directory)
696
 
 
697
 
            # get the contents of this directory.
698
 
 
699
 
            # find the kind of the path being added, and save stat_value
700
 
            # for reuse
701
 
            stat_value = None
702
 
            if this_ie is None:
703
 
                stat_value = osutils.file_stat(abspath)
704
 
                kind = osutils.file_kind_from_stat_mode(stat_value.st_mode)
705
 
            else:
706
 
                kind = this_ie.kind
707
 
            
708
 
            # allow AddAction to skip this file
709
 
            if self.action.skip_file(self.tree,  abspath,  kind,  stat_value):
710
 
                continue
711
 
            if not InventoryEntry.versionable_kind(kind):
712
 
                trace.warning("skipping %s (can't add file of kind '%s')",
713
 
                              abspath, kind)
714
 
                continue
715
 
            if illegalpath_re.search(directory):
716
 
                trace.warning("skipping %r (contains \\n or \\r)" % abspath)
717
 
                continue
718
 
            if directory in self.conflicts_related:
719
 
                # If the file looks like one generated for a conflict, don't
720
 
                # add it.
721
 
                trace.warning(
722
 
                    'skipping %s (generated to help resolve conflicts)',
723
 
                    abspath)
724
 
                continue
725
 
 
726
 
            if kind == 'directory' and directory != '':
727
 
                try:
728
 
                    transport = _mod_transport.get_transport_from_path(abspath)
729
 
                    controldir.ControlDirFormat.find_format(transport)
730
 
                    sub_tree = True
731
 
                except errors.NotBranchError:
732
 
                    sub_tree = False
733
 
                except errors.UnsupportedFormatError:
734
 
                    sub_tree = True
735
 
            else:
736
 
                sub_tree = False
737
 
 
738
 
            if this_ie is not None:
739
 
                pass
740
 
            elif sub_tree:
741
 
                # XXX: This is wrong; people *might* reasonably be trying to
742
 
                # add subtrees as subtrees.  This should probably only be done
743
 
                # in formats which can represent subtrees, and even then
744
 
                # perhaps only when the user asked to add subtrees.  At the
745
 
                # moment you can add them specially through 'join --reference',
746
 
                # which is perhaps reasonable: adding a new reference is a
747
 
                # special operation and can have a special behaviour.  mbp
748
 
                # 20070306
749
 
                trace.warning("skipping nested tree %r", abspath)
750
 
            else:
751
 
                this_ie = self._add_one_and_parent(parent_ie, directory, kind,
752
 
                    inv_path)
753
 
 
754
 
            if kind == 'directory' and not sub_tree:
755
 
                if this_ie.kind != 'directory':
756
 
                    this_ie = self._convert_to_directory(this_ie, inv_path)
757
 
 
758
 
                for subf in sorted(os.listdir(abspath)):
759
 
                    inv_f, _ = osutils.normalized_filename(subf)
760
 
                    # here we could use TreeDirectory rather than
761
 
                    # string concatenation.
762
 
                    subp = osutils.pathjoin(directory, subf)
763
 
                    # TODO: is_control_filename is very slow. Make it faster.
764
 
                    # TreeDirectory.is_control_filename could also make this
765
 
                    # faster - its impossible for a non root dir to have a
766
 
                    # control file.
767
 
                    if self.tree.is_control_filename(subp):
768
 
                        trace.mutter("skip control directory %r", subp)
769
 
                        continue
770
 
                    sub_invp = osutils.pathjoin(inv_path, inv_f)
771
 
                    entry = self._invdelta.get(sub_invp)
772
 
                    if entry is not None:
773
 
                        sub_ie = entry[3]
774
 
                    else:
775
 
                        sub_ie = this_ie.children.get(inv_f)
776
 
                    if sub_ie is not None:
777
 
                        # recurse into this already versioned subdir.
778
 
                        things_to_add.append((subp, sub_invp, sub_ie, this_ie))
779
 
                    else:
780
 
                        # user selection overrides ignores
781
 
                        # ignore while selecting files - if we globbed in the
782
 
                        # outer loop we would ignore user files.
783
 
                        ignore_glob = self.tree.is_ignored(subp)
784
 
                        if ignore_glob is not None:
785
 
                            self.ignored.setdefault(ignore_glob, []).append(subp)
786
 
                        else:
787
 
                            things_to_add.append((subp, sub_invp, None, this_ie))
 
659
class _FastPath(object):
 
660
    """A path object with fast accessors for things like basename."""
 
661
 
 
662
    __slots__ = ['raw_path', 'base_path']
 
663
 
 
664
    def __init__(self, path, base_path=None):
 
665
        """Construct a FastPath from path."""
 
666
        if base_path is None:
 
667
            self.base_path = osutils.basename(path)
 
668
        else:
 
669
            self.base_path = base_path
 
670
        self.raw_path = path
 
671
 
 
672
    def __cmp__(self, other):
 
673
        return cmp(self.raw_path, other.raw_path)
 
674
 
 
675
    def __hash__(self):
 
676
        return hash(self.raw_path)
 
677
 
 
678
 
 
679
def _add_one_and_parent(tree, inv, parent_ie, path, kind, action):
 
680
    """Add a new entry to the inventory and automatically add unversioned parents.
 
681
 
 
682
    :param inv: Inventory which will receive the new entry.
 
683
    :param parent_ie: Parent inventory entry if known, or None.  If
 
684
        None, the parent is looked up by name and used if present, otherwise it
 
685
        is recursively added.
 
686
    :param kind: Kind of new entry (file, directory, etc)
 
687
    :param action: callback(inv, parent_ie, path, kind); return ignored.
 
688
    :return: A list of paths which have been added.
 
689
    """
 
690
    # Nothing to do if path is already versioned.
 
691
    # This is safe from infinite recursion because the tree root is
 
692
    # always versioned.
 
693
    if parent_ie is not None:
 
694
        # we have a parent ie already
 
695
        added = []
 
696
    else:
 
697
        # slower but does not need parent_ie
 
698
        if inv.has_filename(tree._fix_case_of_inventory_path(path.raw_path)):
 
699
            return []
 
700
        # its really not there : add the parent
 
701
        # note that the dirname use leads to some extra str copying etc but as
 
702
        # there are a limited number of dirs we can be nested under, it should
 
703
        # generally find it very fast and not recurse after that.
 
704
        added = _add_one_and_parent(tree, inv, None,
 
705
            _FastPath(osutils.dirname(path.raw_path)), 'directory', action)
 
706
        parent_id = inv.path2id(osutils.dirname(path.raw_path))
 
707
        parent_ie = inv[parent_id]
 
708
    _add_one(tree, inv, parent_ie, path, kind, action)
 
709
    return added + [path.raw_path]
 
710
 
 
711
 
 
712
def _add_one(tree, inv, parent_ie, path, kind, file_id_callback):
 
713
    """Add a new entry to the inventory.
 
714
 
 
715
    :param inv: Inventory which will receive the new entry.
 
716
    :param parent_ie: Parent inventory entry.
 
717
    :param kind: Kind of new entry (file, directory, etc)
 
718
    :param file_id_callback: callback(inv, parent_ie, path, kind); return a
 
719
        file_id or None to generate a new file id
 
720
    :returns: None
 
721
    """
 
722
    file_id = file_id_callback(inv, parent_ie, path, kind)
 
723
    entry = inv.make_entry(kind, path.base_path, parent_ie.file_id,
 
724
        file_id=file_id)
 
725
    inv.add(entry)