~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/check.py

  • Committer: Vincent Ladeuil
  • Date: 2014-04-09 08:00:39 UTC
  • mto: This revision was merged to the branch mainline in revision 6593.
  • Revision ID: v.ladeuil+lp@free.fr-20140409080039-zq5y8wu59qc8bgnz
Fix minor incompatible change in email python 2.7.6 module.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2004, 2005 by Martin Pool
2
 
# Copyright (C) 2005 by Canonical Ltd
3
 
 
 
1
# Copyright (C) 2005, 2006 Canonical Ltd
 
2
#
4
3
# This program is free software; you can redistribute it and/or modify
5
4
# it under the terms of the GNU General Public License as published by
6
5
# the Free Software Foundation; either version 2 of the License, or
7
6
# (at your option) any later version.
8
 
 
 
7
#
9
8
# This program is distributed in the hope that it will be useful,
10
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
11
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
12
11
# GNU General Public License for more details.
13
 
 
 
12
#
14
13
# You should have received a copy of the GNU General Public License
15
14
# along with this program; if not, write to the Free Software
16
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
17
 
 
18
 
 
19
 
 
20
 
 
21
 
def check(branch, update=False):
22
 
    """Run consistency checks on a branch.
23
 
 
24
 
    If update is True, for revisions missing certain information
25
 
    (right now this is inventory_sha1 and revision_sha1),
26
 
    update them to include the appropriate values.
27
 
    """
28
 
    import sys
29
 
 
30
 
    from bzrlib.trace import mutter
31
 
    from bzrlib.errors import BzrCheckError
32
 
    from bzrlib.osutils import fingerprint_file
33
 
    from bzrlib.progress import ProgressBar
34
 
 
35
 
    if update:
36
 
        branch.lock_write()
37
 
    else:
38
 
        branch.lock_read()
39
 
 
40
 
    try:
41
 
 
42
 
        out = sys.stdout
43
 
 
44
 
        pb = ProgressBar(show_spinner=True)
45
 
        last_ptr = None
46
 
        checked_revs = {}
47
 
 
48
 
        missing_inventory_sha_cnt = 0
49
 
 
50
 
        history = branch.revision_history()
51
 
        revno = 0
52
 
        revcount = len(history)
53
 
 
54
 
        checked_texts = {}
55
 
 
56
 
        updated_revisions = []
57
 
 
58
 
        # Set to True in the case that the previous revision entry
59
 
        # was updated, since this will affect future revision entries
60
 
        updated_previous_revision = False
61
 
 
62
 
        for rid in history:
63
 
            revno += 1
64
 
            pb.update('checking revision', revno, revcount)
65
 
            mutter('    revision {%s}' % rid)
66
 
            rev = branch.get_revision(rid)
67
 
            if rev.revision_id != rid:
68
 
                raise BzrCheckError('wrong internal revision id in revision {%s}' % rid)
69
 
            if rev.precursor != last_ptr:
70
 
                raise BzrCheckError('mismatched precursor in revision {%s}' % rid)
71
 
            last_ptr = rid
72
 
            if rid in checked_revs:
73
 
                raise BzrCheckError('repeated revision {%s}' % rid)
74
 
            checked_revs[rid] = True
75
 
 
76
 
            ## TODO: Check all the required fields are present on the revision.
77
 
 
78
 
            updated = False
79
 
            if rev.inventory_sha1:
80
 
                #mutter('    checking inventory hash {%s}' % rev.inventory_sha1)
81
 
                inv_sha1 = branch.get_inventory_sha1(rev.inventory_id)
82
 
                if inv_sha1 != rev.inventory_sha1:
83
 
                    raise BzrCheckError('Inventory sha1 hash doesn\'t match'
84
 
                        ' value in revision {%s}' % rid)
85
 
            elif update:
86
 
                inv_sha1 = branch.get_inventory_sha1(rev.inventory_id)
87
 
                rev.inventory_sha1 = inv_sha1
88
 
                updated = True
89
 
            else:
90
 
                missing_inventory_sha_cnt += 1
91
 
                mutter("no inventory_sha1 on revision {%s}" % rid)
92
 
 
93
 
            if rev.precursor:
94
 
                if rev.precursor_sha1:
