[PATCH 3 of 6 stable] largefiles: drop setting lfstatus in overridelog (issue4334)

Siddharth Agarwal sid0 at fb.com
Wed Aug 13 23:05:22 UTC 2014


# HG changeset patch
# User Siddharth Agarwal <sid0 at fb.com>
# Date 1407968030 25200
#      Wed Aug 13 15:13:50 2014 -0700
# Branch stable
# Node ID f8b7102e2de614132fd1ff5291cd2b33a795b9c7
# Parent  2d22504013642e95deb73811347276b11aefa469
largefiles: drop setting lfstatus in overridelog (issue4334)

lfstatus should only be True for operations where we want standins to be
printed out. We explicitly do not want that for historical operations like log.
Other historical operations like hg diff -r A -r B don't print out standins
either.

This is required to fix issue4334, but doesn't fix anything by itself. That's
why there aren't any tests accompanying this patch.

diff --git a/hgext/largefiles/overrides.py b/hgext/largefiles/overrides.py
--- a/hgext/largefiles/overrides.py
+++ b/hgext/largefiles/overrides.py
@@ -301,10 +301,8 @@
 
     oldmatchandpats = installmatchandpatsfn(overridematchandpats)
     try:
-        repo.lfstatus = True
         return orig(ui, repo, *pats, **opts)
     finally:
-        repo.lfstatus = False
         restorematchandpatsfn()
 
 def overrideverify(orig, ui, repo, *pats, **opts):



More information about the Mercurial-devel mailing list