← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/check.py

  • Committer: Jelmer Vernooij
  • Date: 2011-12-16 19:18:39 UTC
  • mto: This revision was merged to the branch mainline in revision 6391.
  • Revision ID: jelmer@samba.org-20111216191839-eg681lxqibi1qxu1
Fix remaining tests.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/check.py
39
39
so that when the dependent object is checked, matches can be pulled out and
40
40
evaluated in-line rather than re-reading the same data many times.
41
41
check_refs are tuples (kind, value). Currently defined kinds are:
 
42
 
42
43
* 'trees', where value is a revid and the looked up objects are revision trees.
43
44
* 'lefthand-distance', where value is a revid and the looked up objects are the
44
45
  distance along the lefthand path to NULL for that revid.
46
47
  indicating that the revision was found/not found.
47
48
"""
48
49
 
49
 
from bzrlib import errors
 
50
from bzrlib import (
 
51
    errors,
 
52
    ui,
 
53
    )
50
54
from bzrlib.branch import Branch
51
 
from bzrlib.bzrdir import BzrDir
 
55
from bzrlib.controldir import ControlDir
52
56
from bzrlib.revision import NULL_REVISION
53
 
from bzrlib.symbol_versioning import deprecated_function, deprecated_in
54
57
from bzrlib.trace import note
55
 
import bzrlib.ui
56
58
from bzrlib.workingtree import WorkingTree
 
59
from bzrlib.i18n import gettext
57
60
 
58
61
class Check(object):
59
62
    """Check a repository"""
60
63
 
 
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
 
61
74
    # The Check object interacts with InventoryEntry.check, etc.
62
75
 
63
76
    def __init__(self, repository, check_repo=True):
88
101
        if callback_refs is None:
89
102
            callback_refs = {}
90
103
        self.repository.lock_read()
91
 
        self.progress = bzrlib.ui.ui_factory.nested_progress_bar()
 
104
        self.progress = ui.ui_factory.nested_progress_bar()
92
105
        try:
93
 
            self.progress.update('check', 0, 4)
 
106
            self.progress.update(gettext('check'), 0, 4)
94
107
            if self.check_repo:
95
 
                self.progress.update('checking revisions', 0)
 
108
                self.progress.update(gettext('checking revisions'), 0)
96
109
                self.check_revisions()
97
 
                self.progress.update('checking commit contents', 1)
 
110
                self.progress.update(gettext('checking commit contents'), 1)
98
111
                self.repository._check_inventories(self)
99
 
                self.progress.update('checking file graphs', 2)
 
112
                self.progress.update(gettext('checking file graphs'), 2)
100
113
                # check_weaves is done after the revision scan so that
101
114
                # revision index is known to be valid.
102
115
                self.check_weaves()
103
 
            self.progress.update('checking branches and trees', 3)
 
116
            self.progress.update(gettext('checking branches and trees'), 3)
104
117
            if callback_refs:
105
118
                repo = self.repository
106
119
                # calculate all refs, and callback the objects requesting them.
170
183
        # - we can fill out existence flags at this point
171
184
        # - we can read the revision inventory sha at this point
172
185
        # - we can check properties and serialisers etc.
173
 
        if not self.repository.revision_graph_can_have_wrong_parents():
 
186
        if not self.repository._format.revision_graph_can_have_wrong_parents:
174
187
            # The check against the index isn't needed.
175
188
            self.revs_with_bad_parents_in_index = None
176
189
            for thing in revision_iterator:
187
200
            result.report_results(verbose)
188
201
 
189
202
    def _report_repo_results(self, verbose):
190
 
        note('checked repository %s format %s',
 
203
        note(gettext('checked repository {0} format {1}').format(
191
204
            self.repository.user_url,
192
 
            self.repository._format)
193
 
        note('%6d revisions', self.checked_rev_cnt)
194
 
        note('%6d file-ids', len(self.checked_weaves))
 
205
            self.repository._format))
 
206
        note(gettext('%6d revisions'), self.checked_rev_cnt)
 
207
        note(gettext('%6d file-ids'), len(self.checked_weaves))
195
208
        if verbose:
196
 
            note('%6d unreferenced text versions',
 
209
            note(gettext('%6d unreferenced text versions'),
197
210
                len(self.unreferenced_versions))
198
211
        if verbose and len(self.unreferenced_versions):
199
212
                for file_id, revision_id in self.unreferenced_versions:
200
 
                    note('unreferenced version: {%s} in %s', revision_id,
201
 
                        file_id)
 
213
                    note(gettext('unreferenced version: {{{0}}} in {1}').format(revision_id,
 
214
                        file_id))
202
215
        if self.missing_inventory_sha_cnt:
203
 
            note('%6d revisions are missing inventory_sha1',
 
216
            note(gettext('%6d revisions are missing inventory_sha1'),
204
217
                 self.missing_inventory_sha_cnt)
205
218
        if self.missing_revision_cnt:
206
 
            note('%6d revisions are mentioned but not present',
 
219
            note(gettext('%6d revisions are mentioned but not present'),
207
220
                 self.missing_revision_cnt)
208
221
        if len(self.ghosts):
209
 
            note('%6d ghost revisions', len(self.ghosts))
 
222
            note(gettext('%6d ghost revisions'), len(self.ghosts))
210
223
            if verbose:
211
224
                for ghost in self.ghosts:
212
225
                    note('      %s', ghost)
213
226
        if len(self.missing_parent_links):
214
 
            note('%6d revisions missing parents in ancestry',
 
227
            note(gettext('%6d revisions missing parents in ancestry'),
215
228
                 len(self.missing_parent_links))
216
229
            if verbose:
217
230
                for link, linkers in self.missing_parent_links.items():
218
 
                    note('      %s should be in the ancestry for:', link)
 
231
                    note(gettext('      %s should be in the ancestry for:'), link)
219
232
                    for linker in linkers:
220
233
                        note('       * %s', linker)
221
234
        if len(self.inconsistent_parents):
222
 
            note('%6d inconsistent parents', len(self.inconsistent_parents))
 
235
            note(gettext('%6d inconsistent parents'), len(self.inconsistent_parents))
223
236
            if verbose:
224
237
                for info in self.inconsistent_parents:
225
238
                    revision_id, file_id, found_parents, correct_parents = info
226
 
                    note('      * %s version %s has parents %r '
227
 
                         'but should have %r'
228
 
                         % (file_id, revision_id, found_parents,
 
239
                    note(gettext('      * {0} version {1} has parents {2!r} '
 
240
                         'but should have {3!r}').format(
 
241
                         file_id, revision_id, found_parents,
229
242
                             correct_parents))
230
243
        if self.revs_with_bad_parents_in_index:
231
 
            note('%6d revisions have incorrect parents in the revision index',
 
244
            note(gettext(
 
245
                 '%6d revisions have incorrect parents in the revision index'),
232
246
                 len(self.revs_with_bad_parents_in_index))
233
247
            if verbose:
234
248
                for item in self.revs_with_bad_parents_in_index:
235
249
                    revision_id, index_parents, actual_parents = item
236
 
                    note(
237
 
                        '       %s has wrong parents in index: '
238
 
                        '%r should be %r',
239
 
                        revision_id, index_parents, actual_parents)
 
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))
240
254
        for item in self._report_items:
241
255
            note(item)
242
256
 
247
261
        :param rev: A revision or None to indicate a missing revision.
248
262
        """
