# HG changeset patch # User Mads Kiilerich # Date 2013-04-18 16:56:18 # Node ID 20b0c49c032c3ea95fdbb2a658d2dc341c6990c0 # Parent 0d67fccc0d43c1b622ddc1b11e8d416d81fc9ea9 largefiles: update should only create a .orig backup of a largefile once A .orig of a standin after the update do that a .orig of the actual largefile is created. The .orig standin was however never removed again and the largefile .orig was thus overwritten again and again. The fix: remove the standin .orig when it is used. diff --git a/hgext/largefiles/lfcommands.py b/hgext/largefiles/lfcommands.py --- a/hgext/largefiles/lfcommands.py +++ b/hgext/largefiles/lfcommands.py @@ -447,6 +447,7 @@ def updatelfiles(ui, repo, filelist=None if (os.path.exists(absstandin + '.orig') and os.path.exists(abslfile)): shutil.copyfile(abslfile, abslfile + '.orig') + util.unlinkpath(absstandin + '.orig') expecthash = lfutil.readstandin(repo, lfile) if (expecthash != '' and (not os.path.exists(abslfile) or diff --git a/tests/test-largefiles.t b/tests/test-largefiles.t --- a/tests/test-largefiles.t +++ b/tests/test-largefiles.t @@ -1111,6 +1111,12 @@ Rollback on largefiles. $ echo mistake > sub2/large7 $ hg revert sub2/large7 + $ cat sub2/large7 + large7 + $ cat sub2/large7.orig + mistake + $ test ! -f .hglf/sub2/large7.orig + $ hg -q update --clean -r null $ hg update --clean getting changed largefiles @@ -1128,18 +1134,16 @@ Rollback on largefiles. large7 $ cat sub2/large7.orig mistake - $ cat .hglf/sub2/large7.orig - 9dbfb2c79b1c40981b258c3efa1b10b03f18ad31 + $ test ! -f .hglf/sub2/large7.orig -demonstrate misfeature: .orig file is overwritten on every update -C, -also when clean: +verify that largefile .orig file no longer is overwritten on every update -C: $ hg update --clean getting changed largefiles 0 largefiles updated, 0 removed 0 files updated, 0 files merged, 0 files removed, 0 files unresolved $ cat sub2/large7.orig - large7 - $ rm sub2/large7.orig .hglf/sub2/large7.orig + mistake + $ rm sub2/large7.orig Now "update check" is happy. $ hg update --check 8