95
 
                    precursor_sha1 = branch.get_revision_sha1(rev.precursor)
96
 
                    if updated_previous_revision: 
97
 
                        # we don't expect the hashes to match, because
98
 
                        # we had to modify the previous revision_history entry.
99
 
                        rev.precursor_sha1 = precursor_sha1
100
 
                        updated = True
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
 
 
17
# TODO: Check ancestries are correct for every revision: includes
 
18
# every committed so far, and in a reasonable order.
 
19
 
 
20
# TODO: Also check non-mainline revisions mentioned as parents.
 
21
 
 
22
# TODO: Check for extra files in the control directory.
 
23
 
 
24
# TODO: Check revision, inventory and entry objects have all
 
25
# required fields.
 
26
 
 
27
# TODO: Get every revision in the revision-store even if they're not
 
28
# referenced by history and make sure they're all valid.
 
29
 
 
30
# TODO: Perhaps have a way to record errors other than by raising exceptions;
 
31
# would perhaps be enough to accumulate exception objects in a list without
 
32
# raising them.  If there's more than one exception it'd be good to see them
 
33
# all.
 
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 __future__ import absolute_import
 
51
 
 
52
from bzrlib import (
 
53
    errors,
 
54
    ui,
 
55
    )
 
56
from bzrlib.branch import Branch
 
57
from bzrlib.controldir import ControlDir
 
58
from bzrlib.revision import NULL_REVISION
 
59
from bzrlib.trace import note
 
60
from bzrlib.workingtree import WorkingTree
 
61
from bzrlib.i18n import gettext
 
62
 
 
63
class Check(object):
 
64
    """Check a repository"""
 
65
 
 
66
    def __init__(self, repository, check_repo=True):
 
67
        self.repository = repository
 
68
 
 
69
    def report_results(self, verbose):
 
70
        raise NotImplementedError(self.report_results)
 
71
 
 
72
 
 
73
class VersionedFileCheck(Check):
 
74
    """Check a versioned file repository"""
 
75
 
 
76
    # The Check object interacts with InventoryEntry.check, etc.
 
77
 
 
78
    def __init__(self, repository, check_repo=True):
 
79
        self.repository = repository
 
80
        self.checked_rev_cnt = 0
 
81
        self.ghosts = set()
 
82
        self.missing_parent_links = {}
 
83
        self.missing_inventory_sha_cnt = 0
 
84
        self.missing_revision_cnt = 0
 
85
        self.checked_weaves = set()
 
86
        self.unreferenced_versions = set()
 
87
        self.inconsistent_parents = []
 
88
        self.rich_roots = repository.supports_rich_root()
 
89
        self.text_key_references = {}
 
90
        self.check_repo = check_repo
 
91
        self.other_results = []
 
92
        # Plain text lines to include in the report
 
93
        self._report_items = []
 
94
        # Keys we are looking for; may be large and need spilling to disk.
 
95
        # key->(type(revision/inventory/text/signature/map), sha1, first-referer)
 
96
        self.pending_keys = {}
 
97
        # Ancestors map for all of revisions being checked; while large helper
 
98
        # functions we call would create it anyway, so better to have once and
 
99
        # keep.
 
100
        self.ancestors = {}
 
101
 
 
102
    def check(self, callback_refs=None, check_repo=True):
 
103
        if callback_refs is None:
 
104
            callback_refs = {}
 
105
        self.repository.lock_read()
 
106
        self.progress = ui.ui_factory.nested_progress_bar()
 
107
        try:
 
108
            self.progress.update(gettext('check'), 0, 4)
 
109
            if self.check_repo:
 
110
                self.progress.update(gettext('checking revisions'), 0)
 
111
                self.check_revisions()
 
112
                self.progress.update(gettext('checking commit contents'), 1)
 
113
                self.repository._check_inventories(self)
 
114
                self.progress.update(gettext('checking file graphs'), 2)
 
115
                # check_weaves is done after the revision scan so that
 
116
                # revision index is known to be valid.
 
117
                self.check_weaves()
 
118
            self.progress.update(gettext('checking branches and trees'), 3)
 
119
            if callback_refs:
 
120
                repo = self.repository
 
121
                # calculate all refs, and callback the objects requesting them.
 
