~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/mutabletree.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-05-21 13:36:51 UTC
  • mfrom: (5243.2.1 readdir_cleanup)
  • Revision ID: pqm@pqm.ubuntu.com-20100521133651-p62dndo2giy5ls21
(lifeless) Some cleanups to the readdir pyrex code for a little efficiency
 and to avoid compile warnings. (John A Meinel)

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
""")
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
 
378
375
        This is designed more towards DWIM for humans than API clarity.
379
376
        For the specific behaviour see the help for cmd_add().
380
377
 
381
 
        :param file_list: List of zero or more paths.  *NB: these are 
382
 
            interpreted relative to the process cwd, not relative to the 
383
 
            tree.*  (Add and most other tree methods use tree-relative
384
 
            paths.)
385
378
        :param action: A reporter to be called with the inventory, parent_ie,
386
379
            path and kind of the path being added. It may return a file_id if
387
380
            a specific one should be used.
392
385
            of added files, and ignored_files is a dict mapping files that were
393
386
            ignored to the rule that caused them to be ignored.
394
387
        """
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()
 
388
        # not in an inner loop; and we want to remove direct use of this,
 
389
        # so here as a reminder for now. RBC 20070703
 
390
        from bzrlib.inventory import InventoryEntry
 
391
        if action is None:
 
392
            action = add.AddAction()
 
393
 
 
394
        if not file_list:
 
395
            # no paths supplied: add the entire tree.
 
396
            # FIXME: this assumes we are running in a working tree subdir :-/
 
397
            # -- vila 20100208
 
398
            file_list = [u'.']
 
399
        # mutter("smart add of %r")
