D7950: verify: avoid spurious integrity warnings in verbose mode (issue6172)

mharbison72 (Matt Harbison) phabricator at mercurial-scm.org
Mon Jan 20 12:30:46 EST 2020


mharbison72 added a comment.


  In D7950#116781 <https://phab.mercurial-scm.org/D7950#116781>, @pulkit wrote:
  
  > TBH it feels not correct that some code path of `hg verify` needs unfiltered repository and that too around renames. Reading tests, I first thought it's some bug in unamend and then checked the issue to realize it's a general problem.
  
  The `verify.verifier` class uses an unfiltered repo, which I assume is why sometimes it can stray off into filtered revisions.  FWIW, I've got a repo at work that also prints:
  
    subrepo '$subrepo' is hidden in revision $hash
  
  This doesn't purport to fix that (and I struggled coming up with a test repro a few years ago).  Just mentioning that there are other filtering problems around verify.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D7950/new/

REVISION DETAIL
  https://phab.mercurial-scm.org/D7950

To: mharbison72, #hg-reviewers
Cc: pulkit, mercurial-devel


More information about the Mercurial-devel mailing list