122
                refs = {}
 
123
                wanting_items = set()
 
124
                # Current crude version calculates everything and calls
 
125
                # everything at once. Doing a queue and popping as things are
 
126
                # satisfied would be cheaper on memory [but few people have
 
127
                # huge numbers of working trees today. TODO: fix before
 
128
                # landing].
 
129
                distances = set()
 
130
                existences = set()
 
131
                for ref, wantlist in callback_refs.iteritems():
 
132
                    wanting_items.update(wantlist)
 
133
                    kind, value = ref
 
134
                    if kind == 'trees':
 
135
                        refs[ref] = repo.revision_tree(value)
 
136
                    elif kind == 'lefthand-distance':
 
137
                        distances.add(value)
 
138
                    elif kind == 'revision-existence':
 
139
                        existences.add(value)
101
140
                    else:
102
 
                        #mutter('    checking precursor hash {%s}' % rev.precursor_sha1)
103
 
                        if rev.precursor_sha1 != precursor_sha1:
104
 
                            raise BzrCheckError('Precursor sha1 hash doesn\'t match'
105
 
                                ' value in revision {%s}' % rid)
106
 
                elif update:
107
 
                    precursor_sha1 = branch.get_revision_sha1(rev.precursor)
108
 
                    rev.precursor_sha1 = precursor_sha1
109
 
                    updated = True
110
 
 
111
 
            if updated:
112
 
                updated_previous_revision = True
113
 
                # We had to update this revision entries hashes
114
 
                # Now we need to write out a new value
115
 
                # This is a little bit invasive, since we are *rewriting* a
116
 
                # revision entry. I'm not supremely happy about it, but
117
 
                # there should be *some* way of making old entries have
118
 
                # the full meta information.
119
 
                import tempfile, os, errno
120
 
                rev_tmp = tempfile.TemporaryFile()
121
 
                rev.write_xml(rev_tmp)
122
 
                rev_tmp.seek(0)
123
 
 
124
 
                tmpfd, tmp_path = tempfile.mkstemp(prefix=rid, suffix='.gz',
125
 
                    dir=branch.controlfilename('revision-store'))
126
 
                os.close(tmpfd)
127
 
                def special_rename(p1, p2):
128
 
                    if sys.platform == 'win32':
 
141
                        raise AssertionError(
 
142
                            'unknown ref kind for ref %s' % ref)
 
143
                node_distances = repo.get_graph().find_lefthand_distances(distances)
 
144
                for key, distance in node_distances.iteritems():
 
145
                    refs[('lefthand-distance', key)] = distance
 
146
                    if key in existences and distance > 0:
 
147
                        refs[('revision-existence', key)] = True
 
148
                        existences.remove(key)
 
149
                parent_map = repo.get_graph().get_parent_map(existences)
 
150
                for key in parent_map:
 
151
                    refs[('revision-existence', key)] = True
 
152
                    existences.remove(key)
 
153
                for key in existences:
 
154
                    refs[('revision-existence', key)] = False
 
155
                for item in wanting_items:
 
156
                    if isinstance(item, WorkingTree):
 
157
                        item._check(refs)
 
158
                    if isinstance(item, Branch):
 
159
                        self.other_results.append(item.check(refs))
 
160
        finally:
 
161
            self.progress.finished()
 
162
            self.repository.unlock()
 
163
 
 
164
    def _check_revisions(self, revisions_iterator):
 
165
        """Check revision objects by decorating a generator.
 
166
 
 
167
        :param revisions_iterator: An iterator of(revid, Revision-or-None).
 
168
        :return: A generator of the contents of revisions_iterator.
 
169
        """
 
170
        self.planned_revisions = set()
 
171
        for revid, revision in revisions_iterator:
 
172
            yield revid, revision
 
173
            self._check_one_rev(revid, revision)
 
174
        # Flatten the revisions we found to guarantee consistent later
 
175
        # iteration.
 
176
        self.planned_revisions = list(self.planned_revisions)
 
177
        # TODO: extract digital signatures as items to callback on too.
 
178
 
 
179
    def check_revisions(self):
 
180
        """Scan revisions, checking data directly available as we go."""
 
181
        revision_iterator = self.repository._iter_revisions(None)
 