410
400
        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
 
        """
 
401
        added = []
 
402
        ignored = {}
 
403
        dirs_to_add = []
 
404
        user_dirs = set()
 
405
        conflicts_related = set()
441
406
        # Not all mutable trees can have conflicts
442
407
        if getattr(self, 'conflicts', None) is not None:
443
408
            # Collect all related files without checking whether they exist or
444
409
            # are versioned. It's cheaper to do that once for all conflicts
445
410
            # than trying to find the relevant conflict for each added file.
446
 
            conflicts_related = set()
447
411
            for c in self.conflicts():
448
412
                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
 
413
 
 
414
        # validate user file paths and convert all paths to tree
 
415
        # relative : it's cheaper to make a tree relative path an abspath
 
416
        # than to convert an abspath to tree relative, and it's cheaper to
 
417
        # perform the canonicalization in bulk.
 
418
        for filepath in osutils.canonical_relpaths(self.basedir, file_list):
 
419
            rf = _FastPath(filepath)
 
420
            # validate user parameters. Our recursive code avoids adding new
 
421
            # files that need such validation
 
422
            if self.is_control_filename(rf.raw_path):
 
423
                raise errors.ForbiddenControlFileError(filename=rf.raw_path)
 
424
 
 
425
            abspath = self.abspath(rf.raw_path)
 
426
            kind = osutils.file_kind(abspath)
 
427
            if kind == 'directory':
 
428
                # schedule the dir for scanning
 
429
                user_dirs.add(rf)
 
430
            else:
 
431
                if not InventoryEntry.versionable_kind(kind):
 
432
                    raise errors.BadFileKindError(filename=abspath, kind=kind)
 
433
            # ensure the named path is added, so that ignore rules in the later
 
434
            # directory walk dont skip it.
 
435
            # we dont have a parent ie known yet.: use the relatively slower
 
436
            # inventory probing method
 
437
            versioned = inv.has_filename(rf.raw_path)
 
438
            if versioned:
 
439
                continue
 
440
            added.extend(_add_one_and_parent(self, inv, None, rf, kind, action))
 
441
 
 
442
        if not recurse:
 
443
            # no need to walk any directories at all.
 
444
            if len(added) > 0 and save:
 
445
                self._write_inventory(inv)
 
446
            return added, ignored
 
447
 
 
448
        # only walk the minimal parents needed: we have user_dirs to override
 
449
        # ignores.
 
450
        prev_dir = None
 
451
 
 
452
        is_inside = osutils.is_inside_or_parent_of_any
 
453
        for path in sorted(user_dirs):
 
454
            if (prev_dir is None or not is_inside([prev_dir], path.raw_path)):
 
455
                dirs_to_add.append((path, None))
 
456
            prev_dir = path.raw_path
 
457
 
 
458
        illegalpath_re = re.compile(r'[\r\n]')
 
459
        # dirs_to_add is initialised to a list of directories, but as we scan
 
460
        # directories we append files to it.
 
461
        # XXX: We should determine kind of files when we scan them rather than
 
462
        # adding to this list. RBC 20070703
 
463
        for directory, parent_ie in dirs_to_add:
 
464
            # directory is tree-relative
 
465
            abspath = self.abspath(directory.raw_path)
 
466
 
 
467
            # get the contents of this directory.
 
468
 
 
469
            # find the kind of the path being added.
 
470
            kind = osutils.file_kind(abspath)
 
471
 
 
472
            if not InventoryEntry.versionable_kind(kind):
 
473
                trace.warning("skipping %s (can't add file of kind '%s')",
 
474
                              abspath, kind)
 
475
                continue
 
476
            if illegalpath_re.search(directory.raw_path):
 
477
                trace.warning("skipping %r (contains \\n or \\r)" % abspath)
 
478
                continue
 
479
            if directory.raw_path in conflicts_related:
 
480
                # If the file looks like one generated for a conflict, don't
 
481
                # add it.
 
482
                trace.warning(
 
483
                    'skipping %s (generated to help resolve conflicts)',
 
484
                    abspath)
 
485
                continue
 
486
 
 
487
            if parent_ie is not None:
 
488
                versioned = directory.base_path in parent_ie.children
 
489
            else:
 
490
                # without the parent ie, use the relatively slower inventory
 
491
                # probing method
 
492
                versioned = inv.has_filename(
 
493
                        self._fix_case_of_inventory_path(directory.raw_path))
 
494
 
 
495
            if kind == 'directory':
 
496
                try:
 
497
                    sub_branch = bzrdir.BzrDir.open(abspath)
 
498
                    sub_tree = True
 
499
                except errors.NotBranchError:
 
500
                    sub_tree = False
 
501
                except errors.UnsupportedFormatError:
 
502
                    sub_tree = True
 
503
            else:
 
504
                sub_tree = False
 
505
 
 
506
            if directory.raw_path == '':
 
507
                # mutter("tree root doesn't need to be added")
 
508
                sub_tree = False
 
509
            elif versioned:
 
510
                pass
 
511
                # mutter("%r is already versioned", abspath)
 
512
            elif sub_tree:
 
513
                # XXX: This is wrong; people *might* reasonably be trying to
 
514
                # add subtrees as subtrees.  This should probably only be done
 
515
                # in formats which can represent subtrees, and even then
 
516
                # perhaps only when the user asked to add subtrees.  At the
 
517
                # moment you can add them specially through 'join --reference',
 
518
                # which is perhaps reasonable: adding a new reference is a
 
519
                # special operation and can have a special behaviour.  mbp
 
520
                # 20070306
 
521
                trace.mutter("%r is a nested bzr tree", abspath)
 
522
            else:
 
523
                _add_one(self, inv, parent_ie, directory, kind, action)
 
524
                added.append(directory.raw_path)
 
525
 
 
526
            if kind == 'directory' and not sub_tree:
 
527
                if parent_ie is not None:
 
528
                    # must be present:
 
529
                    this_ie = parent_ie.children[directory.base_path]
 
530
                else:
 
531
                    # without the parent ie, use the relatively slower inventory
 
532
                    # probing method
 
533
                    this_id = inv.path2id(
 
534
                        self._fix_case_of_inventory_path(directory.raw_path))
 
535
                    if this_id is None:
 
536
                        this_ie = None
 
537
                    else:
 
538
                        this_ie = inv[this_id]
 
539
 
 
540
                for subf in sorted(os.listdir(abspath)):
 
541
                    # here we could use TreeDirectory rather than
 
542
                    # string concatenation.
 
543
                    subp = osutils.pathjoin(directory.raw_path, subf)
 
544
                    # TODO: is_control_filename is very slow. Make it faster.
 
545
                    # TreeDirectory.is_control_filename could also make this
 
546
                    # faster - its impossible for a non root dir to have a
 
547
                    # control file.
 
548
                    if self.is_control_filename(subp):
 
549
                        trace.mutter("skip control directory %r", subp)
 
550
                    elif subf in this_ie.children:
 
551
                        # recurse into this already versioned subdir.
 
552
                        dirs_to_add.append((_FastPath(subp, subf), this_ie))
 
553
                    else:
 
554
                        # user selection overrides ignoes
 
555
                        # ignore while selecting files - if we globbed in the
 
556
                        # outer loop we would ignore user files.
 
557
                        ignore_glob = self.is_ignored(subp)
 
558
                        if ignore_glob is not None:
 
559
                            # mutter("skip ignored sub-file %r", subp)
 
560
                            ignored.setdefault(ignore_glob, []).append(subp)
 
561
                        else:
 
562
                            #mutter("queue to add sub-file %r", subp)
 
563
                            dirs_to_add.append((_FastPath(subp, subf), this_ie))
 
564
 
 
565
        if len(added) > 0:
 
566
            if save:
 
567
                self._write_inventory(inv)
 
568
            else:
 
569
                self.read_working_inventory()
 
570
        return added, ignored
457
571
 
458
572
    def update_basis_by_delta(self, new_revid, delta):
459
573
        """Update the parents of this tree after a commit.
491
605
        # TODO: Consider re-evaluating the need for this with CHKInventory
492
606
        # we don't strictly need to mutate an inventory for this
