~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/check.py

  • Committer: John Arbash Meinel
  • Date: 2008-10-04 14:10:13 UTC
  • mto: This revision was merged to the branch mainline in revision 3805.
  • Revision ID: john@arbash-meinel.com-20081004141013-yskxjlwtuy2k18ue
Playing around with expanding requests for btree index nodes into neighboring nodes.

Show diffs side-by-side

added added

removed removed

Lines of Context:
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
# TODO: Check ancestries are correct for every revision: includes
18
18
# every committed so far, and in a reasonable order.
21
21
 
22
22
# TODO: Check for extra files in the control directory.
23
23
 
24
 
# TODO: Check revision, inventory and entry objects have all
 
24
# TODO: Check revision, inventory and entry objects have all 
25
25
# required fields.
26
26
 
27
27
# TODO: Get every revision in the revision-store even if they're not
32
32
# raising them.  If there's more than one exception it'd be good to see them
33
33
# all.
34
34
 
35
 
"""Checking of bzr objects.
36
 
 
37
 
check_refs is a concept used for optimising check. Objects that depend on other
38
 
objects (e.g. tree on repository) can list the objects they would be requesting
39
 
so that when the dependent object is checked, matches can be pulled out and
40
 
evaluated in-line rather than re-reading the same data many times.
41
 
check_refs are tuples (kind, value). Currently defined kinds are:
42
 
 
43
 
* 'trees', where value is a revid and the looked up objects are revision trees.
44
 
* 'lefthand-distance', where value is a revid and the looked up objects are the
45
 
  distance along the lefthand path to NULL for that revid.
46
 
* 'revision-existence', where value is a revid, and the result is True or False
47
 
  indicating that the revision was found/not found.
48
 
"""
49
 
 
50
 
from bzrlib import (
51
 
    errors,
52
 
    ui,
53
 
    )
 
35
from bzrlib import errors, osutils
 
36
from bzrlib import repository as _mod_repository
 
37
from bzrlib import revision
54
38
from bzrlib.branch import Branch
55
39
from bzrlib.bzrdir import BzrDir
56
 
from bzrlib.revision import NULL_REVISION
57
 
from bzrlib.trace import note
 
40
from bzrlib.errors import BzrCheckError
 
41
from bzrlib.repository import Repository
 
42
from bzrlib.symbol_versioning import deprecated_function, deprecated_in
 
43
from bzrlib.trace import log_error, note
 
44
import bzrlib.ui
58
45
from bzrlib.workingtree import WorkingTree
59
46
 
60
 
 
61
47
class Check(object):
62
48
    """Check a repository"""
63
49
 
64
 
    def __init__(self, repository, check_repo=True):
65
 
        self.repository = repository
66
 
 
67
 
    def report_results(self, verbose):
68
 
        raise NotImplementedError(self.report_results)
69
 
 
70
 
 
71
 
class VersionedFileCheck(Check):
72
 
    """Check a versioned file repository"""
73
 
 
74
50
    # The Check object interacts with InventoryEntry.check, etc.
75
51
 
76
 
    def __init__(self, repository, check_repo=True):
 
52
    def __init__(self, repository):
77
53
        self.repository = repository
 
54
        self.checked_text_cnt = 0
78
55
        self.checked_rev_cnt = 0
79
 
        self.ghosts = set()
 
56
        self.ghosts = []
 
57
        self.repeated_text_cnt = 0
80
58
        self.missing_parent_links = {}
81
59
        self.missing_inventory_sha_cnt = 0
82
60
        self.missing_revision_cnt = 0
 
61
        # maps (file-id, version) -> sha1; used by InventoryFile._check
 
62
        self.checked_texts = {}
83
63
        self.checked_weaves = set()
84
64
        self.unreferenced_versions = set()
85
65
        self.inconsistent_parents = []
86
 
        self.rich_roots = repository.supports_rich_root()
87
 
        self.text_key_references = {}
88
 
        self.check_repo = check_repo
89
 
        self.other_results = []
90
 
        # Plain text lines to include in the report
91
 
        self._report_items = []
92
 
        # Keys we are looking for; may be large and need spilling to disk.
93
 
        # key->(type(revision/inventory/text/signature/map), sha1, first-referer)
94
 
        self.pending_keys = {}
95
 
        # Ancestors map for all of revisions being checked; while large helper
96
 
        # functions we call would create it anyway, so better to have once and