182
        revision_iterator = self._check_revisions(revision_iterator)
 
183
        # We read the all revisions here:
 
184
        # - doing this allows later code to depend on the revision index.
 
185
        # - we can fill out existence flags at this point
 
186
        # - we can read the revision inventory sha at this point
 
187
        # - we can check properties and serialisers etc.
 
188
        if not self.repository._format.revision_graph_can_have_wrong_parents:
 
189
            # The check against the index isn't needed.
 
190
            self.revs_with_bad_parents_in_index = None
 
191
            for thing in revision_iterator:
 
192
                pass
 
193
        else:
 
194
            bad_revisions = self.repository._find_inconsistent_revision_parents(
 
195
                revision_iterator)
 
196
            self.revs_with_bad_parents_in_index = list(bad_revisions)
 
197
 
 
198
    def report_results(self, verbose):
 
199
        if self.check_repo:
 
200
            self._report_repo_results(verbose)
 
201
        for result in self.other_results:
 
202
            result.report_results(verbose)
 
203
 
 
204
    def _report_repo_results(self, verbose):
 
205
        note(gettext('checked repository {0} format {1}').format(
 
206
            self.repository.user_url,
 
207
            self.repository._format))
 
208
        note(gettext('%6d revisions'), self.checked_rev_cnt)
 
209
        note(gettext('%6d file-ids'), len(self.checked_weaves))
 
210
        if verbose:
 
211
            note(gettext('%6d unreferenced text versions'),
 
212
                len(self.unreferenced_versions))
 
213
        if verbose and len(self.unreferenced_versions):
 
214
                for file_id, revision_id in self.unreferenced_versions:
 
215
                    note(gettext('unreferenced version: {{{0}}} in {1}').format(revision_id,
 
216
                        file_id))
 
217
        if self.missing_inventory_sha_cnt:
 
218
            note(gettext('%6d revisions are missing inventory_sha1'),
 
219
                 self.missing_inventory_sha_cnt)
 
220
        if self.missing_revision_cnt:
 
221
            note(gettext('%6d revisions are mentioned but not present'),
 
222
                 self.missing_revision_cnt)
 
223
        if len(self.ghosts):
 
224
            note(gettext('%6d ghost revisions'), len(self.ghosts))
 
225
            if verbose:
 
226
                for ghost in self.ghosts:
 
227
                    note('      %s', ghost)
 
228
        if len(self.missing_parent_links):
 
229
            note(gettext('%6d revisions missing parents in ancestry'),
 
230
                 len(self.missing_parent_links))
 
231
            if verbose:
 
232
                for link, linkers in self.missing_parent_links.items():
 
233
                    note(gettext('      %s should be in the ancestry for:'), link)
 
234
                    for linker in linkers:
 
235
                        note('       * %s', linker)
 
236
        if len(self.inconsistent_parents):
 
237
            note(gettext('%6d inconsistent parents'), len(self.inconsistent_parents))
 
238
            if verbose:
 
239
                for info in self.inconsistent_parents:
 
240
                    revision_id, file_id, found_parents, correct_parents = info
 
241
                    note(gettext('      * {0} version {1} has parents {2!r} '
 
242
                         'but should have {3!r}').format(
 
243
                         file_id, revision_id, found_parents,
 
244
                             correct_parents))
 
245
        if self.revs_with_bad_parents_in_index:
 
246
            note(gettext(
 
247
                 '%6d revisions have incorrect parents in the revision index'),
 
248
                 len(self.revs_with_bad_parents_in_index))
 
249
            if verbose:
 
250
                for item in self.revs_with_bad_parents_in_index:
 
251
                    revision_id, index_parents, actual_parents = item
 
252
                    note(gettext(
 
253
                        '       {0} has wrong parents in index: '
 
254
                        '{1!r} should be {2!r}').format(
 
255
                        revision_id, index_parents, actual_parents))
 
256
        for item in self._report_items:
 
257
            note(item)
 
258
 
 
259
    def _check_one_rev(self, rev_id, rev):
 
260
        """Cross-check one revision.
 
261
 
 
262
        :param rev_id: A revision id to check.
 
263
        :param rev: A revision or None to indicate a missing revision.
 
264
        """
 
