D7709: verify: update comment to say that lfs doesn't need fulltext to check renames

mharbison72 (Matt Harbison) phabricator at mercurial-scm.org
Mon Dec 23 12:45:48 EST 2019


Closed by commit rHGa447efd991b9: verify: update comment to say that lfs doesn't need fulltext to check renames (authored by mharbison72).
This revision was automatically updated to reflect the committed changes.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D7709?vs=18905&id=18915

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

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

AFFECTED FILES
  mercurial/verify.py

CHANGE DETAILS

diff --git a/mercurial/verify.py b/mercurial/verify.py
--- a/mercurial/verify.py
+++ b/mercurial/verify.py
@@ -565,8 +565,9 @@
 
                 # check renames
                 try:
-                    # This requires resolving fulltext (at least on revlogs). We
-                    # may want ``verifyintegrity()`` to pass a set of nodes with
+                    # This requires resolving fulltext (at least on revlogs,
+                    # though not with LFS revisions). We may want
+                    # ``verifyintegrity()`` to pass a set of nodes with
                     # rename metadata as an optimization.
                     rp = fl.renamed(n)
                     if rp:



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


More information about the Mercurial-devel mailing list