97
 
        # keep.
98
 
        self.ancestors = {}
99
66
 
100
 
    def check(self, callback_refs=None, check_repo=True):
101
 
        if callback_refs is None:
102
 
            callback_refs = {}
 
67
    def check(self):
103
68
        self.repository.lock_read()
104
 
        self.progress = ui.ui_factory.nested_progress_bar()
 
69
        self.progress = bzrlib.ui.ui_factory.nested_progress_bar()
105
70
        try:
106
 
            self.progress.update('check', 0, 4)
107
 
            if self.check_repo:
108
 
                self.progress.update('checking revisions', 0)
109
 
                self.check_revisions()
110
 
                self.progress.update('checking commit contents', 1)
111
 
                self.repository._check_inventories(self)
112
 
                self.progress.update('checking file graphs', 2)
113
 
                # check_weaves is done after the revision scan so that
114
 
                # revision index is known to be valid.
115
 
                self.check_weaves()
116
 
            self.progress.update('checking branches and trees', 3)
117
 
            if callback_refs:
118
 
                repo = self.repository
119
 
                # calculate all refs, and callback the objects requesting them.
120
 
                refs = {}
121
 
                wanting_items = set()
122
 
                # Current crude version calculates everything and calls
123
 
                # everything at once. Doing a queue and popping as things are
124
 
                # satisfied would be cheaper on memory [but few people have
125
 
                # huge numbers of working trees today. TODO: fix before
126
 
                # landing].
127
 
                distances = set()
128
 
                existences = set()
129
 
                for ref, wantlist in callback_refs.iteritems():
130
 
                    wanting_items.update(wantlist)
131
 
                    kind, value = ref
132
 
                    if kind == 'trees':
133
 
                        refs[ref] = repo.revision_tree(value)
134
 
                    elif kind == 'lefthand-distance':
135
 
                        distances.add(value)
136
 
                    elif kind == 'revision-existence':
137
 
                        existences.add(value)
138
 
                    else:
139
 
                        raise AssertionError(
140
 
                            'unknown ref kind for ref %s' % ref)
141
 
                node_distances = repo.get_graph().find_lefthand_distances(distances)
142
 
                for key, distance in node_distances.iteritems():
143
 
                    refs[('lefthand-distance', key)] = distance
144
 
                    if key in existences and distance > 0:
145
 
                        refs[('revision-existence', key)] = True
146
 
                        existences.remove(key)
147
 
                parent_map = repo.get_graph().get_parent_map(existences)
148
 
                for key in parent_map:
149
 
                    refs[('revision-existence', key)] = True
150
 
                    existences.remove(key)
151
 
                for key in existences:
152
 
                    refs[('revision-existence', key)] = False
153
 
                for item in wanting_items:
154
 
                    if isinstance(item, WorkingTree):
155
 
                        item._check(refs)
156
 
                    if isinstance(item, Branch):
157
 
                        self.other_results.append(item.check(refs))
 
71
            self.progress.update('retrieving inventory', 0, 2)
 
72
            # do not put in init, as it should be done with progess,
 
73
            # and inside the lock.
 
74
            self.inventory_weave = self.repository.inventories
 
75
            self.progress.update('checking revision graph', 1)
 
76
            self.check_revision_graph()
 
77
            self.plan_revisions()
 
78
            revno = 0
 
79
            while revno < len(self.planned_revisions):
 
80
                rev_id = self.planned_revisions[revno]
 
81
                self.progress.update('checking revision', revno,
 
82
                                     len(self.planned_revisions))
 
83
                revno += 1
 
84
                self.check_one_rev(rev_id)
 
85
            # check_weaves is done after the revision scan so that
 
86
            # revision index is known to be valid.
 
87
            self.check_weaves()
158
88
        finally:
159
89
            self.progress.finished()
160
90
            self.repository.unlock()
161
91
 
162
 
    def _check_revisions(self, revisions_iterator):
163
 
        """Check revision objects by decorating a generator.
164
 
 
165
 
        :param revisions_iterator: An iterator of(revid, Revision-or-None).
166
 
        :return: A generator of the contents of revisions_iterator.
167
 
        """
168
 
        self.planned_revisions = set()
169
 
        for revid, revision in revisions_iterator:
170
 
            yield revid, revision
171
 
            self._check_one_rev(revid, revision)
172
 
        # Flatten the revisions we found to guarantee consistent later
