comparison mercurial/verify.py @ 28115:bd279da57e4b

verify: extract "manifest" constant into variable The "manifest" label that's used in error messages will instead be the directory path for subdirectory manifests (not the root manifest), so let's extract the constant to a variable already to make future patches simpler.
author Martin von Zweigbergk <martinvonz@google.com>
date Wed, 03 Feb 2016 15:53:48 -0800
parents 2a03a365f645
children 7297e9e13a8a
comparison
equal deleted inserted replaced
28114:2a03a365f645 28115:bd279da57e4b
203 mf = self.repo.manifest 203 mf = self.repo.manifest
204 204
205 ui.status(_("checking manifests\n")) 205 ui.status(_("checking manifests\n"))
206 filenodes = {} 206 filenodes = {}
207 seen = {} 207 seen = {}
208 label = "manifest"
208 if self.refersmf: 209 if self.refersmf:
209 # Do not check manifest if there are only changelog entries with 210 # Do not check manifest if there are only changelog entries with
210 # null manifests. 211 # null manifests.
211 self.checklog(mf, "manifest", 0) 212 self.checklog(mf, label, 0)
212 total = len(mf) 213 total = len(mf)
213 for i in mf: 214 for i in mf:
214 ui.progress(_('checking'), i, total=total, unit=_('manifests')) 215 ui.progress(_('checking'), i, total=total, unit=_('manifests'))
215 n = mf.node(i) 216 n = mf.node(i)
216 lr = self.checkentry(mf, i, n, seen, mflinkrevs.get(n, []), 217 lr = self.checkentry(mf, i, n, seen, mflinkrevs.get(n, []), label)
217 "manifest")
218 if n in mflinkrevs: 218 if n in mflinkrevs:
219 del mflinkrevs[n] 219 del mflinkrevs[n]
220 else: 220 else:
221 self.err(lr, _("%s not in changesets") % short(n), "manifest") 221 self.err(lr, _("%s not in changesets") % short(n), label)
222 222
223 try: 223 try:
224 for f, fn in mf.readdelta(n).iteritems(): 224 for f, fn in mf.readdelta(n).iteritems():
225 if not f: 225 if not f:
226 self.err(lr, _("file without name in manifest")) 226 self.err(lr, _("file without name in manifest"))
227 elif f != "/dev/null": # ignore this in very old repos 227 elif f != "/dev/null": # ignore this in very old repos
228 if _validpath(repo, f): 228 if _validpath(repo, f):
229 filenodes.setdefault( 229 filenodes.setdefault(
230 _normpath(f), {}).setdefault(fn, lr) 230 _normpath(f), {}).setdefault(fn, lr)
231 except Exception as inst: 231 except Exception as inst:
232 self.exc(lr, _("reading delta %s") % short(n), inst) 232 self.exc(lr, _("reading delta %s") % short(n), inst, label)
233 ui.progress(_('checking'), None) 233 ui.progress(_('checking'), None)
234 234
235 if self.havemf: 235 if self.havemf:
236 for c, m in sorted([(c, m) for m in mflinkrevs 236 for c, m in sorted([(c, m) for m in mflinkrevs
237 for c in mflinkrevs[m]]): 237 for c in mflinkrevs[m]]):
238 self.err(c, _("changeset refers to unknown revision %s") % 238 self.err(c, _("changeset refers to unknown revision %s") %
239 short(m), "manifest") 239 short(m), label)
240 240
241 return filenodes 241 return filenodes
242 242
243 def _crosscheckfiles(self, filelinkrevs, filenodes): 243 def _crosscheckfiles(self, filelinkrevs, filenodes):
244 repo = self.repo 244 repo = self.repo