265
        if rev.revision_id != rev_id:
 
266
            self._report_items.append(gettext(
 
267
                'Mismatched internal revid {{{0}}} and index revid {{{1}}}').format(
 
268
                rev.revision_id, rev_id))
 
269
            rev_id = rev.revision_id
 
270
        # Check this revision tree etc, and count as seen when we encounter a
 
271
        # reference to it.
 
272
        self.planned_revisions.add(rev_id)
 
273
        # It is not a ghost
 
274
        self.ghosts.discard(rev_id)
 
275
        # Count all parents as ghosts if we haven't seen them yet.
 
276
        for parent in rev.parent_ids:
 
277
            if not parent in self.planned_revisions:
 
278
                self.ghosts.add(parent)
 
279
        
 
280
        self.ancestors[rev_id] = tuple(rev.parent_ids) or (NULL_REVISION,)
 
281
        self.add_pending_item(rev_id, ('inventories', rev_id), 'inventory',
 
282
            rev.inventory_sha1)
 
283
        self.checked_rev_cnt += 1
 
284
 
 
285
    def add_pending_item(self, referer, key, kind, sha1):
 
286
        """Add a reference to a sha1 to be cross checked against a key.
 
287
 
 
288
        :param referer: The referer that expects key to have sha1.
 
289
        :param key: A storage key e.g. ('texts', 'foo@bar-20040504-1234')
 
290
        :param kind: revision/inventory/text/map/signature
 
291
        :param sha1: A hex sha1 or None if no sha1 is known.
 
292
        """
 
293
        existing = self.pending_keys.get(key)
 
294
        if existing:
 
295
            if sha1 != existing[1]:
 
296
                self._report_items.append(gettext('Multiple expected sha1s for {0}. {{{1}}}'
 
297
                    ' expects {{{2}}}, {{{3}}} expects {{{4}}}').format(
 
298
                    key, referer, sha1, existing[1], existing[0]))
 
299
        else:
 
300
            self.pending_keys[key] = (kind, sha1, referer)
 
301
 
 
302
    def check_weaves(self):
 
303
        """Check all the weaves we can get our hands on.
 
304
        """
 
305
        weave_ids = []
 
306
        storebar = ui.ui_factory.nested_progress_bar()
 
307
        try:
 
308
            self._check_weaves(storebar)
 
309
        finally:
 
310
            storebar.finished()
 
311
 
 
312
    def _check_weaves(self, storebar):
 
313
        storebar.update('text-index', 0, 2)
 
314
        if self.repository._format.fast_deltas:
 
315
            # We haven't considered every fileid instance so far.
 
316
            weave_checker = self.repository._get_versioned_file_checker(
 
317
                ancestors=self.ancestors)
 
318
        else:
 
319
            weave_checker = self.repository._get_versioned_file_checker(
 
320
                text_key_references=self.text_key_references,
 
321
                ancestors=self.ancestors)
 
322
        storebar.update('file-graph', 1)
 
323
        result = weave_checker.check_file_version_parents(
 
324
            self.repository.texts)
 
325
        self.checked_weaves = weave_checker.file_ids
 
326
        bad_parents, unused_versions = result
 
327
        bad_parents = bad_parents.items()
 
328
        for text_key, (stored_parents, correct_parents) in bad_parents:
 
329
            # XXX not ready for id join/split operations.
 
330
            weave_id = text_key[0]
 
331
            revision_id = text_key[-1]
 
332
            weave_parents = tuple([parent[-1] for parent in stored_parents])
 
333
            correct_parents = tuple([parent[-1] for parent in correct_parents])
 
334
            self.inconsistent_parents.append(
 
335
                (revision_id, weave_id, weave_parents, correct_parents))
 
336
        self.unreferenced_versions.update(unused_versions)
 
337
 
 
338
    def _add_entry_to_text_key_references(self, inv, entry):
 
339
        if not self.rich_roots and entry.name == '':
 
340
            return
 
341
        key = (entry.file_id, entry.revision)
 
342
        self.text_key_references.setdefault(key, False)
 
343
        if entry.revision == inv.revision_id:
 
344
            self.text_key_references[key] = True
 
345
 
 
346
 
 
347
def scan_branch(branch, needed_refs, to_unlock):
 
