/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/mutabletree.py

  • Committer: John Arbash Meinel
  • Date: 2010-01-06 17:46:15 UTC
  • mto: (4634.119.1 2.0)
  • mto: This revision was merged to the branch mainline in revision 4951.
  • Revision ID: john@arbash-meinel.com-20100106174615-cq1nckxhbuyemgjx
Fix bug #503886, errors setting up logging go to stderr.

The basic issue is that we were using logging to describe failures
to set up logging. However, those fail with bad error messages
rather than giving us the output we want. This was especially bad
when the failure was occuring on the server. Since 'ssh' will pass
back the stderr stream without bzr handling it at all.

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
28
28
from bzrlib import (
29
29
    add,
30
30
    bzrdir,
 
31
    hooks,
 
32
    symbol_versioning,
 
33
    )
 
34
from bzrlib.osutils import dirname
 
35
from bzrlib.revisiontree import RevisionTree
 
36
from bzrlib.trace import mutter, warning
 
37
""")
 
38
 
 
39
from bzrlib import (
31
40
    errors,
32
 
    hooks,
33
41
    osutils,
34
 
    revisiontree,
35
 
    symbol_versioning,
36
 
    trace,
37
42
    tree,
38
43
    )
39
 
""")
40
 
 
41
44
from bzrlib.decorators import needs_read_lock, needs_write_lock
 
45
from bzrlib.osutils import splitpath
42
46
 
43
47
 
44
48
def needs_tree_write_lock(unbound):
126
130
            # generic constraint checks:
127
131
            if self.is_control_filename(f):
128
132
                raise errors.ForbiddenControlFileError(filename=f)
129
 
            fp = osutils.splitpath(f)
 
133
            fp = splitpath(f)
130
134
        # fill out file kinds for all files [not needed when we stop
131
135
        # caring about the instantaneous file kind within a uncommmitted tree
132
136
        #
233
237
        raise NotImplementedError(self._gather_kinds)
234
238
 
235
239
    @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).
 
240
    def has_changes(self, from_tree):
 
241
        """Quickly check that the tree contains at least one change.
241
242
 
242
243
        :return: True if a change is found. False otherwise
243
244
        """
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
        changes = self.iter_changes(from_tree)
250
246
        try:
251
247
            change = changes.next()
252
248
            # Exclude root (talk about black magic... --vila 20090629)
258
254
            return False
259
255
 
260
256
    @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
257
    def last_revision(self):
300
258
        """Return the revision id of the last commit performed in this tree.
301
259
 
418
376
 
419
377
        if not file_list:
420
378
            # no paths supplied: add the entire tree.
421
 
            # FIXME: this assumes we are running in a working tree subdir :-/
422
 
            # -- vila 20100208
423
379
            file_list = [u'.']
424
380
        # mutter("smart add of %r")
425
381
        inv = self.inventory
427
383
        ignored = {}
428
384
        dirs_to_add = []
429
385
        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
386
 
439
387
        # validate user file paths and convert all paths to tree
440
388
        # relative : it's cheaper to make a tree relative path an abspath
442
390
        # perform the canonicalization in bulk.
443
391
        for filepath in osutils.canonical_relpaths(self.basedir, file_list):
444
392
            rf = _FastPath(filepath)
445
 
            # validate user parameters. Our recursive code avoids adding new
446
 
            # files that need such validation
 
393
            # validate user parameters. Our recursive code avoids adding new files
 
394
            # that need such validation
447
395
            if self.is_control_filename(rf.raw_path):
448
396
                raise errors.ForbiddenControlFileError(filename=rf.raw_path)
449
397
 
455
403
            else:
456
404
                if not InventoryEntry.versionable_kind(kind):
457
405
                    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
 
406
            # ensure the named path is added, so that ignore rules in the later directory
 
407
            # walk dont skip it.
 
408
            # we dont have a parent ie known yet.: use the relatively slower inventory
 
409
            # probing method
462
410
            versioned = inv.has_filename(rf.raw_path)
463
411
            if versioned:
464
412
                continue
495
443
            kind = osutils.file_kind(abspath)
496
444
 
497
445
            if not InventoryEntry.versionable_kind(kind):
498
 
                trace.warning("skipping %s (can't add file of kind '%s')",
499
 
                              abspath, kind)
 
446
                warning("skipping %s (can't add file of kind '%s')", abspath, kind)
500
447
                continue
501
448
            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)
 
449
                warning("skipping %r (contains \\n or \\r)" % abspath)
510
450
                continue
511
451
 
512
452
            if parent_ie is not None:
535
475
                pass
536
476
                # mutter("%r is already versioned", abspath)
537
477
            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)
 
478
                # XXX: This is wrong; people *might* reasonably be trying to add
 
479
                # subtrees as subtrees.  This should probably only be done in formats
 
480
                # which can represent subtrees, and even then perhaps only when
 
481
                # the user asked to add subtrees.  At the moment you can add them
 
482
                # specially through 'join --reference', which is perhaps
 
483
                # reasonable: adding a new reference is a special operation and
 
484
                # can have a special behaviour.  mbp 20070306
 
485
                mutter("%r is a nested bzr tree", abspath)
547
486
            else:
548
487
                _add_one(self, inv, parent_ie, directory, kind, action)
549
488
                added.append(directory.raw_path)
556
495
                    # without the parent ie, use the relatively slower inventory
557
496
                    # probing method
558
497
                    this_id = inv.path2id(
559
 
                        self._fix_case_of_inventory_path(directory.raw_path))
 
498
                            self._fix_case_of_inventory_path(directory.raw_path))
560
499
                    if this_id is None:
561
500
                        this_ie = None
562
501
                    else:
571
510
                    # faster - its impossible for a non root dir to have a
572
511
                    # control file.
573
512
                    if self.is_control_filename(subp):
574
 
                        trace.mutter("skip control directory %r", subp)
 
513
                        mutter("skip control directory %r", subp)
575
514
                    elif subf in this_ie.children:
576
515
                        # recurse into this already versioned subdir.
577
516
                        dirs_to_add.append((_FastPath(subp, subf), this_ie))
633
572
        inventory = basis.inventory._get_mutable_inventory()
634
573
        basis.unlock()
635
574
        inventory.apply_delta(delta)
636
 
        rev_tree = revisiontree.RevisionTree(self.branch.repository,
637
 
                                             inventory, new_revid)
 
575
        rev_tree = RevisionTree(self.branch.repository, inventory, new_revid)
638
576
        self.set_parent_trees([(new_revid, rev_tree)])
639
577
 
640
578
 
723
661
        # there are a limited number of dirs we can be nested under, it should
724
662
        # generally find it very fast and not recurse after that.
725
663
        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))
 
664
            _FastPath(dirname(path.raw_path)), 'directory', action)
 
665
        parent_id = inv.path2id(dirname(path.raw_path))
728
666
        parent_ie = inv[parent_id]
729
667
    _add_one(tree, inv, parent_ie, path, kind, action)
730
668
    return added + [path.raw_path]