# HG changeset patch # User Pulkit Goyal <7895pulkit@gmail.com> # Date 1598263534 -19800 # Node ID 7684124726632e5d2ad6bc9822403c92b18d7b2f # Parent 8c02c6262a5ea25923455334040556be03881260 merge: store cases when a file is absent post merge in commitinfo Some merges can result in file being absent form working directory. This can be one of file was kept deleted or file was removed by merge code. User might revert the file back before committing. In such cases we will like to have better handling and create new filenodes. We store this info in mergestate as commitinfo so that we can use it while committing to create new filenode if required. Differential Revision: https://phab.mercurial-scm.org/D9003 diff -r 8c02c6262a5e -r 768412472663 mercurial/merge.py --- a/mercurial/merge.py Mon Sep 14 15:08:15 2020 +0530 +++ b/mercurial/merge.py Mon Aug 24 15:35:34 2020 +0530 @@ -923,6 +923,15 @@ mresult.addfile( f, mergestatemod.ACTION_REMOVE, None, b'other deleted', ) + if branchmerge: + # the file must be absent after merging, + # howeber the user might make + # the file reappear using revert and if they does, + # we force create a new node + mresult.addcommitinfo( + f, b'merge-removal-candidate', b'yes' + ) + else: # file not in ancestor, not in remote mresult.addfile( f, @@ -1033,6 +1042,12 @@ None, b'local not present, remote unchanged', ) + if branchmerge: + # the file must be absent after merging + # however the user might make + # the file reappear using revert and if they does, + # we force create a new node + mresult.addcommitinfo(f, b'merge-removal-candidate', b'yes') if repo.ui.configbool(b'experimental', b'merge.checkpathconflicts'): # If we are merging, look for path conflicts. diff -r 8c02c6262a5e -r 768412472663 tests/test-merge-criss-cross.t --- a/tests/test-merge-criss-cross.t Mon Sep 14 15:08:15 2020 +0530 +++ b/tests/test-merge-criss-cross.t Mon Aug 24 15:35:34 2020 +0530 @@ -682,7 +682,9 @@ extra: the-file (merge-removal-candidate = yes) #else $ hg debugmergestate - no merge state found + local (working copy): adfd88e5d7d3d3e22bdd26512991ee64d59c1d8f + other (merge rev): e9b7081317232edce73f7ad5ae0b7807ff5c326a + extra: the-file (merge-removal-candidate = yes) #endif (merging a deletion with keeping → conflict) @@ -697,7 +699,9 @@ $ ls -1 other-file $ hg debugmergestate - no merge state found + local (working copy): adfd88e5d7d3d3e22bdd26512991ee64d59c1d8f + other (merge rev): 5e3eccec60d88f94a7ba57c351f32cb24c15fe0c + extra: the-file (merge-removal-candidate = yes) (merging two deletion together → no conflict) @@ -730,7 +734,9 @@ extra: the-file (merge-removal-candidate = yes) #else $ hg debugmergestate - no merge state found + local (working copy): a4e0e44229dc130be2915b92c957c093f8c7ee3e + other (merge rev): e9b7081317232edce73f7ad5ae0b7807ff5c326a + extra: the-file (merge-removal-candidate = yes) #endif (merging a deletion with keeping → conflict) @@ -745,7 +751,9 @@ $ ls -1 other-file $ hg debugmergestate - no merge state found + local (working copy): a4e0e44229dc130be2915b92c957c093f8c7ee3e + other (merge rev): 5e3eccec60d88f94a7ba57c351f32cb24c15fe0c + extra: the-file (merge-removal-candidate = yes) (merging two "keeping" together → no conflict) @@ -795,7 +803,9 @@ other-file the-file $ hg debugmergestate - no merge state found + local (working copy): 5e3eccec60d88f94a7ba57c351f32cb24c15fe0c + other (merge rev): a4e0e44229dc130be2915b92c957c093f8c7ee3e + extra: the-file (merge-removal-candidate = yes) (merging two "keeping" together → no conflict) @@ -843,5 +853,7 @@ extra: the-file (merge-removal-candidate = yes) #else $ hg debugmergestate - no merge state found + local (working copy): e9b7081317232edce73f7ad5ae0b7807ff5c326a + other (merge rev): a4e0e44229dc130be2915b92c957c093f8c7ee3e + extra: the-file (merge-removal-candidate = yes) #endif