Patchwork D683: largefiles: force an on-disk merge

login
register
mail settings
Submitter phabricator
Date Sept. 22, 2017, 6:32 p.m.
Message ID <eb3281b5d2d9c385c887e4a2de47fc3d@localhost.localdomain>
Download mbox | patch
Permalink /patch/24116/
State Not Applicable
Headers show

Comments

phabricator - Sept. 22, 2017, 6:32 p.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG575097b4dce0: largefiles: force an on-disk merge (authored by phillco, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D683?vs=1727&id=2043

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

AFFECTED FILES
  hgext/largefiles/overrides.py

CHANGE DETAILS




To: phillco, #hg-reviewers, durham, martinvonz
Cc: mercurial-devel

Patch

diff --git a/hgext/largefiles/overrides.py b/hgext/largefiles/overrides.py
--- a/hgext/largefiles/overrides.py
+++ b/hgext/largefiles/overrides.py
@@ -1432,7 +1432,10 @@ 
         lfdirstate.write()
 
         oldstandins = lfutil.getstandinsstate(repo)
-
+        # Make sure the merge runs on disk, not in-memory. largefiles is not a
+        # good candidate for in-memory merge (large files, custom dirstate,
+        # matcher usage).
+        kwargs['wc'] = repo[None]
         result = orig(repo, node, branchmerge, force, *args, **kwargs)
 
         newstandins = lfutil.getstandinsstate(repo)