493
607
        # it only makes sense when apply_delta is cheaper than get_inventory()
494
 
        inventory = _mod_inventory.mutable_inventory_from_tree(basis)
 
608
        inventory = basis.inventory._get_mutable_inventory()
495
609
        basis.unlock()
496
610
        inventory.apply_delta(delta)
497
 
        rev_tree = revisiontree.InventoryRevisionTree(self.branch.repository,
 
611
        rev_tree = revisiontree.RevisionTree(self.branch.repository,
498
612
                                             inventory, new_revid)
499
613
        self.set_parent_trees([(new_revid, rev_tree)])
500
614
 
508
622
        """Create the default hooks.
509
623
 
510
624
        """
511
 
        hooks.Hooks.__init__(self, "bzrlib.mutabletree", "MutableTree.hooks")
512
 
        self.add_hook('start_commit',
 
625
        hooks.Hooks.__init__(self)
 
626
        self.create_hook(hooks.HookPoint('start_commit',
513
627
            "Called before a commit is performed on a tree. The start commit "
514
628
            "hook is able to change the tree before the commit takes place. "
515
629
            "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',
 
630
            "that the commit is being performed on.", (1, 4), None))
 
631
        self.create_hook(hooks.HookPoint('post_commit',
518
632
            "Called after a commit is performed on a tree. The hook is "
519
633
            "called with a bzrlib.mutabletree.PostCommitHookParams object. "
520
634
            "The mutable tree the commit was performed on is available via "
521
 
            "the mutable_tree attribute of that object.", (2, 0))
 
635
            "the mutable_tree attribute of that object.", (2, 0), None))
522
636
 
523
637
 
524
638
# install the default hooks into the MutableTree class.
538
652
        self.mutable_tree = mutable_tree
539
653
 
540
654
 
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))
 
655
class _FastPath(object):
 
656
    """A path object with fast accessors for things like basename."""
 
657
 
 
658
    __slots__ = ['raw_path', 'base_path']
 
659
 
 
660
    def __init__(self, path, base_path=None):
 
661
        """Construct a FastPath from path."""
 
662
        if base_path is None:
 
663
            self.base_path = osutils.basename(path)
 
664
        else:
 
665
            self.base_path = base_path
 
666
        self.raw_path = path
 
667
 
 
668
    def __cmp__(self, other):
 
669
        return cmp(self.raw_path, other.raw_path)
 
670
 
 
671
    def __hash__(self):
 
672
        return hash(self.raw_path)
 
673
 
 
674
 
 
675
def _add_one_and_parent(tree, inv, parent_ie, path, kind, action):
 
676
    """Add a new entry to the inventory and automatically add unversioned parents.
 
677
 
 
678
    :param inv: Inventory which will receive the new entry.
 
679
    :param parent_ie: Parent inventory entry if known, or None.  If
 
680
        None, the parent is looked up by name and used if present, otherwise it
 
681
        is recursively added.
 
682
    :param kind: Kind of new entry (file, directory, etc)
 
683
    :param action: callback(inv, parent_ie, path, kind); return ignored.
 
684
    :return: A list of paths which have been added.
 
685
    """
 
686
    # Nothing to do if path is already versioned.
 
687
    # This is safe from infinite recursion because the tree root is
 
688
    # always versioned.
 
689
    if parent_ie is not None:
 
690
        # we have a parent ie already
 
691
        added = []
 
692
    else:
 
693
        # slower but does not need parent_ie
 
694
        if inv.has_filename(tree._fix_case_of_inventory_path(path.raw_path)):
 
695
            return []
 
696
        # its really not there : add the parent
 
697
        # note that the dirname use leads to some extra str copying etc but as
 
698
        # there are a limited number of dirs we can be nested under, it should
 
699
        # generally find it very fast and not recurse after that.
 
700
        added = _add_one_and_parent(tree, inv, None,
 
701
            _FastPath(osutils.dirname(path.raw_path)), 'directory', action)
 
702
        parent_id = inv.path2id(osutils.dirname(path.raw_path))
 
703
        parent_ie = inv[parent_id]
 
704
    _add_one(tree, inv, parent_ie, path, kind, action)
 
705
    return added + [path.raw_path]
 
706
 
 
707
 
 
708
def _add_one(tree, inv, parent_ie, path, kind, file_id_callback):
 
709
    """Add a new entry to the inventory.
 
710
 
 
711
    :param inv: Inventory which will receive the new entry.
 
712
    :param parent_ie: Parent inventory entry.
 
713
    :param kind: Kind of new entry (file, directory, etc)
 
714
    :param file_id_callback: callback(inv, parent_ie, path, kind); return a
 
715
        file_id or None to generate a new file id
 
716
    :returns: None
 
717
    """
 
718
    file_id = file_id_callback(inv, parent_ie, path, kind)
 
719
    entry = inv.make_entry(kind, path.base_path, parent_ie.file_id,
 
720
        file_id=file_id)
 
721
    inv.add(entry)