← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/mutabletree.py

Merge lp:bzr.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/mutabletree.py
23
23
from bzrlib.lazy_import import lazy_import
24
24
lazy_import(globals(), """
25
25
import os
 
26
import re
26
27
 
27
28
from bzrlib import (
28
29
    add,
29
30
    bzrdir,
 
31
    errors,
30
32
    hooks,
 
33
    osutils,
 
34
    revisiontree,
31
35
    symbol_versioning,
 
36
    trace,
 
37
    tree,
32
38
    )
33
 
from bzrlib.osutils import dirname
34
 
from bzrlib.revisiontree import RevisionTree
35
 
from bzrlib.trace import mutter, warning
36
39
""")
37
40
 
38
 
from bzrlib import (
39
 
    errors,
40
 
    osutils,
41
 
    tree,
42
 
    )
43
41
from bzrlib.decorators import needs_read_lock, needs_write_lock
44
 
from bzrlib.osutils import splitpath
45
42
 
46
43
 
47
44
def needs_tree_write_lock(unbound):
129
126
            # generic constraint checks:
130
127
            if self.is_control_filename(f):
131
128
                raise errors.ForbiddenControlFileError(filename=f)
132
 
            fp = splitpath(f)
 
129
            fp = osutils.splitpath(f)
133
130
        # fill out file kinds for all files [not needed when we stop
134
131
        # caring about the instantaneous file kind within a uncommmitted tree
135
132
        #
236
233
        raise NotImplementedError(self._gather_kinds)
237
234
 
238
235
    @needs_read_lock
239
 
    def has_changes(self, from_tree):
240
 
        """Quickly check that the tree contains at least one change.
 
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
241
 
242
242
        :return: True if a change is found. False otherwise
243
243
        """
244
 
        changes = self.iter_changes(from_tree)
 
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)
245
250
        try:
246
251
            change = changes.next()
247
252
            # Exclude root (talk about black magic... --vila 20090629)
389
394
        # perform the canonicalization in bulk.
390
395
        for filepath in osutils.canonical_relpaths(self.basedir, file_list):
391
396
            rf = _FastPath(filepath)
392
 
            # validate user parameters. Our recursive code avoids adding new files
393
 
            # that need such validation
 
397
            # validate user parameters. Our recursive code avoids adding new
 
398
            # files that need such validation
394
399
            if self.is_control_filename(rf.raw_path):
395
400
                raise errors.ForbiddenControlFileError(filename=rf.raw_path)
396
401
 
402
407
            else:
403
408
                if not InventoryEntry.versionable_kind(kind):
404
409
                    raise errors.BadFileKindError(filename=abspath, kind=kind)
405
 
            # ensure the named path is added, so that ignore rules in the later directory
406
 
            # walk dont skip it.
407
 
            # we dont have a parent ie known yet.: use the relatively slower inventory
408
 
            # probing method
 
410
            # ensure the named path is added, so that ignore rules in the later
 
411
            # directory walk dont skip it.
 
412
            # we dont have a parent ie known yet.: use the relatively slower
 
413
            # inventory probing method
409
414
            versioned = inv.has_filename(rf.raw_path)
410
415
            if versioned:
411
416
                continue
427
432
                dirs_to_add.append((path, None))
428
433
            prev_dir = path.raw_path
429
434
 
 
435
        illegalpath_re = re.compile(r'[\r\n]')
430
436
        # dirs_to_add is initialised to a list of directories, but as we scan
431
437
        # directories we append files to it.
432
438
        # XXX: We should determine kind of files when we scan them rather than
441
447
            kind = osutils.file_kind(abspath)
442
448
 
443
449
            if not InventoryEntry.versionable_kind(kind):
444
 
                warning("skipping %s (can't add file of kind '%s')", abspath, kind)
 
450
                trace.warning("skipping %s (can't add file of kind '%s')",
 
451
                              abspath, kind)
 
452
                continue
 
453
            if illegalpath_re.search(directory.raw_path):
 
454
                trace.warning("skipping %r (contains \\n or \\r)" % abspath)
445
455
                continue
446
456
 
447
457
            if parent_ie is not None:
470
480
                pass
471
481
                # mutter("%r is already versioned", abspath)
472
482
            elif sub_tree:
473
 
                # XXX: This is wrong; people *might* reasonably be trying to add
474
 
                # subtrees as subtrees.  This should probably only be done in formats
475
 
                # which can represent subtrees, and even then perhaps only when
476
 
                # the user asked to add subtrees.  At the moment you can add them
477
 
                # specially through 'join --reference', which is perhaps
478
 
                # reasonable: adding a new reference is a special operation and
479
 
                # can have a special behaviour.  mbp 20070306
480
 
                mutter("%r is a nested bzr tree", abspath)
 
483
                # XXX: This is wrong; people *might* reasonably be trying to
 
484
                # add subtrees as subtrees.  This should probably only be done
 
485
                # in formats which can represent subtrees, and even then
 
486
                # perhaps only when the user asked to add subtrees.  At the
 
487
                # moment you can add them specially through 'join --reference',
 
488
                # which is perhaps reasonable: adding a new reference is a
 
489
                # special operation and can have a special behaviour.  mbp
 
490
                # 20070306
 
491
                trace.mutter("%r is a nested bzr tree", abspath)
481
492
            else:
482
493
                _add_one(self, inv, parent_ie, directory, kind, action)
483
494
                added.append(directory.raw_path)
490
501
                    # without the parent ie, use the relatively slower inventory
491
502
                    # probing method
492
503
                    this_id = inv.path2id(
493
 
                            self._fix_case_of_inventory_path(directory.raw_path))
 
504
                        self._fix_case_of_inventory_path(directory.raw_path))
494
505
                    if this_id is None:
495
506
                        this_ie = None
496
507
                    else:
505
516
                    # faster - its impossible for a non root dir to have a
506
517
                    # control file.
507
518
                    if self.is_control_filename(subp):
508
 
                        mutter("skip control directory %r", subp)
 
519
                        trace.mutter("skip control directory %r", subp)
509
520
                    elif subf in this_ie.children:
510
521
                        # recurse into this already versioned subdir.
511
522
                        dirs_to_add.append((_FastPath(subp, subf), this_ie))
567
578
        inventory = basis.inventory._get_mutable_inventory()
568
579
        basis.unlock()
569
580
        inventory.apply_delta(delta)
570
 
        rev_tree = RevisionTree(self.branch.repository, inventory, new_revid)
 
581
        rev_tree = revisiontree.RevisionTree(self.branch.repository,
 
582
                                             inventory, new_revid)
571
583
        self.set_parent_trees([(new_revid, rev_tree)])
572
584
 
573
585
 
656
668
        # there are a limited number of dirs we can be nested under, it should
657
669
        # generally find it very fast and not recurse after that.
658
670
        added = _add_one_and_parent(tree, inv, None,
659
 
            _FastPath(dirname(path.raw_path)), 'directory', action)
660
 
        parent_id = inv.path2id(dirname(path.raw_path))
 
671
            _FastPath(osutils.dirname(path.raw_path)), 'directory', action)
 
672
        parent_id = inv.path2id(osutils.dirname(path.raw_path))
661
673
        parent_ie = inv[parent_id]
662
674
    _add_one(tree, inv, parent_ie, path, kind, action)
663
675
    return added + [path.raw_path]