mercurial-scm/hg-stable: mercurial/verify.py annotate

annotate mercurial/verify.py @ 25660:328739ea70c3

global: mass rewrite to use modern exception syntax Python 2.6 introduced the "except type as instance" syntax, replacing the "except type, instance" syntax that came before. Python 3 dropped support for the latter syntax. Since we no longer support Python 2.4 or 2.5, we have no need to continue supporting the "except type, instance". This patch mass rewrites the exception syntax to be Python 2.6+ and Python 3 compatible. This patch was produced by running `2to3 -f except -w -n .`.

author	Gregory Szorc <gregory.szorc@gmail.com>
date	Tue, 23 Jun 2015 22:20:08 -0700
parents	9d1e04f5dca7
children	c55eac3f388d

rev	line source
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	1 # verify.py - repository integrity checking for Mercurial
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	2 #
4635 63b9d2deed48 Updated copyright notices and add "and others" to "hg version" Thomas Arendsen Hein <thomas@intevation.de> parents: 4395 diff changeset	3 # Copyright 2006, 2007 Matt Mackall <mpm@selenic.com>
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	4 #
8225 46293a0c7e9f updated license to be explicit about GPL version 2 Martin Geisler <mg@lazybytes.net> parents: 8209 diff changeset	5 # This software may be used and distributed according to the terms of the
10263 25e572394f5c Update license to GPLv2+ Matt Mackall <mpm@selenic.com> parents: 9690 diff changeset	6 # GNU General Public License version 2 or any later version.
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	7
6211 f89fd07fc51d Expand import * to allow Pyflakes to find problems Joel Rosdahl <joel@rosdahl.net> parents: 5541 diff changeset	8 from node import nullid, short
3891 6b4127c7d52a Simplify i18n imports Matt Mackall <mpm@selenic.com> parents: 3744 diff changeset	9 from i18n import _
17860 a45b33f12627 verify: fix all doubled-slash sites (issue3665) Bryan O'Sullivan <bryano@fb.com> parents: 17851 diff changeset	10 import os
7832 1fce19b9b011 verify: do not abort on fully corrupted revlog Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 7676 diff changeset	11 import revlog, util, error
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	12
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	13 def verify(repo):
4915 97b734fb9c6f Use try/finally pattern to cleanup locks and transactions Matt Mackall <mpm@selenic.com> parents: 4635 diff changeset	14 lock = repo.lock()
97b734fb9c6f Use try/finally pattern to cleanup locks and transactions Matt Mackall <mpm@selenic.com> parents: 4635 diff changeset	15 try:
97b734fb9c6f Use try/finally pattern to cleanup locks and transactions Matt Mackall <mpm@selenic.com> parents: 4635 diff changeset	16 return _verify(repo)
97b734fb9c6f Use try/finally pattern to cleanup locks and transactions Matt Mackall <mpm@selenic.com> parents: 4635 diff changeset	17 finally:
8109 496ae1ea4698 switch lock releasing in the core from gc to explicit Ronny Pfannschmidt <Ronny.Pfannschmidt@gmx.de> parents: 7926 diff changeset	18 lock.release()
4915 97b734fb9c6f Use try/finally pattern to cleanup locks and transactions Matt Mackall <mpm@selenic.com> parents: 4635 diff changeset	19
17860 a45b33f12627 verify: fix all doubled-slash sites (issue3665) Bryan O'Sullivan <bryano@fb.com> parents: 17851 diff changeset	20 def _normpath(f):
a45b33f12627 verify: fix all doubled-slash sites (issue3665) Bryan O'Sullivan <bryano@fb.com> parents: 17851 diff changeset	21 # under hg < 2.4, convert didn't sanitize paths properly, so a
a45b33f12627 verify: fix all doubled-slash sites (issue3665) Bryan O'Sullivan <bryano@fb.com> parents: 17851 diff changeset	22 # converted repo may contain repeated slashes
a45b33f12627 verify: fix all doubled-slash sites (issue3665) Bryan O'Sullivan <bryano@fb.com> parents: 17851 diff changeset	23 while '//' in f:
a45b33f12627 verify: fix all doubled-slash sites (issue3665) Bryan O'Sullivan <bryano@fb.com> parents: 17851 diff changeset	24 f = f.replace('//', '/')
a45b33f12627 verify: fix all doubled-slash sites (issue3665) Bryan O'Sullivan <bryano@fb.com> parents: 17851 diff changeset	25 return f
a45b33f12627 verify: fix all doubled-slash sites (issue3665) Bryan O'Sullivan <bryano@fb.com> parents: 17851 diff changeset	26
4915 97b734fb9c6f Use try/finally pattern to cleanup locks and transactions Matt Mackall <mpm@selenic.com> parents: 4635 diff changeset	27 def _verify(repo):
18003 e1e181a64de7 clfilter: verify logic should be unfiltered Pierre-Yves David <pierre-yves.david@ens-lyon.org> parents: 17860 diff changeset	28 repo = repo.unfiltered()
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	29 mflinkrevs = {}
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	30 filelinkrevs = {}
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	31 filenodes = {}
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	32 revisions = 0
8466 afb3e504b558 verify: use set instead of dict Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 8394 diff changeset	33 badrevs = set()
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	34 errors = [0]
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	35 warnings = [0]
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	36 ui = repo.ui
7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	37 cl = repo.changelog
7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	38 mf = repo.manifest
9657 96c803e9018f verify: filter the candidate list for broken linkrevs Matt Mackall <mpm@selenic.com> parents: 9545 diff changeset	39 lrugetctx = util.lrucachefunc(repo.changectx)
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	40
20530 78f547cdc362 verify: do not prevent verify repository containing hidden changesets Yuya Nishihara <yuya@tcha.org> parents: 18003 diff changeset	41 if not repo.url().startswith('file:'):
7141 8d1bdaf842de issue 1144: prevent traceback on verify of bundles John Mulligan <phlogistonjohn@asynchrono.us> parents: 7004 diff changeset	42 raise util.Abort(_("cannot verify bundle or remote repos"))
8d1bdaf842de issue 1144: prevent traceback on verify of bundles John Mulligan <phlogistonjohn@asynchrono.us> parents: 7004 diff changeset	43
5313 29be4228303b verify: report first bad changeset Matt Mackall <mpm@selenic.com> parents: 5179 diff changeset	44 def err(linkrev, msg, filename=None):
13031 3da456d0c885 code style: prefer 'is' and 'is not' tests with singletons Martin Geisler <mg@aragost.com> parents: 12745 diff changeset	45 if linkrev is not None:
8466 afb3e504b558 verify: use set instead of dict Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 8394 diff changeset	46 badrevs.add(linkrev)
5313 29be4228303b verify: report first bad changeset Matt Mackall <mpm@selenic.com> parents: 5179 diff changeset	47 else:
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	48 linkrev = '?'
5313 29be4228303b verify: report first bad changeset Matt Mackall <mpm@selenic.com> parents: 5179 diff changeset	49 msg = "%s: %s" % (linkrev, msg)
29be4228303b verify: report first bad changeset Matt Mackall <mpm@selenic.com> parents: 5179 diff changeset	50 if filename:
29be4228303b verify: report first bad changeset Matt Mackall <mpm@selenic.com> parents: 5179 diff changeset	51 msg = "%s@%s" % (filename, msg)
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	52 ui.warn(" " + msg + "\n")
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	53 errors[0] += 1
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	54
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	55 def exc(linkrev, msg, inst, filename=None):
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	56 if isinstance(inst, KeyboardInterrupt):
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	57 ui.warn(_("interrupted"))
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	58 raise
11752 12998fd17fbc verify: report exceptions that don't have a str() method Matt Mackall <mpm@selenic.com> parents: 10906 diff changeset	59 if not str(inst):
12998fd17fbc verify: report exceptions that don't have a str() method Matt Mackall <mpm@selenic.com> parents: 10906 diff changeset	60 inst = repr(inst)
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	61 err(linkrev, "%s: %s" % (msg, inst), filename)
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	62
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	63 def warn(msg):
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	64 ui.warn(msg + "\n")
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	65 warnings[0] += 1
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	66
8292 29540554def8 verify: reference the correct linkrev when a filelog is missing Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 8291 diff changeset	67 def checklog(obj, name, linkrev):
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	68 if not len(obj) and (havecl or havemf):
8292 29540554def8 verify: reference the correct linkrev when a filelog is missing Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 8291 diff changeset	69 err(linkrev, _("empty or missing %s") % name)
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	70 return
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	71
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	72 d = obj.checksize()
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	73 if d[0]:
5313 29be4228303b verify: report first bad changeset Matt Mackall <mpm@selenic.com> parents: 5179 diff changeset	74 err(None, _("data length off by %d bytes") % d[0], name)
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	75 if d[1]:
5313 29be4228303b verify: report first bad changeset Matt Mackall <mpm@selenic.com> parents: 5179 diff changeset	76 err(None, _("index contains %d extra bytes") % d[1], name)
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	77
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	78 if obj.version != revlog.REVLOGV0:
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	79 if not revlogv1:
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	80 warn(_("warning: `%s' uses revlog format 1") % name)
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	81 elif revlogv1:
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	82 warn(_("warning: `%s' uses revlog format 0") % name)
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	83
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	84 def checkentry(obj, i, node, seen, linkrevs, f):
7361 9fe97eea5510 linkrev: take a revision number rather than a hash Matt Mackall <mpm@selenic.com> parents: 7141 diff changeset	85 lr = obj.linkrev(obj.rev(node))
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	86 if lr < 0 or (havecl and lr not in linkrevs):
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	87 if lr < 0 or lr >= len(cl):
7926 edf2d83a11aa verify, i18n: fix unmarked strings Wagner Bruna <wbruna@yahoo.com> parents: 7874 diff changeset	88 msg = _("rev %d points to nonexistent changeset %d")
edf2d83a11aa verify, i18n: fix unmarked strings Wagner Bruna <wbruna@yahoo.com> parents: 7874 diff changeset	89 else:
edf2d83a11aa verify, i18n: fix unmarked strings Wagner Bruna <wbruna@yahoo.com> parents: 7874 diff changeset	90 msg = _("rev %d points to unexpected changeset %d")
edf2d83a11aa verify, i18n: fix unmarked strings Wagner Bruna <wbruna@yahoo.com> parents: 7874 diff changeset	91 err(None, msg % (i, lr), f)
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	92 if linkrevs:
9657 96c803e9018f verify: filter the candidate list for broken linkrevs Matt Mackall <mpm@selenic.com> parents: 9545 diff changeset	93 if f and len(linkrevs) > 1:
96c803e9018f verify: filter the candidate list for broken linkrevs Matt Mackall <mpm@selenic.com> parents: 9545 diff changeset	94 try:
96c803e9018f verify: filter the candidate list for broken linkrevs Matt Mackall <mpm@selenic.com> parents: 9545 diff changeset	95 # attempt to filter down to real linkrevs
96c803e9018f verify: filter the candidate list for broken linkrevs Matt Mackall <mpm@selenic.com> parents: 9545 diff changeset	96 linkrevs = [l for l in linkrevs
96c803e9018f verify: filter the candidate list for broken linkrevs Matt Mackall <mpm@selenic.com> parents: 9545 diff changeset	97 if lrugetctx(l)[f].filenode() == node]
16689 f366d4c2ff34 cleanup: replace naked excepts with except Exception: ... Brodie Rao <brodie@sf.io> parents: 14879 diff changeset	98 except Exception:
9657 96c803e9018f verify: filter the candidate list for broken linkrevs Matt Mackall <mpm@selenic.com> parents: 9545 diff changeset	99 pass
9198 061eeb602354 coding style: use a space after comma Martin Geisler <mg@lazybytes.net> parents: 9033 diff changeset	100 warn(_(" (expected %s)") % " ".join(map(str, linkrevs)))
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	101 lr = None # can't be trusted
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	102
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	103 try:
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	104 p1, p2 = obj.parents(node)
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	105 if p1 not in seen and p1 != nullid:
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	106 err(lr, _("unknown parent 1 %s of %s") %
17719 2e3ceb59c312 verify: use appropriate local variable in "checkentry()" FUJIWARA Katsunori <foozy@lares.dti.ne.jp> parents: 17718 diff changeset	107 (short(p1), short(node)), f)
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	108 if p2 not in seen and p2 != nullid:
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	109 err(lr, _("unknown parent 2 %s of %s") %
17719 2e3ceb59c312 verify: use appropriate local variable in "checkentry()" FUJIWARA Katsunori <foozy@lares.dti.ne.jp> parents: 17718 diff changeset	110 (short(p2), short(node)), f)
25660 328739ea70c3 global: mass rewrite to use modern exception syntax Gregory Szorc <gregory.szorc@gmail.com> parents: 25653 diff changeset	111 except Exception as inst:
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	112 exc(lr, _("checking parents of %s") % short(node), inst, f)
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	113
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	114 if node in seen:
17719 2e3ceb59c312 verify: use appropriate local variable in "checkentry()" FUJIWARA Katsunori <foozy@lares.dti.ne.jp> parents: 17718 diff changeset	115 err(lr, _("duplicate revision %d (%d)") % (i, seen[node]), f)
2e3ceb59c312 verify: use appropriate local variable in "checkentry()" FUJIWARA Katsunori <foozy@lares.dti.ne.jp> parents: 17718 diff changeset	116 seen[node] = i
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	117 return lr
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	118
9690 b33d70849a20 verify: report existence of journal Sune Foldager <cryo@cyanite.org> parents: 9657 diff changeset	119 if os.path.exists(repo.sjoin("journal")):
b33d70849a20 verify: report existence of journal Sune Foldager <cryo@cyanite.org> parents: 9657 diff changeset	120 ui.warn(_("abandoned transaction found - run hg recover\n"))
b33d70849a20 verify: report existence of journal Sune Foldager <cryo@cyanite.org> parents: 9657 diff changeset	121
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	122 revlogv1 = cl.version != revlog.REVLOGV0
7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	123 if ui.verbose or not revlogv1:
7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	124 ui.status(_("repository uses revlog format %d\n") %
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	125 (revlogv1 and 1 or 0))
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	126
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	127 havecl = len(cl) > 0
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	128 havemf = len(mf) > 0
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	129
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	130 ui.status(_("checking changesets\n"))
17720 9a1796af4f8d verify: rename "hasmanifest" variable for source code readability FUJIWARA Katsunori <foozy@lares.dti.ne.jp> parents: 17719 diff changeset	131 refersmf = False
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	132 seen = {}
8292 29540554def8 verify: reference the correct linkrev when a filelog is missing Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 8291 diff changeset	133 checklog(cl, "changelog", 0)
10433 767fbacb3ddc verify: call ui.progress() Augie Fackler <durin42@gmail.com> parents: 10282 diff changeset	134 total = len(repo)
6750 fb42030d79d6 add __len__ and __iter__ methods to repo and revlog Matt Mackall <mpm@selenic.com> parents: 6534 diff changeset	135 for i in repo:
12745 5a1912b5aa42 verify/progress: using gerund to indicate action and adding units timeless <timeless@gmail.com> parents: 12170 diff changeset	136 ui.progress(_('checking'), i, total=total, unit=_('changesets'))
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	137 n = cl.node(i)
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	138 checkentry(cl, i, n, seen, [i], "changelog")
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	139
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	140 try:
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	141 changes = cl.read(n)
17385 b32a30da608d verify: do not choke on valid changelog without manifest Patrick Mezard <patrick@mezard.eu> parents: 16689 diff changeset	142 if changes[0] != nullid:
b32a30da608d verify: do not choke on valid changelog without manifest Patrick Mezard <patrick@mezard.eu> parents: 16689 diff changeset	143 mflinkrevs.setdefault(changes[0], []).append(i)
17720 9a1796af4f8d verify: rename "hasmanifest" variable for source code readability FUJIWARA Katsunori <foozy@lares.dti.ne.jp> parents: 17719 diff changeset	144 refersmf = True
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	145 for f in changes[3]:
17860 a45b33f12627 verify: fix all doubled-slash sites (issue3665) Bryan O'Sullivan <bryano@fb.com> parents: 17851 diff changeset	146 filelinkrevs.setdefault(_normpath(f), []).append(i)
25660 328739ea70c3 global: mass rewrite to use modern exception syntax Gregory Szorc <gregory.szorc@gmail.com> parents: 25653 diff changeset	147 except Exception as inst:
17720 9a1796af4f8d verify: rename "hasmanifest" variable for source code readability FUJIWARA Katsunori <foozy@lares.dti.ne.jp> parents: 17719 diff changeset	148 refersmf = True
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	149 exc(i, _("unpacking changeset %s") % short(n), inst)
12745 5a1912b5aa42 verify/progress: using gerund to indicate action and adding units timeless <timeless@gmail.com> parents: 12170 diff changeset	150 ui.progress(_('checking'), None)
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	151
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	152 ui.status(_("checking manifests\n"))
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	153 seen = {}
17720 9a1796af4f8d verify: rename "hasmanifest" variable for source code readability FUJIWARA Katsunori <foozy@lares.dti.ne.jp> parents: 17719 diff changeset	154 if refersmf:
17385 b32a30da608d verify: do not choke on valid changelog without manifest Patrick Mezard <patrick@mezard.eu> parents: 16689 diff changeset	155 # Do not check manifest if there are only changelog entries with
b32a30da608d verify: do not choke on valid changelog without manifest Patrick Mezard <patrick@mezard.eu> parents: 16689 diff changeset	156 # null manifests.
b32a30da608d verify: do not choke on valid changelog without manifest Patrick Mezard <patrick@mezard.eu> parents: 16689 diff changeset	157 checklog(mf, "manifest", 0)
10433 767fbacb3ddc verify: call ui.progress() Augie Fackler <durin42@gmail.com> parents: 10282 diff changeset	158 total = len(mf)
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	159 for i in mf:
12745 5a1912b5aa42 verify/progress: using gerund to indicate action and adding units timeless <timeless@gmail.com> parents: 12170 diff changeset	160 ui.progress(_('checking'), i, total=total, unit=_('manifests'))
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	161 n = mf.node(i)
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	162 lr = checkentry(mf, i, n, seen, mflinkrevs.get(n, []), "manifest")
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	163 if n in mflinkrevs:
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	164 del mflinkrevs[n]
8394 850b5a7c210d verify: detect manifest revs not in any changeset Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 8292 diff changeset	165 else:
850b5a7c210d verify: detect manifest revs not in any changeset Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 8292 diff changeset	166 err(lr, _("%s not in changesets") % short(n), "manifest")
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	167
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	168 try:
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	169 for f, fn in mf.readdelta(n).iteritems():
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	170 if not f:
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	171 err(lr, _("file without name in manifest"))
24481 dded1eeeff64 verify: add a note about a paleo-bug Matt Mackall <mpm@selenic.com> parents: 22933 diff changeset	172 elif f != "/dev/null": # ignore this in very old repos
17860 a45b33f12627 verify: fix all doubled-slash sites (issue3665) Bryan O'Sullivan <bryano@fb.com> parents: 17851 diff changeset	173 filenodes.setdefault(_normpath(f), {}).setdefault(fn, lr)
25660 328739ea70c3 global: mass rewrite to use modern exception syntax Gregory Szorc <gregory.szorc@gmail.com> parents: 25653 diff changeset	174 except Exception as inst:
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	175 exc(lr, _("reading manifest delta %s") % short(n), inst)
12745 5a1912b5aa42 verify/progress: using gerund to indicate action and adding units timeless <timeless@gmail.com> parents: 12170 diff changeset	176 ui.progress(_('checking'), None)
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	177
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	178 ui.status(_("crosschecking files in changesets and manifests\n"))
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	179
10433 767fbacb3ddc verify: call ui.progress() Augie Fackler <durin42@gmail.com> parents: 10282 diff changeset	180 total = len(mflinkrevs) + len(filelinkrevs) + len(filenodes)
767fbacb3ddc verify: call ui.progress() Augie Fackler <durin42@gmail.com> parents: 10282 diff changeset	181 count = 0
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	182 if havemf:
10282 08a0f04b56bd many, many trivial check-code fixups Matt Mackall <mpm@selenic.com> parents: 10263 diff changeset	183 for c, m in sorted([(c, m) for m in mflinkrevs
08a0f04b56bd many, many trivial check-code fixups Matt Mackall <mpm@selenic.com> parents: 10263 diff changeset	184 for c in mflinkrevs[m]]):
10433 767fbacb3ddc verify: call ui.progress() Augie Fackler <durin42@gmail.com> parents: 10282 diff changeset	185 count += 1
14865 eb914541a950 verify: filter messages about missing null manifests (issue2900) Matt Mackall <mpm@selenic.com> parents: 13031 diff changeset	186 if m == nullid:
eb914541a950 verify: filter messages about missing null manifests (issue2900) Matt Mackall <mpm@selenic.com> parents: 13031 diff changeset	187 continue
10698 e930017f9e2a progress: use a verb (+noun) in present participle Martin Geisler <mg@lazybytes.net> parents: 10496 diff changeset	188 ui.progress(_('crosschecking'), count, total=total)
5541 ceaa752fa316 verify: improve handling of empty or missing files Matt Mackall <mpm@selenic.com> parents: 5313 diff changeset	189 err(c, _("changeset refers to unknown manifest %s") % short(m))
9033 98a5652bfed9 verify: fix scope issues with del statement Alejandro Santos <alejolp@alejolp.com> parents: 8993 diff changeset	190 mflinkrevs = None # del is bad here due to scope issues
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	191
8209 a1a5a57efe90 replace util.sort with sorted built-in Matt Mackall <mpm@selenic.com> parents: 8164 diff changeset	192 for f in sorted(filelinkrevs):
10433 767fbacb3ddc verify: call ui.progress() Augie Fackler <durin42@gmail.com> parents: 10282 diff changeset	193 count += 1
10698 e930017f9e2a progress: use a verb (+noun) in present participle Martin Geisler <mg@lazybytes.net> parents: 10496 diff changeset	194 ui.progress(_('crosschecking'), count, total=total)
5541 ceaa752fa316 verify: improve handling of empty or missing files Matt Mackall <mpm@selenic.com> parents: 5313 diff changeset	195 if f not in filenodes:
ceaa752fa316 verify: improve handling of empty or missing files Matt Mackall <mpm@selenic.com> parents: 5313 diff changeset	196 lr = filelinkrevs[f][0]
ceaa752fa316 verify: improve handling of empty or missing files Matt Mackall <mpm@selenic.com> parents: 5313 diff changeset	197 err(lr, _("in changeset but not in manifest"), f)
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	198
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	199 if havecl:
8209 a1a5a57efe90 replace util.sort with sorted built-in Matt Mackall <mpm@selenic.com> parents: 8164 diff changeset	200 for f in sorted(filenodes):
10433 767fbacb3ddc verify: call ui.progress() Augie Fackler <durin42@gmail.com> parents: 10282 diff changeset	201 count += 1
10698 e930017f9e2a progress: use a verb (+noun) in present participle Martin Geisler <mg@lazybytes.net> parents: 10496 diff changeset	202 ui.progress(_('crosschecking'), count, total=total)
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	203 if f not in filelinkrevs:
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	204 try:
7361 9fe97eea5510 linkrev: take a revision number rather than a hash Matt Mackall <mpm@selenic.com> parents: 7141 diff changeset	205 fl = repo.file(f)
9fe97eea5510 linkrev: take a revision number rather than a hash Matt Mackall <mpm@selenic.com> parents: 7141 diff changeset	206 lr = min([fl.linkrev(fl.rev(n)) for n in filenodes[f]])
16689 f366d4c2ff34 cleanup: replace naked excepts with except Exception: ... Brodie Rao <brodie@sf.io> parents: 14879 diff changeset	207 except Exception:
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	208 lr = None
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	209 err(lr, _("in manifest but not in changeset"), f)
e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	210
10698 e930017f9e2a progress: use a verb (+noun) in present participle Martin Geisler <mg@lazybytes.net> parents: 10496 diff changeset	211 ui.progress(_('crosschecking'), None)
10433 767fbacb3ddc verify: call ui.progress() Augie Fackler <durin42@gmail.com> parents: 10282 diff changeset	212
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	213 ui.status(_("checking files\n"))
6892 dab95717058d verify: check repo.store Adrian Buehlmann <adrian@cadifra.com> parents: 6889 diff changeset	214
8466 afb3e504b558 verify: use set instead of dict Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 8394 diff changeset	215 storefiles = set()
6900 def492d1b592 store: change handling of decoding errors Matt Mackall <mpm@selenic.com> parents: 6892 diff changeset	216 for f, f2, size in repo.store.datafiles():
def492d1b592 store: change handling of decoding errors Matt Mackall <mpm@selenic.com> parents: 6892 diff changeset	217 if not f:
def492d1b592 store: change handling of decoding errors Matt Mackall <mpm@selenic.com> parents: 6892 diff changeset	218 err(None, _("cannot decode filename '%s'") % f2)
12170 581066a319e5 verify: fix "missing revlog!" errors for revlog format v0 and add test Thomas Arendsen Hein <thomas@intevation.de> parents: 11756 diff changeset	219 elif size > 0 or not revlogv1:
17860 a45b33f12627 verify: fix all doubled-slash sites (issue3665) Bryan O'Sullivan <bryano@fb.com> parents: 17851 diff changeset	220 storefiles.add(_normpath(f))
6892 dab95717058d verify: check repo.store Adrian Buehlmann <adrian@cadifra.com> parents: 6889 diff changeset	221
25653 9d1e04f5dca7 verify: print hint to run debugrebuildfncache Gregory Szorc <gregory.szorc@gmail.com> parents: 25627 diff changeset	222 fncachewarned = False
8209 a1a5a57efe90 replace util.sort with sorted built-in Matt Mackall <mpm@selenic.com> parents: 8164 diff changeset	223 files = sorted(set(filenodes) \| set(filelinkrevs))
10433 767fbacb3ddc verify: call ui.progress() Augie Fackler <durin42@gmail.com> parents: 10282 diff changeset	224 total = len(files)
767fbacb3ddc verify: call ui.progress() Augie Fackler <durin42@gmail.com> parents: 10282 diff changeset	225 for i, f in enumerate(files):
10698 e930017f9e2a progress: use a verb (+noun) in present participle Martin Geisler <mg@lazybytes.net> parents: 10496 diff changeset	226 ui.progress(_('checking'), i, item=f, total=total)
8291 f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	227 try:
f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	228 linkrevs = filelinkrevs[f]
f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	229 except KeyError:
f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	230 # in manifest but not in changelog
f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	231 linkrevs = []
f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	232
f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	233 if linkrevs:
f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	234 lr = linkrevs[0]
f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	235 else:
f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	236 lr = None
f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	237
7832 1fce19b9b011 verify: do not abort on fully corrupted revlog Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 7676 diff changeset	238 try:
1fce19b9b011 verify: do not abort on fully corrupted revlog Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 7676 diff changeset	239 fl = repo.file(f)
25660 328739ea70c3 global: mass rewrite to use modern exception syntax Gregory Szorc <gregory.szorc@gmail.com> parents: 25653 diff changeset	240 except error.RevlogError as e:
7833 794def2fe232 verify: find correct first corrupted cset for missing/corrupted revlogs Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 7832 diff changeset	241 err(lr, _("broken revlog! (%s)") % e, f)
7832 1fce19b9b011 verify: do not abort on fully corrupted revlog Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 7676 diff changeset	242 continue
6892 dab95717058d verify: check repo.store Adrian Buehlmann <adrian@cadifra.com> parents: 6889 diff changeset	243
6900 def492d1b592 store: change handling of decoding errors Matt Mackall <mpm@selenic.com> parents: 6892 diff changeset	244 for ff in fl.files():
6892 dab95717058d verify: check repo.store Adrian Buehlmann <adrian@cadifra.com> parents: 6889 diff changeset	245 try:
8466 afb3e504b558 verify: use set instead of dict Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 8394 diff changeset	246 storefiles.remove(ff)
6892 dab95717058d verify: check repo.store Adrian Buehlmann <adrian@cadifra.com> parents: 6889 diff changeset	247 except KeyError:
25627 9573d8f346f1 verify: clarify misleading fncache message Matt Mackall <mpm@selenic.com> parents: 24481 diff changeset	248 warn(_(" warning: revlog '%s' not in fncache!") % ff)
25653 9d1e04f5dca7 verify: print hint to run debugrebuildfncache Gregory Szorc <gregory.szorc@gmail.com> parents: 25627 diff changeset	249 fncachewarned = True
6892 dab95717058d verify: check repo.store Adrian Buehlmann <adrian@cadifra.com> parents: 6889 diff changeset	250
8292 29540554def8 verify: reference the correct linkrev when a filelog is missing Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 8291 diff changeset	251 checklog(fl, f, lr)
5313 29be4228303b verify: report first bad changeset Matt Mackall <mpm@selenic.com> parents: 5179 diff changeset	252 seen = {}
11756 0299240b849b verify: initialize rp variable in case we hit out of memory Matt Mackall <mpm@selenic.com> parents: 11753 diff changeset	253 rp = None
6750 fb42030d79d6 add __len__ and __iter__ methods to repo and revlog Matt Mackall <mpm@selenic.com> parents: 6534 diff changeset	254 for i in fl:
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	255 revisions += 1
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	256 n = fl.node(i)
8291 f5c1a9094e41 verify: avoid exception on missing file revlog Henrik Stuart <hg@hstuart.dk> parents: 8225 diff changeset	257 lr = checkentry(fl, i, n, seen, linkrevs, f)
5541 ceaa752fa316 verify: improve handling of empty or missing files Matt Mackall <mpm@selenic.com> parents: 5313 diff changeset	258 if f in filenodes:
ceaa752fa316 verify: improve handling of empty or missing files Matt Mackall <mpm@selenic.com> parents: 5313 diff changeset	259 if havemf and n not in filenodes[f]:
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	260 err(lr, _("%s not in manifests") % (short(n)), f)
5541 ceaa752fa316 verify: improve handling of empty or missing files Matt Mackall <mpm@selenic.com> parents: 5313 diff changeset	261 else:
ceaa752fa316 verify: improve handling of empty or missing files Matt Mackall <mpm@selenic.com> parents: 5313 diff changeset	262 del filenodes[f][n]
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	263
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	264 # verify contents
fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	265 try:
11753 eb7b04657dae verify: reduce memory footprint when unpacking files Matt Mackall <mpm@selenic.com> parents: 11752 diff changeset	266 l = len(fl.read(n))
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	267 rp = fl.renamed(n)
11753 eb7b04657dae verify: reduce memory footprint when unpacking files Matt Mackall <mpm@selenic.com> parents: 11752 diff changeset	268 if l != fl.size(i):
7675 011e69b96c69 verify: don't trip over binary files starting with 01 0a Matt Mackall <mpm@selenic.com> parents: 7361 diff changeset	269 if len(fl.revision(n)) != fl.size(i):
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	270 err(lr, _("unpacked size is %s, %s expected") %
11753 eb7b04657dae verify: reduce memory footprint when unpacking files Matt Mackall <mpm@selenic.com> parents: 11752 diff changeset	271 (l, fl.size(i)), f)
22933 3a60cd44e619 verify: report censored nodes if configured policy is abort Mike Edgar <adgar@google.com> parents: 20530 diff changeset	272 except error.CensoredNodeError:
3a60cd44e619 verify: report censored nodes if configured policy is abort Mike Edgar <adgar@google.com> parents: 20530 diff changeset	273 if ui.config("censor", "policy", "abort") == "abort":
3a60cd44e619 verify: report censored nodes if configured policy is abort Mike Edgar <adgar@google.com> parents: 20530 diff changeset	274 err(lr, _("censored file data"), f)
25660 328739ea70c3 global: mass rewrite to use modern exception syntax Gregory Szorc <gregory.szorc@gmail.com> parents: 25653 diff changeset	275 except Exception as inst:
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	276 exc(lr, _("unpacking %s") % short(n), inst, f)
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	277
3744 d626fc9e3985 verify: add rename link checking Matt Mackall <mpm@selenic.com> parents: 3473 diff changeset	278 # check renames
d626fc9e3985 verify: add rename link checking Matt Mackall <mpm@selenic.com> parents: 3473 diff changeset	279 try:
d626fc9e3985 verify: add rename link checking Matt Mackall <mpm@selenic.com> parents: 3473 diff changeset	280 if rp:
9545 98feea5659d9 verify: detect file copy sources not in parents with --verbose Patrick Mezard <pmezard@gmail.com> parents: 9198 diff changeset	281 if lr is not None and ui.verbose:
98feea5659d9 verify: detect file copy sources not in parents with --verbose Patrick Mezard <pmezard@gmail.com> parents: 9198 diff changeset	282 ctx = lrugetctx(lr)
98feea5659d9 verify: detect file copy sources not in parents with --verbose Patrick Mezard <pmezard@gmail.com> parents: 9198 diff changeset	283 found = False
98feea5659d9 verify: detect file copy sources not in parents with --verbose Patrick Mezard <pmezard@gmail.com> parents: 9198 diff changeset	284 for pctx in ctx.parents():
98feea5659d9 verify: detect file copy sources not in parents with --verbose Patrick Mezard <pmezard@gmail.com> parents: 9198 diff changeset	285 if rp[0] in pctx:
98feea5659d9 verify: detect file copy sources not in parents with --verbose Patrick Mezard <pmezard@gmail.com> parents: 9198 diff changeset	286 found = True
98feea5659d9 verify: detect file copy sources not in parents with --verbose Patrick Mezard <pmezard@gmail.com> parents: 9198 diff changeset	287 break
98feea5659d9 verify: detect file copy sources not in parents with --verbose Patrick Mezard <pmezard@gmail.com> parents: 9198 diff changeset	288 if not found:
98feea5659d9 verify: detect file copy sources not in parents with --verbose Patrick Mezard <pmezard@gmail.com> parents: 9198 diff changeset	289 warn(_("warning: copy source of '%s' not"
98feea5659d9 verify: detect file copy sources not in parents with --verbose Patrick Mezard <pmezard@gmail.com> parents: 9198 diff changeset	290 " in parents of %s") % (f, ctx))
3744 d626fc9e3985 verify: add rename link checking Matt Mackall <mpm@selenic.com> parents: 3473 diff changeset	291 fl2 = repo.file(rp[0])
6750 fb42030d79d6 add __len__ and __iter__ methods to repo and revlog Matt Mackall <mpm@selenic.com> parents: 6534 diff changeset	292 if not len(fl2):
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	293 err(lr, _("empty or missing copy source revlog %s:%s")
6534 9b35a9f34675 verify: check copy source revlog and nodeid Patrick Mezard <pmezard@gmail.com> parents: 6211 diff changeset	294 % (rp[0], short(rp[1])), f)
9b35a9f34675 verify: check copy source revlog and nodeid Patrick Mezard <pmezard@gmail.com> parents: 6211 diff changeset	295 elif rp[1] == nullid:
8993 46441934c585 verify: demote warning about nullid in copy to note Matt Mackall <mpm@selenic.com> parents: 8466 diff changeset	296 ui.note(_("warning: %s@%s: copy source"
46441934c585 verify: demote warning about nullid in copy to note Matt Mackall <mpm@selenic.com> parents: 8466 diff changeset	297 " revision is nullid %s:%s\n")
7004 90227c42b5f6 c0bd7d8b69ef uses err() instead of warn() but prototype doesn't match Benoit Boissinot <benoit.boissinot@ens-lyon.org> parents: 6900 diff changeset	298 % (f, lr, rp[0], short(rp[1])))
6534 9b35a9f34675 verify: check copy source revlog and nodeid Patrick Mezard <pmezard@gmail.com> parents: 6211 diff changeset	299 else:
7874 d812029cda85 cleanup: drop variables for unused return values Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 7833 diff changeset	300 fl2.rev(rp[1])
25660 328739ea70c3 global: mass rewrite to use modern exception syntax Gregory Szorc <gregory.szorc@gmail.com> parents: 25653 diff changeset	301 except Exception as inst:
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	302 exc(lr, _("checking rename of %s") % short(n), inst, f)
3744 d626fc9e3985 verify: add rename link checking Matt Mackall <mpm@selenic.com> parents: 3473 diff changeset	303
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	304 # cross-check
5541 ceaa752fa316 verify: improve handling of empty or missing files Matt Mackall <mpm@selenic.com> parents: 5313 diff changeset	305 if f in filenodes:
10282 08a0f04b56bd many, many trivial check-code fixups Matt Mackall <mpm@selenic.com> parents: 10263 diff changeset	306 fns = [(lr, n) for n, lr in filenodes[f].iteritems()]
8209 a1a5a57efe90 replace util.sort with sorted built-in Matt Mackall <mpm@selenic.com> parents: 8164 diff changeset	307 for lr, node in sorted(fns):
5541 ceaa752fa316 verify: improve handling of empty or missing files Matt Mackall <mpm@selenic.com> parents: 5313 diff changeset	308 err(lr, _("%s in manifests not found") % short(node), f)
10698 e930017f9e2a progress: use a verb (+noun) in present participle Martin Geisler <mg@lazybytes.net> parents: 10496 diff changeset	309 ui.progress(_('checking'), None)
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	310
6892 dab95717058d verify: check repo.store Adrian Buehlmann <adrian@cadifra.com> parents: 6889 diff changeset	311 for f in storefiles:
dab95717058d verify: check repo.store Adrian Buehlmann <adrian@cadifra.com> parents: 6889 diff changeset	312 warn(_("warning: orphan revlog '%s'") % f)
dab95717058d verify: check repo.store Adrian Buehlmann <adrian@cadifra.com> parents: 6889 diff changeset	313
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	314 ui.status(_("%d files, %d changesets, %d total revisions\n") %
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	315 (len(files), len(cl), revisions))
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	316 if warnings[0]:
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	317 ui.warn(_("%d warnings encountered!\n") % warnings[0])
25653 9d1e04f5dca7 verify: print hint to run debugrebuildfncache Gregory Szorc <gregory.szorc@gmail.com> parents: 25627 diff changeset	318 if fncachewarned:
9d1e04f5dca7 verify: print hint to run debugrebuildfncache Gregory Szorc <gregory.szorc@gmail.com> parents: 25627 diff changeset	319 ui.warn(_('hint: run "hg debugrebuildfncache" to recover from '
9d1e04f5dca7 verify: print hint to run debugrebuildfncache Gregory Szorc <gregory.szorc@gmail.com> parents: 25627 diff changeset	320 'corrupt fncache\n'))
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	321 if errors[0]:
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	322 ui.warn(_("%d integrity errors encountered!\n") % errors[0])
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	323 if badrevs:
6751 7424a75f919a verify: add some local variables Matt Mackall <mpm@selenic.com> parents: 6750 diff changeset	324 ui.warn(_("(first damaged changeset appears to be %d)\n")
6752 e79a8f36c2a5 verify: lots of refactoring Matt Mackall <mpm@selenic.com> parents: 6751 diff changeset	325 % min(badrevs))
2778 fdc232d8a193 Move repo.verify Matt Mackall <mpm@selenic.com> parents: diff changeset	326 return 1

Mercurial > public > mercurial-scm > hg-stable

annotate mercurial/verify.py @ 25660:328739ea70c3