Patchwork [1,of,3] largefile: backout ca54fb3d71ce

login
register
mail settings
Submitter Pierre-Yves David
Date Dec. 17, 2014, 8:42 p.m.
Message ID <6232babf277b57b826f3.1418848936@marginatus.alto.octopoid.net>
Download mbox | patch
Permalink /patch/7147/
State Accepted
Headers show

Comments

Pierre-Yves David - Dec. 17, 2014, 8:42 p.m.
# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@fb.com>
# Date 1418847016 28800
#      Wed Dec 17 12:10:16 2014 -0800
# Node ID 6232babf277b57b826f337628e4372f9c45f2862
# Parent  495bc1b65d25872324a0220354f048b220304bd1
largefile: backout ca54fb3d71ce

The hack for methods monkey patching on repoview have been ruled out as
fragile, so we are rolling it back. We'll expand the explanation in the next
changeset.

Patch

diff --git a/hgext/largefiles/overrides.py b/hgext/largefiles/overrides.py
--- a/hgext/largefiles/overrides.py
+++ b/hgext/largefiles/overrides.py
@@ -1114,10 +1114,13 @@  def scmutiladdremove(orig, repo, matcher
     return orig(repo, matcher, prefix, opts, dry_run, similarity)
 
 # Calling purge with --all will cause the largefiles to be deleted.
 # Override repo.status to prevent this from happening.
 def overridepurge(orig, ui, repo, *dirs, **opts):
+    # XXX large file status is buggy when used on repo proxy.
+    # XXX this needs to be investigate.
+    repo = repo.unfiltered()
     oldstatus = repo.status
     def overridestatus(node1='.', node2=None, match=None, ignored=False,
                         clean=False, unknown=False, listsubrepos=False):
         r = oldstatus(node1, node2, match, ignored, clean, unknown,
                       listsubrepos)