348
    """Scan a branch for refs.
 
349
 
 
350
    :param branch:  The branch to schedule for checking.
 
351
    :param needed_refs: Refs we are accumulating.
 
352
    :param to_unlock: The unlock list accumulating.
 
353
    """
 
354
    note(gettext("Checking branch at '%s'.") % (branch.base,))
 
355
    branch.lock_read()
 
356
    to_unlock.append(branch)
 
357
    branch_refs = branch._get_check_refs()
 
358
    for ref in branch_refs:
 
359
        reflist = needed_refs.setdefault(ref, [])
 
360
        reflist.append(branch)
 
361
 
 
362
 
 
363
def scan_tree(base_tree, tree, needed_refs, to_unlock):
 
364
    """Scan a tree for refs.
 
365
 
 
366
    :param base_tree: The original tree check opened, used to detect duplicate
 
367
        tree checks.
 
368
    :param tree:  The tree to schedule for checking.
 
369
    :param needed_refs: Refs we are accumulating.
 
370
    :param to_unlock: The unlock list accumulating.
 
371
    """
 
372
    if base_tree is not None and tree.basedir == base_tree.basedir:
 
373
        return
 
374
    note(gettext("Checking working tree at '%s'.") % (tree.basedir,))
 
375
    tree.lock_read()
 
376
    to_unlock.append(tree)
 
377
    tree_refs = tree._get_check_refs()
 
378
    for ref in tree_refs:
 
379
        reflist = needed_refs.setdefault(ref, [])
 
380
        reflist.append(tree)
 
381
 
 
382
 
 
383
def check_dwim(path, verbose, do_branch=False, do_repo=False, do_tree=False):
 
384
    """Check multiple objects.
 
385
 
 
386
    If errors occur they are accumulated and reported as far as possible, and
 
387
    an exception raised at the end of the process.
 
388
    """
 
389
    try:
 
390
        base_tree, branch, repo, relpath = \
 
391
                        ControlDir.open_containing_tree_branch_or_repository(path)
 
392
    except errors.NotBranchError:
 
393
        base_tree = branch = repo = None
 
394
 
 
395
    to_unlock = []
 
396
    needed_refs= {}
 
397
    try:
 
398
        if base_tree is not None:
 
399
            # If the tree is a lightweight checkout we won't see it in
 
400
            # repo.find_branches - add now.
 
401
            if do_tree:
 
402
                scan_tree(None, base_tree, needed_refs, to_unlock)
 
403
            branch = base_tree.branch
 
404
        if branch is not None:
 
405
            # We have a branch
 
406
            if repo is None:
 
407
                # The branch is in a shared repository
 
408
                repo = branch.repository
 
409
        if repo is not None:
 
410
            repo.lock_read()
 
411
            to_unlock.append(repo)
 
412
            branches = repo.find_branches(using=True)
 
413
            saw_tree = False
 
414
            if do_branch or do_tree:
 
415
                for branch in branches:
 
416
                    if do_tree:
129
417
                        try:
130
 
                            os.remove(p2)
131
 
                        except OSError, e:
132
 
                            if e.errno != e.ENOENT:
133
 
                                raise
134
 
                    os.rename(p1, p2)
135
 
 
136
 
                try:
137
 
                    # TODO: We may need to handle the case where the old revision
138
 
                    # entry was not compressed (and thus did not end with .gz)
139
 
 
140
 
                    # Remove the old revision entry out of the way
141
 
                    rev_path = branch.controlfilename(['revision-store', rid+'.gz'])
142
 
                    special_rename(rev_path, tmp_path)
143
 
                    branch.revision_store.add(rev_tmp, rid) # Add the new one
144
 
                    os.remove(tmp_path) # Remove the old name
145
 
                    mutter('    Updated revision entry {%s}' % rid)
146
 
                except:
147
 
                    # On any exception, restore the old entry
148
 
                    special_rename(tmp_path, rev_path)
149
 
                    raise
150
 
                rev_tmp.close()
151
 
                updated_revisions.append(rid)
152
 
            else:
153
 
                updated_previous_revision = False
154
 
 
155
 
            inv = branch.get_inventory(rev.inventory_id)
156
 
            seen_ids = {}
157
 
            seen_names = {}
