~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/check.py

  • Committer: John Arbash Meinel
  • Date: 2007-04-12 21:33:07 UTC
  • mfrom: (2413.4.1 api-doc-builders)
  • mto: This revision was merged to the branch mainline in revision 2566.
  • Revision ID: john@arbash-meinel.com-20070412213307-kuh07cnzaud12wx1
[merge] api-doc-builder and remove the pydoctor build code for now.

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
 
    )
54
 
from bzrlib.branch import Branch
55
 
from bzrlib.bzrdir import BzrDir
56
 
from bzrlib.revision import NULL_REVISION
 
35
from bzrlib.errors import BzrCheckError
 
36
import bzrlib.ui
57
37
from bzrlib.trace import note
58
 
from bzrlib.workingtree import WorkingTree
59
 
from bzrlib.i18n import gettext
60
38
 
61
39
class Check(object):
62
40
    """Check a repository"""
63
41
 
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
42
    # The Check object interacts with InventoryEntry.check, etc.
75
43
 
76
 
    def __init__(self, repository, check_repo=True):
 
44
    def __init__(self, repository):
77
45
        self.repository = repository
 
46
        self.checked_text_cnt = 0
78
47
        self.checked_rev_cnt = 0
79
 
        self.ghosts = set()
 
48
        self.ghosts = []
 
49
        self.repeated_text_cnt = 0
80
50
        self.missing_parent_links = {}
81
51
        self.missing_inventory_sha_cnt = 0
82
52
        self.missing_revision_cnt = 0
83
 
        self.checked_weaves = set()
84
 
        self.unreferenced_versions = set()
85
 
        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 = {}
 
53
        # maps (file-id, version) -> sha1; used by InventoryFile._check
 
54
        self.checked_texts = {}
 
55
        self.checked_weaves = {}
99
56
 
100
 
    def check(self, callback_refs=None, check_repo=True):
101
 
        if callback_refs is None:
102
 
            callback_refs = {}
 
57
    def check(self):
103
58
        self.repository.lock_read()
104
 
        self.progress = ui.ui_factory.nested_progress_bar()
 
59
        self.progress = bzrlib.ui.ui_factory.nested_progress_bar()
105
60
        try:
106
 
            self.progress.update(gettext('check'), 0, 4)
107
 
            if self.check_repo:
108
 
                self.progress.update(gettext('checking revisions'), 0)
109
 
                self.check_revisions()
110
 
                self.progress.update(gettext('checking commit contents'), 1)
111
 
                self.repository._check_inventories(self)
112
 
                self.progress.update(gettext('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(gettext('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))
 
61
            self.progress.update('retrieving inventory', 0, 0)
 
62
            # do not put in init, as it should be done with progess,
 
63
            # and inside the lock.
 
64
            self.inventory_weave = self.repository.get_inventory_weave()
 
65
            self.plan_revisions()
 
66
            revno = 0
 
67
            self.check_weaves()
 
68
            while revno < len(self.planned_revisions):
 
69
                rev_id = self.planned_revisions[revno]
 
70
                self.progress.update('checking revision', revno,
 
71
                                     len(self.planned_revisions))
 
72
                revno += 1
 
73
                self.check_one_rev(rev_id)
158
74
        finally:
159
75
            self.progress.finished()
160
76
            self.repository.unlock()
161
77
 
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.
 
78
    def plan_revisions(self):
 
79
        repository = self.repository
 
80
        self.planned_revisions = set(repository.all_revision_ids())
 
81
        self.progress.clear()
 
82
        inventoried = set(self.inventory_weave.versions())
 
83
        awol = self.planned_revisions - inventoried
 
84
        if len(awol) > 0:
 
85
            raise BzrCheckError('Stored revisions missing from inventory'
 
86
                '{%s}' % ','.join([f for f in awol]))
174
87
        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.
188
 
            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)
195
88
 
196
89
    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
 
        note(gettext('checked repository {0} format {1}').format(
204
 
            self.repository.user_url,
205
 
            self.repository._format))