173
 
        # iteration.
174
 
        self.planned_revisions = list(self.planned_revisions)
175
 
        # TODO: extract digital signatures as items to callback on too.
176
 
 
177
 
    def check_revisions(self):
178
 
        """Scan revisions, checking data directly available as we go."""
179
 
        revision_iterator = self.repository._iter_revisions(None)
180
 
        revision_iterator = self._check_revisions(revision_iterator)
181
 
        # We read the all revisions here:
182
 
        # - doing this allows later code to depend on the revision index.
183
 
        # - we can fill out existence flags at this point
184
 
        # - we can read the revision inventory sha at this point
185
 
        # - we can check properties and serialisers etc.
186
 
        if not self.repository._format.revision_graph_can_have_wrong_parents:
187
 
            # The check against the index isn't needed.
 
92
    def check_revision_graph(self):
 
93
        if not self.repository.revision_graph_can_have_wrong_parents():
 
94
            # This check is not necessary.
188
95
            self.revs_with_bad_parents_in_index = None
189
 
            for thing in revision_iterator:
190
 
                pass
191
 
        else:
192
 
            bad_revisions = self.repository._find_inconsistent_revision_parents(
193
 
                revision_iterator)
194
 
            self.revs_with_bad_parents_in_index = list(bad_revisions)
 
96
            return
 
97
        bad_revisions = self.repository._find_inconsistent_revision_parents()
 
98
        self.revs_with_bad_parents_in_index = list(bad_revisions)
 
99
 
 
100
    def plan_revisions(self):
 
101
        repository = self.repository
 
102
        self.planned_revisions = repository.all_revision_ids()
 
103
        self.progress.clear()
 
104
        inventoried = set(key[-1] for key in self.inventory_weave.keys())
 
105
        awol = set(self.planned_revisions) - inventoried
 
106
        if len(awol) > 0:
 
107
            raise BzrCheckError('Stored revisions missing from inventory'
 
108
                '{%s}' % ','.join([f for f in awol]))
195
109
 
196
110
    def report_results(self, verbose):
197
 
        if self.check_repo:
198
 
            self._report_repo_results(verbose)
199
 
        for result in self.other_results:
200
 
            result.report_results(verbose)
201
 
 
202
 
    def _report_repo_results(self, verbose):
203
111
        note('checked repository %s format %s',
204
 
            self.repository.user_url,
205
 
            self.repository._format)
 
112
             self.repository.bzrdir.root_transport,
 
113
             self.repository._format)
206
114
        note('%6d revisions', self.checked_rev_cnt)
207
115
        note('%6d file-ids', len(self.checked_weaves))
208
 
        if verbose:
209
 
            note('%6d unreferenced text versions',
210
 
                len(self.unreferenced_versions))
211
 
        if verbose and len(self.unreferenced_versions):
212
 
                for file_id, revision_id in self.unreferenced_versions:
213
 
                    note('unreferenced version: {%s} in %s', revision_id,
214
 
                        file_id)
 
116
        note('%6d unique file texts', self.checked_text_cnt)
 
117
        note('%6d repeated file texts', self.repeated_text_cnt)
 
118
        note('%6d unreferenced text versions',
 
119
             len(self.unreferenced_versions))
215
120
        if self.missing_inventory_sha_cnt:
216
121
            note('%6d revisions are missing inventory_sha1',
217
122
                 self.missing_inventory_sha_cnt)
231
136
                    note('      %s should be in the ancestry for:', link)
232
137
                    for linker in linkers:
233
138
                        note('       * %s', linker)
 
139
            if verbose:
 
140
                for file_id, revision_id in self.unreferenced_versions:
 
141
                    log_error('unreferenced version: {%s} in %s', revision_id,
 
142
                        file_id)
234
143
        if len(self.inconsistent_parents):
235
144
            note('%6d inconsistent parents', len(self.inconsistent_parents))
236
145
            if verbose:
250
159
                        '       %s has wrong parents in index: '
251
160
                        '%r should be %r',
252
161
                        revision_id, index_parents, actual_parents)
253
 
        for item in self._report_items:
254
 
            note(item)
255
 
 
256
 
    def _check_one_rev(self, rev_id, rev):
257
 
        """Cross-check one revision.
258
 
 
259
 
        :param rev_id: A revision id to check.
260
 
        :param rev: A revision or None to indicate a missing revision.
 
162
 
 
163
    def check_one_rev(self, rev_id):
 
164
        """Check one revision.
 
