~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/inventory.py

Lots of updates for 'bzr check'
Added get_sha1() to Weave, 
Updated RevisionTree so that it takes a Branch 
(thus it can use get_transaction()),
changed 'bzr check' so that it uses Weave.check(),
rather than having each file extract its own text

Show diffs side-by-side

added added

removed removed

Lines of Context:
482
482
            else:
483
483
                checker.repeated_text_cnt += 1
484
484
                return
 
485
 
 
486
        if self.file_id not in checker.checked_weaves:
 
487
            mutter('check weave {%s}', self.file_id)
 
488
            w = tree.get_weave(self.file_id)
 
489
            # Not passing a progress bar, because it creates a new
 
490
            # progress, which overwrites the current progress,
 
491
            # and doesn't look nice
 
492
            w.check()
 
493
            checker.checked_weaves[self.file_id] = True
 
494
        else:
 
495
            w = tree.get_weave_prelude(self.file_id)
 
496
 
485
497
        mutter('check version {%s} of {%s}', rev_id, self.file_id)
486
 
        file_lines = tree.get_file_lines(self.file_id)
487
498
        checker.checked_text_cnt += 1 
488
 
        if self.text_size != sum(map(len, file_lines)):
489
 
            raise BzrCheckError('text {%s} wrong size' % self.text_id)
490
 
        if self.text_sha1 != sha_strings(file_lines):
491
 
            raise BzrCheckError('text {%s} wrong sha1' % self.text_id)
 
499
        # We can't check the length, because Weave doesn't store that
 
500
        # information, and the whole point of looking at the weave's
 
501
        # sha1sum is that we don't have to extract the text.
 
502
        if self.text_sha1 != w.get_sha1(self.revision):
 
503
            raise BzrCheckError('text {%s} version {%s} wrong sha1' 
 
504
                                % (self.file_id, self.revision))
492
505
        checker.checked_texts[t] = self.text_sha1
493
506
 
494
507
    def copy(self):