Mercurial > hg
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 |