165
 
 
166
        rev_id - the one to check
261
167
        """
 
168
        rev = self.repository.get_revision(rev_id)
 
169
                
262
170
        if rev.revision_id != rev_id:
263
 
            self._report_items.append(
264
 
                'Mismatched internal revid {%s} and index revid {%s}' % (
265
 
                rev.revision_id, rev_id))
266
 
            rev_id = rev.revision_id
267
 
        # Check this revision tree etc, and count as seen when we encounter a
268
 
        # reference to it.
269
 
        self.planned_revisions.add(rev_id)
270
 
        # It is not a ghost
271
 
        self.ghosts.discard(rev_id)
272
 
        # Count all parents as ghosts if we haven't seen them yet.
 
171
            raise BzrCheckError('wrong internal revision id in revision {%s}'
 
172
                                % rev_id)
 
173
 
273
174
        for parent in rev.parent_ids:
274
175
            if not parent in self.planned_revisions:
275
 
                self.ghosts.add(parent)
276
 
        
277
 
        self.ancestors[rev_id] = tuple(rev.parent_ids) or (NULL_REVISION,)
278
 
        self.add_pending_item(rev_id, ('inventories', rev_id), 'inventory',
279
 
            rev.inventory_sha1)
 
176
                missing_links = self.missing_parent_links.get(parent, [])
 
177
                missing_links.append(rev_id)
 
178
                self.missing_parent_links[parent] = missing_links
 
179
                # list based so somewhat slow,
 
180
                # TODO have a planned_revisions list and set.
 
181
                if self.repository.has_revision(parent):
 
182
                    missing_ancestry = self.repository.get_ancestry(parent)
 
183
                    for missing in missing_ancestry:
 
184
                        if (missing is not None 
 
185
                            and missing not in self.planned_revisions):
 
186
                            self.planned_revisions.append(missing)
 
187
                else:
 
188
                    self.ghosts.append(rev_id)
 
189
 
 
190
        if rev.inventory_sha1:
 
191
            inv_sha1 = self.repository.get_inventory_sha1(rev_id)
 
192
            if inv_sha1 != rev.inventory_sha1:
 
193
                raise BzrCheckError('Inventory sha1 hash doesn\'t match'
 
194
                    ' value in revision {%s}' % rev_id)
 
195
        self._check_revision_tree(rev_id)
280
196
        self.checked_rev_cnt += 1
281
197
 
282
 
    def add_pending_item(self, referer, key, kind, sha1):
283
 
        """Add a reference to a sha1 to be cross checked against a key.
284
 
 
285
 
        :param referer: The referer that expects key to have sha1.
286
 
        :param key: A storage key e.g. ('texts', 'foo@bar-20040504-1234')
287
 
        :param kind: revision/inventory/text/map/signature
288
 
        :param sha1: A hex sha1 or None if no sha1 is known.
289
 
        """
290
 
        existing = self.pending_keys.get(key)
291
 
        if existing:
292
 
            if sha1 != existing[1]:
293
 
                self._report_items.append('Multiple expected sha1s for %s. {%s}'
294
 
                    ' expects {%s}, {%s} expects {%s}', (
295
 
                    key, referer, sha1, existing[1], existing[0]))
296
 
        else:
297
 
            self.pending_keys[key] = (kind, sha1, referer)
298
 
 
299
198
    def check_weaves(self):
300
199
        """Check all the weaves we can get our hands on.