158
 
 
159
 
            ## p('revision %d/%d file ids' % (revno, revcount))
160
 
            for file_id in inv:
161
 
                if file_id in seen_ids:
162
 
                    raise BzrCheckError('duplicated file_id {%s} '
163
 
                                        'in inventory for revision {%s}'
164
 
                                        % (file_id, rid))
165
 
                seen_ids[file_id] = True
166
 
 
167
 
            i = 0
168
 
            len_inv = len(inv)
169
 
            for file_id in inv:
170
 
                i += 1
171
 
                if i & 31 == 0:
172
 
                    pb.tick()
173
 
 
174
 
                ie = inv[file_id]
175
 
 
176
 
                if ie.parent_id != None:
177
 
                    if ie.parent_id not in seen_ids:
178
 
                        raise BzrCheckError('missing parent {%s} in inventory for revision {%s}'
179
 
                                % (ie.parent_id, rid))
180
 
 
181
 
                if ie.kind == 'file':
182
 
                    if ie.text_id in checked_texts:
183
 
                        fp = checked_texts[ie.text_id]
184
 
                    else:
185
 
                        if not ie.text_id in branch.text_store:
186
 
                            raise BzrCheckError('text {%s} not in text_store' % ie.text_id)
187
 
 
188
 
                        tf = branch.text_store[ie.text_id]
189
 
                        fp = fingerprint_file(tf)
190
 
                        checked_texts[ie.text_id] = fp
191
 
 
192
 
                    if ie.text_size != fp['size']:
193
 
                        raise BzrCheckError('text {%s} wrong size' % ie.text_id)
194
 
                    if ie.text_sha1 != fp['sha1']:
195
 
                        raise BzrCheckError('text {%s} wrong sha1' % ie.text_id)
196
 
                elif ie.kind == 'directory':
197
 
                    if ie.text_sha1 != None or ie.text_size != None or ie.text_id != None:
198
 
                        raise BzrCheckError('directory {%s} has text in revision {%s}'
199
 
                                % (file_id, rid))
200
 
 
201
 
            pb.tick()
202
 
            for path, ie in inv.iter_entries():
203
 
                if path in seen_names:
204
 
                    raise BzrCheckError('duplicated path %r '
205
 
                                        'in inventory for revision {%s}'
206
 
                                        % (path, revid))
207
 
            seen_names[path] = True
208
 
 
 
418
                            tree = branch.bzrdir.open_workingtree()
 
419
                            saw_tree = True
 
420
                        except (errors.NotLocalUrl, errors.NoWorkingTree):
 
421
                            pass
 
422
                        else:
 
423
                            scan_tree(base_tree, tree, needed_refs, to_unlock)
 
424
                    if do_branch:
 
425
                        scan_branch(branch, needed_refs, to_unlock)
 
426
            if do_branch and not branches:
 
427
                note(gettext("No branch found at specified location."))
 
428
            if do_tree and base_tree is None and not saw_tree:
 
429
                note(gettext("No working tree found at specified location."))
 
430
            if do_repo or do_branch or do_tree:
 
431
                if do_repo:
 
432
                    note(gettext("Checking repository at '%s'.")
 
433
                         % (repo.user_url,))
 
434
                result = repo.check(None, callback_refs=needed_refs,
 
435
                    check_repo=do_repo)
 
436
                result.report_results(verbose)
 
437
        else:
 
438
            if do_tree:
 
439
                note(gettext("No working tree found at specified location."))
 
440
            if do_branch:
 
441
                note(gettext("No branch found at specified location."))
 
442
            if do_repo:
 
443
                note(gettext("No repository found at specified location."))
209
444
    finally:
210
 
        branch.unlock()
211
 
 
212
 
    pb.clear()
213
 
 
214
 
    print 'checked %d revisions, %d file texts' % (revcount, len(checked_texts))
215
 
    if updated_revisions:
216
 
        print '%d revisions updated to current format' % len(updated_revisions)
217
 
    if missing_inventory_sha_cnt:
218
 
        print '%d revisions are missing inventory_sha1' % missing_inventory_sha_cnt
219
 
        print '  (use bzr check --update to fix them)'
 
445
        for thing in to_unlock:
 
446
            thing.unlock()