206
 
        note(gettext('%6d revisions'), self.checked_rev_cnt)
207
 
        note(gettext('%6d file-ids'), len(self.checked_weaves))
208
 
        if verbose:
209
 
            note(gettext('%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(gettext('unreferenced version: {{{0}}} in {1}').format(revision_id,
214
 
                        file_id))
 
90
        note('checked repository %s format %s',
 
91
             self.repository.bzrdir.root_transport,
 
92
             self.repository._format)
 
93
        note('%6d revisions', self.checked_rev_cnt)
 
94
        note('%6d unique file texts', self.checked_text_cnt)
 
95
        note('%6d repeated file texts', self.repeated_text_cnt)
 
96
        note('%6d weaves', len(self.checked_weaves))
215
97
        if self.missing_inventory_sha_cnt:
216
 
            note(gettext('%6d revisions are missing inventory_sha1'),
 
98
            note('%6d revisions are missing inventory_sha1',
217
99
                 self.missing_inventory_sha_cnt)
218
100
        if self.missing_revision_cnt:
219
 
            note(gettext('%6d revisions are mentioned but not present'),
 
101
            note('%6d revisions are mentioned but not present',
220
102
                 self.missing_revision_cnt)
221
103
        if len(self.ghosts):
222
 
            note(gettext('%6d ghost revisions'), len(self.ghosts))
 
104
            note('%6d ghost revisions', len(self.ghosts))
223
105
            if verbose:
224
106
                for ghost in self.ghosts:
225
107
                    note('      %s', ghost)
226
108
        if len(self.missing_parent_links):
227
 
            note(gettext('%6d revisions missing parents in ancestry'),
 
109
            note('%6d revisions missing parents in ancestry',
228
110
                 len(self.missing_parent_links))
229
111
            if verbose:
230
112
                for link, linkers in self.missing_parent_links.items():
231
 
                    note(gettext('      %s should be in the ancestry for:'), link)
 
113
                    note('      %s should be in the ancestry for:', link)
232
114
                    for linker in linkers:
233
115
                        note('       * %s', linker)
234
 
        if len(self.inconsistent_parents):
235
 
            note(gettext('%6d inconsistent parents'), len(self.inconsistent_parents))
236
 
            if verbose:
237
 
                for info in self.inconsistent_parents:
238
 
                    revision_id, file_id, found_parents, correct_parents = info
239
 
                    note(gettext('      * {0} version {1} has parents {2!r} '
240
 
                         'but should have {3!r}').format(
241
 
                         file_id, revision_id, found_parents,
242
 
                             correct_parents))
243
 
        if self.revs_with_bad_parents_in_index:
244
 
            note(gettext(
245
 
                 '%6d revisions have incorrect parents in the revision index'),
246
 
                 len(self.revs_with_bad_parents_in_index))
247
 
            if verbose:
248
 
                for item in self.revs_with_bad_parents_in_index:
249
 
                    revision_id, index_parents, actual_parents = item
250
 
                    note(gettext(
251
 
                        '       {0} has wrong parents in index: '
252
 
                        '{1!r} should be {2!r}').format(
253
 
                        revision_id, index_parents, actual_parents))
254
 
        for item in self._report_items:
255
 
            note(item)
256
 
 
257
 
    def _check_one_rev(self, rev_id, rev):
258
 
        """Cross-check one revision.
259
 
 
260
 
        :param rev_id: A revision id to check.
261
 
        :param rev: A revision or None to indicate a missing revision.
 
116
 
 
117
    def check_one_rev(self, rev_id):
 
118
        """Check one revision.
 
119
 
 
120
        rev_id - the one to check
262
121
        """
 
122
        rev = self.repository.get_revision(rev_id)
 
123
                
263
124
        if rev.revision_id != rev_id:
264
 
            self._report_items.append(gettext(
265
 
                'Mismatched internal revid {{{0}}} and index revid {{{1}}}').format(
266
 
                rev.revision_id, rev_id))
267
 
            rev_id = rev.revision_id
268
 
        # Check this revision tree etc, and count as seen when we encounter a
269
 
        # reference to it.
270
 
        self.planned_revisions.add(rev_id)
271
 
        # It is not a ghost
272
 
        self.ghosts.discard(rev_id)
273
 
        # Count all parents as ghosts if we haven't seen them yet.
 
125
            raise BzrCheckError('wrong internal revision id in revision {%s}'
 
126
                                % rev_id)
 
127
 
274
128
        for parent in rev.parent_ids:
275
129
            if not parent in self.planned_revisions:
276
 
                self.ghosts.add(parent)
277
 
        
278
 
        self.ancestors[rev_id] = tuple(rev.parent_ids) or (NULL_REVISION,)
279
 
        self.add_pending_item(rev_id, ('inventories', rev_id), 'inventory',
280
 
            rev.inventory_sha1)
 
130
                missing_links = self.missing_parent_links.get(parent, [])
 
131
                missing_links.append(rev_id)
 
132
                self.missing_parent_links[parent] = missing_links
 
133
                # list based so somewhat slow,
 
134
                # TODO have a planned_revisions list and set.
 
135
                if self.repository.has_revision(parent):
 
136
                    missing_ancestry = self.repository.get_ancestry(parent)
 
137
                    for missing in missing_ancestry:
 
138
                        if (missing is not None 
 
139
                            and missing not in self.planned_revisions):
 
140
                            self.planned_revisions.append(missing)
 
141
                else:
 
142
                    self.ghosts.append(rev_id)
 
143
 
 
144
        if rev.inventory_sha1:
 
145
            inv_sha1 = self.repository.get_inventory_sha1(rev_id)
 
146
            if inv_sha1 != rev.inventory_sha1:
 
147
                raise BzrCheckError('Inventory sha1 hash doesn\'t match'
 
148
                    ' value in revision {%s}' % rev_id)
 
149
        self._check_revision_tree(rev_id)
281
150
        self.checked_rev_cnt += 1
282
151
 
283
 
    def add_pending_item(self, referer, key, kind, sha1):
284
 
        """Add a reference to a sha1 to be cross checked against a key.
285
 
 
286
 
        :param referer: The referer that expects key to have sha1.
287
 
        :param key: A storage key e.g. ('texts', 'foo@bar-20040504-1234')
288
 
        :param kind: revision/inventory/text/map/signature
289
 
        :param sha1: A hex sha1 or None if no sha1 is known.
290
 
        """
291
 
        existing = self.pending_keys.get(key)
292
 
        if existing:
293
 
            if sha1 != existing[1]:
294
 
                self._report_items.append(gettext('Multiple expected sha1s for {0}. {{{1}}}'
295
 
                    ' expects {{{2}}}, {{{3}}} expects {{{4}}}').format(
296
 
                    key, referer, sha1, existing[1], existing[0]))
297
 
        else:
298
 
            self.pending_keys[key] = (kind, sha1, referer)
299
 
 
300
152
    def check_weaves(self):
301
153
        """Check all the weaves we can get our hands on.
302
154
        """
 
155
        n_weaves = 1
303
156
        weave_ids = []
304
 
        storebar = ui.ui_factory.nested_progress_bar()
305
 
        try:
306
 
            self._check_weaves(storebar)
307
 
        finally:
308
 
            storebar.finished()
309
 
 
310
 
    def _check_weaves(self, storebar):
311
 
        storebar.update('text-index', 0, 2)
312
 
        if self.repository._format.fast_deltas:
313
 
            # We haven't considered every fileid instance so far.
314
 
            weave_checker = self.repository._get_versioned_file_checker(
315
 
                ancestors=self.ancestors)
316
 
        else:
317
 
            weave_checker = self.repository._get_versioned_file_checker(
318
 
                text_key_references=self.text_key_references,
319
 
                ancestors=self.ancestors)
320
 
        storebar.update('file-graph', 1)
321
 
        result = weave_checker.check_file_version_parents(
322
 
            self.repository.texts)
323
 
        self.checked_weaves = weave_checker.file_ids
324
 
        bad_parents, unused_versions = result
325
 
        bad_parents = bad_parents.items()
326
 
        for text_key, (stored_parents, correct_parents) in bad_parents:
327
 
            # XXX not ready for id join/split operations.
328
 
            weave_id = text_key[0]
329
 
            revision_id = text_key[-1]
330
 
            weave_parents = tuple([parent[-1] for parent in stored_parents])
331
 
            correct_parents = tuple([parent[-1] for parent in correct_parents])
332
 
            self.inconsistent_parents.append(
333
 
                (revision_id, weave_id, weave_parents, correct_parents))
334
 
        self.unreferenced_versions.update(unused_versions)
335
 
 
336
 
    def _add_entry_to_text_key_references(self, inv, entry):
337
 
        if not self.rich_roots and entry.name == '':
338
 
            return
339
 
        key = (entry.file_id, entry.revision)
340
 
        self.text_key_references.setdefault(key, False)
341
 
        if entry.revision == inv.revision_id:
342
 
            self.text_key_references[key] = True
343
 
 
344
 
 
345
 
def scan_branch(branch, needed_refs, to_unlock):
346
 
    """Scan a branch for refs.
347
 
 
348
 
    :param branch:  The branch to schedule for checking.
349
 
    :param needed_refs: Refs we are accumulating.
350
 
    :param to_unlock: The unlock list accumulating.
 
157
        if self.repository.weave_store.listable():
 
158
            weave_ids = list(self.repository.weave_store)
 
159
            n_weaves = len(weave_ids)
 
160
        self.progress.update('checking weave', 0, n_weaves)
 
161
        self.inventory_weave.check(progress_bar=self.progress)
 
162
        for i, weave_id in enumerate(weave_ids):
 
163
            self.progress.update('checking weave', i, n_weaves)
 
164
            w = self.repository.weave_store.get_weave(weave_id,
 
165
                    self.repository.get_transaction())
 
166
            # No progress here, because it looks ugly.
 
167
            w.check()
 
168
            self.checked_weaves[weave_id] = True
 
169
 
 
170
    def _check_revision_tree(self, rev_id):
 
171
        tree = self.repository.revision_tree(rev_id)
 
172
        inv = tree.inventory
 
173
        seen_ids = {}
 
174
        for file_id in inv:
 
175
            if file_id in seen_ids:
 
176
                raise BzrCheckError('duplicated file_id {%s} '
 
177
                                    'in inventory for revision {%s}'
 
178
                                    % (file_id, rev_id))
 
179
            seen_ids[file_id] = True
 
180
        for file_id in inv:
 
181
            ie = inv[file_id]
 
182
            ie.check(self, rev_id, inv, tree)
 
183
        seen_names = {}
 
184
        for path, ie in inv.iter_entries():
 
185
            if path in seen_names:
 
186
                raise BzrCheckError('duplicated path %s '
 
187
                                    'in inventory for revision {%s}'
 
188
                                    % (path, rev_id))
 
189
            seen_names[path] = True
 
190
 
 
191
 
 
192
def check(branch, verbose):
 
193
    """Run consistency checks on a branch.
 
194
    
 
195
    Results are reported through logging.
 
196
    
 
197
    :raise BzrCheckError: if there's a consistency error.
351
198
    """
352
 
    note(gettext("Checking branch at '%s'.") % (branch.base,))
353
199
    branch.lock_read()
354
 
    to_unlock.append(branch)
355
 
    branch_refs = branch._get_check_refs()
356
 
    for ref in branch_refs:
357
 
        reflist = needed_refs.setdefault(ref, [])
358
 
        reflist.append(branch)
359
 
 
360
 
 
361
 
def scan_tree(base_tree, tree, needed_refs, to_unlock):
362
 
    """Scan a tree for refs.
363
 
 
364
 
    :param base_tree: The original tree check opened, used to detect duplicate
365
 
        tree checks.
366
 
    :param tree:  The tree to schedule for checking.
367
 
    :param needed_refs: Refs we are accumulating.
368
 
    :param to_unlock: The unlock list accumulating.
369
 
    """
370
 
    if base_tree is not None and tree.basedir == base_tree.basedir:
371
 
        return
372
 
    note(gettext("Checking working tree at '%s'.") % (tree.basedir,))
373
 
    tree.lock_read()
374
 
    to_unlock.append(tree)
375
 
    tree_refs = tree._get_check_refs()
376
 
    for ref in tree_refs:
377
 
        reflist = needed_refs.setdefault(ref, [])
378
 
        reflist.append(tree)
379
 
 
380
 
 
381
 
def check_dwim(path, verbose, do_branch=False, do_repo=False, do_tree=False):
382
 
    """Check multiple objects.
383
 
 
384
 
    If errors occur they are accumulated and reported as far as possible, and
385
 
    an exception raised at the end of the process.
386
 
    """
387
 
    try:
388
 
        base_tree, branch, repo, relpath = \
389
 
                        BzrDir.open_containing_tree_branch_or_repository(path)
390
 
    except errors.NotBranchError:
391
 
        base_tree = branch = repo = None
392
 
 
393
 
    to_unlock = []
394
 
    needed_refs= {}
395
 
    try:
396
 
        if base_tree is not None:
397
 
            # If the tree is a lightweight checkout we won't see it in
398
 
            # repo.find_branches - add now.
399
 
            if do_tree:
400
 
                scan_tree(None, base_tree, needed_refs, to_unlock)
401
 
            branch = base_tree.branch
402
 
        if branch is not None:
403
 
            # We have a branch
404
 
            if repo is None:
405
 
                # The branch is in a shared repository
406
 
                repo = branch.repository
407
 
        if repo is not None:
408
 
            repo.lock_read()
409
 
            to_unlock.append(repo)
410
 
            branches = repo.find_branches(using=True)
411
 
            saw_tree = False
412
 
            if do_branch or do_tree:
413
 
                for branch in branches:
414
 
                    if do_tree:
415
 
                        try:
416
 
                            tree = branch.bzrdir.open_workingtree()
417
 
                            saw_tree = True
418
 
                        except (errors.NotLocalUrl, errors.NoWorkingTree):
419
 
                            pass
420
 
                        else:
421
 
                            scan_tree(base_tree, tree, needed_refs, to_unlock)
422
 
                    if do_branch:
423
 
                        scan_branch(branch, needed_refs, to_unlock)
424
 
            if do_branch and not branches:
425
 
                note(gettext("No branch found at specified location."))
426
 
            if do_tree and base_tree is None and not saw_tree:
427
 
                note(gettext("No working tree found at specified location."))
428
 
            if do_repo or do_branch or do_tree:
429
 
                if do_repo:
430
 
                    note(gettext("Checking repository at '%s'.")
431
 
                         % (repo.user_url,))
432
 
                result = repo.check(None, callback_refs=needed_refs,
433
 
                    check_repo=do_repo)
434
 
                result.report_results(verbose)
435
 
        else:
436
 
            if do_tree:
437
 
                note(gettext("No working tree found at specified location."))
438
 
            if do_branch:
439
 
                note(gettext("No branch found at specified location."))
440
 
            if do_repo:
441
 
                note(gettext("No repository found at specified location."))
 
200
    try:
 
201
        branch_result = branch.check()
 
202
        repo_result = branch.repository.check([branch.last_revision()])
442
203
    finally:
443
 
        for thing in to_unlock:
444
 
            thing.unlock()
 
204
        branch.unlock()
 
205
    branch_result.report_results(verbose)
 
206
    repo_result.report_results(verbose)