301
200
        """
302
201
        weave_ids = []
303
 
        storebar = ui.ui_factory.nested_progress_bar()
304
 
        try:
305
 
            self._check_weaves(storebar)
306
 
        finally:
307
 
            storebar.finished()
308
 
 
309
 
    def _check_weaves(self, storebar):
310
 
        storebar.update('text-index', 0, 2)
311
 
        if self.repository._format.fast_deltas:
312
 
            # We haven't considered every fileid instance so far.
313
 
            weave_checker = self.repository._get_versioned_file_checker(
314
 
                ancestors=self.ancestors)
315
 
        else:
316
 
            weave_checker = self.repository._get_versioned_file_checker(
317
 
                text_key_references=self.text_key_references,
318
 
                ancestors=self.ancestors)
319
 
        storebar.update('file-graph', 1)
 
202
        self.progress.update('checking inventory', 0, 2)
 
203
        self.inventory_weave.check(progress_bar=self.progress)
 
204
        self.progress.update('checking text storage', 1, 2)
 
205
        self.repository.texts.check(progress_bar=self.progress)
 
206
        weave_checker = self.repository._get_versioned_file_checker()
320
207
        result = weave_checker.check_file_version_parents(
321
 
            self.repository.texts)
 
208
            self.repository.texts, progress_bar=self.progress)
322
209
        self.checked_weaves = weave_checker.file_ids
323
210
        bad_parents, unused_versions = result
324
211
        bad_parents = bad_parents.items()
332
219
                (revision_id, weave_id, weave_parents, correct_parents))
333
220
        self.unreferenced_versions.update(unused_versions)
334
221
 
335
 
    def _add_entry_to_text_key_references(self, inv, entry):
336
 
        if not self.rich_roots and entry.name == '':
337
 
            return
338
 
        key = (entry.file_id, entry.revision)
339
 
        self.text_key_references.setdefault(key, False)
340
 
        if entry.revision == inv.revision_id:
341
 
            self.text_key_references[key] = True
342
 
 
343
 
 
344
 
def scan_branch(branch, needed_refs, to_unlock):
345
 
    """Scan a branch for refs.
346
 
 
347
 
    :param branch:  The branch to schedule for checking.
348
 
    :param needed_refs: Refs we are accumulating.
349
 
    :param to_unlock: The unlock list accumulating.
350
 
    """
351
 
    note("Checking branch at '%s'." % (branch.base,))
 
222
    def _check_revision_tree(self, rev_id):
 
223
        tree = self.repository.revision_tree(rev_id)
 
224
        inv = tree.inventory
 
225
        seen_ids = {}
 
226
        for file_id in inv:
 
227
            if file_id in seen_ids:
 
228
                raise BzrCheckError('duplicated file_id {%s} '
 
229
                                    'in inventory for revision {%s}'
 
230
                                    % (file_id, rev_id))
 
231
            seen_ids[file_id] = True
 
232
        for file_id in inv:
 
233
            ie = inv[file_id]
 
234
            ie.check(self, rev_id, inv, tree)
 
235
        seen_names = {}
 
236
        for path, ie in inv.iter_entries():
 
237
            if path in seen_names:
 
238
                raise BzrCheckError('duplicated path %s '
 
239
                                    'in inventory for revision {%s}'
 
240
                                    % (path, rev_id))
 
241
            seen_names[path] = True
 
242
 
 
243
 
 
244
@deprecated_function(deprecated_in((1,6,0)))
 
245
def check(branch, verbose):
 
246
    """Run consistency checks on a branch.
 
247
    
 
248
    Results are reported through logging.
 
249
    
 
250
    Deprecated in 1.6.  Please use check_branch instead.
 
251
 
 
252
    :raise BzrCheckError: if there's a consistency error.
 
253
    """
 
254
    check_branch(branch, verbose)
 
255
 
 
256
 
 
257
def check_branch(branch, verbose):
 
258
    """Run consistency checks on a branch.
 
259
 
 
260
    Results are reported through logging.
 
261
 
 
262
    :raise BzrCheckError: if there's a consistency error.
 
263
    """
352
264
    branch.lock_read()
353
 
    to_unlock.append(branch)
354
 
    branch_refs = branch._get_check_refs()
355
 
    for ref in branch_refs:
356
 
        reflist = needed_refs.setdefault(ref, [])
357
 
        reflist.append(branch)
358
 
 
359
 
 
360
 
def scan_tree(base_tree, tree, needed_refs, to_unlock):
361
 
    """Scan a tree for refs.
362
 
 
363
 
    :param base_tree: The original tree check opened, used to detect duplicate
364
 
        tree checks.
365
 
    :param tree:  The tree to schedule for checking.
366
 
    :param needed_refs: Refs we are accumulating.
367
 
    :param to_unlock: The unlock list accumulating.
368
 
    """
369
 
    if base_tree is not None and tree.basedir == base_tree.basedir:
370
 
        return
371
 
    note("Checking working tree at '%s'." % (tree.basedir,))
372
 
    tree.lock_read()
373
 
    to_unlock.append(tree)
374
 
    tree_refs = tree._get_check_refs()
375
 
    for ref in tree_refs:
376
 
        reflist = needed_refs.setdefault(ref, [])
377
 
        reflist.append(tree)
 
265
    try:
 
266
        branch_result = branch.check()
 
267
    finally:
 
268
        branch.unlock()
 
269
    branch_result.report_results(verbose)
378
270
 
379
271
 
380
272
def check_dwim(path, verbose, do_branch=False, do_repo=False, do_tree=False):
381
 
    """Check multiple objects.
