Patchwork [2,of,8] largefiles: restore standins from non branch-tip parent at rollback correctly

mail settings
Submitter Katsunori FUJIWARA
Date Aug. 24, 2014, 2:54 p.m.
Message ID <2ced8eedb543131596d1.1408892088@feefifofum>
Download mbox | patch
Permalink /patch/5569/
State Accepted
Headers show


Katsunori FUJIWARA - Aug. 24, 2014, 2:54 p.m.
# HG changeset patch
# User FUJIWARA Katsunori <>
# Date 1408891645 -32400
#      Sun Aug 24 23:47:25 2014 +0900
# Node ID 2ced8eedb543131596d13ceb3a47d84f60538c3d
# Parent  7704d05858a4c56db12c28b90e7d2271192b8a83
largefiles: restore standins from non branch-tip parent at rollback correctly

Before this patch, "hg rollback" can't restore standins correclty, if:

  - old parent of the working directory is rollback-ed, and
  - new parent of the working directory is not branch-tip

"overriderollback" uses "merge.update" as a kind of "revert" utility
to restore only standins with "node=None", and this makes
"merge.update" choose "branch-tip" revision as the updating target

Then, "merge.update" restores standins from the branch-tip revision
regardless of the parent of the working directory after rollback and
this may cause unexpected behavior.

This patch invokes "merge.update" with "node='.'" to restore standins
from the parent revision of the working directory.

In fact, this "merge.update" invocation will be replaced in the
subsequent patch to fix another problem, but this change is usefull to
inform reason why such complicated case should be tested.


diff --git a/hgext/largefiles/ b/hgext/largefiles/
--- a/hgext/largefiles/
+++ b/hgext/largefiles/
@@ -1202,7 +1202,7 @@ 
         if before == after:
             return result # no need to restore standins
-        merge.update(repo, node=None, branchmerge=False, force=True,
+        merge.update(repo, node='.', branchmerge=False, force=True,
         lfdirstate = lfutil.openlfdirstate(ui, repo)
diff --git a/tests/test-largefiles-update.t b/tests/test-largefiles-update.t
--- a/tests/test-largefiles-update.t
+++ b/tests/test-largefiles-update.t
@@ -152,6 +152,25 @@ 
   $ cat .hglf/large2
+(rollback the parent of the working directory, when the parent of it
+is not branch-tip)
+  $ hg update -q -C 1
+  $ cat .hglf/large1
+  58e24f733a964da346e2407a2bee99d9001184f5
+  $ cat .hglf/large2
+  1deebade43c8c498a3c8daddac0244dc55d1331d
+  $ echo normalX > normalX
+  $ hg add normalX
+  $ hg commit -m 'will be rollback-ed soon'
+  $ hg rollback -q
+  $ cat .hglf/large1
+  58e24f733a964da346e2407a2bee99d9001184f5
+  $ cat .hglf/large2
+  1deebade43c8c498a3c8daddac0244dc55d1331d
 Test that "hg status" shows status of largefiles correctly just after
 automated commit like rebase/transplant