249
263
        if rev.revision_id != rev_id:
250
 
            self._report_items.append(
251
 
                'Mismatched internal revid {%s} and index revid {%s}' % (
 
264
            self._report_items.append(gettext(
 
265
                'Mismatched internal revid {{{0}}} and index revid {{{1}}}').format(
252
266
                rev.revision_id, rev_id))
253
267
            rev_id = rev.revision_id
254
268
        # Check this revision tree etc, and count as seen when we encounter a
277
291
        existing = self.pending_keys.get(key)
278
292
        if existing:
279
293
            if sha1 != existing[1]:
280
 
                self._report_items.append('Multiple expected sha1s for %s. {%s}'
281
 
                    ' expects {%s}, {%s} expects {%s}', (
 
294
                self._report_items.append(gettext('Multiple expected sha1s for {0}. {{{1}}}'
 
295
                    ' expects {{{2}}}, {{{3}}} expects {{{4}}}').format(
282
296
                    key, referer, sha1, existing[1], existing[0]))
283
297
        else:
284
298
            self.pending_keys[key] = (kind, sha1, referer)
287
301
        """Check all the weaves we can get our hands on.
288
302
        """
289
303
        weave_ids = []
290
 
        storebar = bzrlib.ui.ui_factory.nested_progress_bar()
 
304
        storebar = ui.ui_factory.nested_progress_bar()
291
305
        try:
292
306
            self._check_weaves(storebar)
293
307
        finally:
328
342
            self.text_key_references[key] = True
329
343
 
330
344
 
331
 
@deprecated_function(deprecated_in((1,6,0)))
332
 
def check(branch, verbose):
333
 
    """Run consistency checks on a branch.
334
 
 
335
 
    Results are reported through logging.
336
 
 
337
 
    Deprecated in 1.6.  Please use check_dwim instead.
338
 
 
339
 
    :raise BzrCheckError: if there's a consistency error.
340
 
    """
341
 
    check_branch(branch, verbose)
342
 
 
343
 
 
344
 
@deprecated_function(deprecated_in((1,16,0)))
345
 
def check_branch(branch, verbose):
346
 
    """Run consistency checks on a branch.
347
 
 
348
 
    Results are reported through logging.
349
 
 
350
 
    :raise BzrCheckError: if there's a consistency error.
351
 
    """
352
 
    branch.lock_read()
353
 
    try:
354
 
        needed_refs = {}
355
 
        for ref in branch._get_check_refs():
356
 
            needed_refs.setdefault(ref, []).append(branch)
357
 
        result = branch.repository.check([branch.last_revision()], needed_refs)
358
 
        branch_result = result.other_results[0]
359
 
    finally:
360
 
        branch.unlock()
361
 
    branch_result.report_results(verbose)
362
 
 
363
 
 
364
345
def scan_branch(branch, needed_refs, to_unlock):
365
346
    """Scan a branch for refs.
366
347
 
368
349
    :param needed_refs: Refs we are accumulating.
369
350
    :param to_unlock: The unlock list accumulating.
370
351
    """
371
 
    note("Checking branch at '%s'." % (branch.base,))
 
352
    note(gettext("Checking branch at '%s'.") % (branch.base,))
372
353
    branch.lock_read()
373
354
    to_unlock.append(branch)
374
355
    branch_refs = branch._get_check_refs()
388
369
    """
389
370
    if base_tree is not None and tree.basedir == base_tree.basedir:
390
371
        return
391
 
    note("Checking working tree at '%s'." % (tree.basedir,))
 
372
    note(gettext("Checking working tree at '%s'.") % (tree.basedir,))
392
373
    tree.lock_read()
393
374
    to_unlock.append(tree)
394
375
    tree_refs = tree._get_check_refs()
405
386
    """
406
387
    try:
407
388
        base_tree, branch, repo, relpath = \
408
 
                        BzrDir.open_containing_tree_branch_or_repository(path)
 
389
                        ControlDir.open_containing_tree_branch_or_repository(path)
409
390
    except errors.NotBranchError:
410
391
        base_tree = branch = repo = None
411
392
 
441
422
                    if do_branch:
442
423
                        scan_branch(branch, needed_refs, to_unlock)
443
424
            if do_branch and not branches:
444
 
                note("No branch found at specified location.")
 
425
                note(gettext("No branch found at specified location."))
445
426
            if do_tree and base_tree is None and not saw_tree:
446
 
                note("No working tree found at specified location.")
 
427
                note(gettext("No working tree found at specified location."))
447
428
            if do_repo or do_branch or do_tree:
448
429
                if do_repo:
449
 
                    note("Checking repository at '%s'."
 
430
                    note(gettext("Checking repository at '%s'.")
450
431
                         % (repo.user_url,))
451
432
                result = repo.check(None, callback_refs=needed_refs,
452
433
                    check_repo=do_repo)
453
434
                result.report_results(verbose)
454
435
        else:
455
436
            if do_tree:
456
 
                note("No working tree found at specified location.")
 
437
                note(gettext("No working tree found at specified location."))
457
438
            if do_branch:
458
 
                note("No branch found at specified location.")
 
439
                note(gettext("No branch found at specified location."))
459
440
            if do_repo:
460
 
                note("No repository found at specified location.")
 
441
                note(gettext("No repository found at specified location."))
461
442
    finally:
462
443
        for thing in to_unlock:
463
444
            thing.unlock()