382
 
 
383
 
    If errors occur they are accumulated and reported as far as possible, and
384
 
    an exception raised at the end of the process.
385
 
    """
386
273
    try:
387
 
        base_tree, branch, repo, relpath = \
 
274
        tree, branch, repo, relpath = \
388
275
                        BzrDir.open_containing_tree_branch_or_repository(path)
389
276
    except errors.NotBranchError:
390
 
        base_tree = branch = repo = None
391
 
 
392
 
    to_unlock = []
393
 
    needed_refs= {}
394
 
    try:
395
 
        if base_tree is not None:
396
 
            # If the tree is a lightweight checkout we won't see it in
397
 
            # repo.find_branches - add now.
398
 
            if do_tree:
399
 
                scan_tree(None, base_tree, needed_refs, to_unlock)
400
 
            branch = base_tree.branch
401
 
        if branch is not None:
402
 
            # We have a branch
403
 
            if repo is None:
404
 
                # The branch is in a shared repository
405
 
                repo = branch.repository
406
 
        if repo is not None:
407
 
            repo.lock_read()
408
 
            to_unlock.append(repo)
409
 
            branches = repo.find_branches(using=True)
410
 
            saw_tree = False
411
 
            if do_branch or do_tree:
412
 
                for branch in branches:
413
 
                    if do_tree:
414
 
                        try:
415
 
                            tree = branch.bzrdir.open_workingtree()
416
 
                            saw_tree = True
417
 
                        except (errors.NotLocalUrl, errors.NoWorkingTree):
418
 
                            pass
419
 
                        else:
420
 
                            scan_tree(base_tree, tree, needed_refs, to_unlock)
421
 
                    if do_branch:
422
 
                        scan_branch(branch, needed_refs, to_unlock)
423
 
            if do_branch and not branches:
424
 
                note("No branch found at specified location.")
425
 
            if do_tree and base_tree is None and not saw_tree:
426
 
                note("No working tree found at specified location.")
427
 
            if do_repo or do_branch or do_tree:
428
 
                if do_repo:
429
 
                    note("Checking repository at '%s'."
430
 
                         % (repo.user_url,))
431
 
                result = repo.check(None, callback_refs=needed_refs,
432
 
                    check_repo=do_repo)
 
277
        tree = branch = repo = None
 
278
 
 
279
    if do_tree:
 
280
        if tree is not None:
 
281
            note("Checking working tree at '%s'." 
 
282
                 % (tree.bzrdir.root_transport.base,))
 
283
            tree._check()
 
284
        else:
 
285
            log_error("No working tree found at specified location.")
 
286
 
 
287
    if branch is not None:
 
288
        # We have a branch
 
289
        if repo is None:
 
290
            # The branch is in a shared repository
 
291
            repo = branch.repository
 
292
        branches = [branch]
 
293
    elif repo is not None:
 
294
        branches = repo.find_branches(using=True)
 
295
 
 
296
    if repo is not None:
 
297
        repo.lock_read()
 
298
        try:
 
299
            if do_repo:
 
300
                note("Checking repository at '%s'."
 
301
                     % (repo.bzrdir.root_transport.base,))
 
302
                result = repo.check()
433
303
                result.report_results(verbose)
434
 
        else:
435
 
            if do_tree:
436
 
                note("No working tree found at specified location.")
437
304
            if do_branch:
438
 
                note("No branch found at specified location.")
439
 
            if do_repo:
440
 
                note("No repository found at specified location.")
441
 
    finally:
442
 
        for thing in to_unlock:
443
 
            thing.unlock()
 
305
                if branches == []:
 
306
                    log_error("No branch found at specified location.")
 
307
                else:
 
308
                    for branch in branches:
 
309
                        note("Checking branch at '%s'."
 
310
                             % (branch.bzrdir.root_transport.base,))
 
311
                        check_branch(branch, verbose)
 
312
        finally:
 
313
            repo.unlock()
 
314
    else:
 
315
        if do_branch:
 
316
            log_error("No branch found at specified location.")
 
317
        if do_repo:
 
318
            log_error("No repository found at specified location.")