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

login
register
mail settings
Submitter phabricator
Date Dec. 23, 2019, 5:45 p.m.
Message ID <4cd0027332a5249d091e5f22d43e344d@localhost.localdomain>
Download mbox | patch
Permalink /patch/44028/
State Not Applicable
Headers show

Comments

phabricator - Dec. 23, 2019, 5:45 p.m.
Closed by commit rHGa447efd991b9: verify: update comment to say that lfs doesn&#039;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




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

Patch

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: