[Request] [+- ] D11788: dirstate: do no use `set_clean` in revert
marmoute (Pierre-Yves David)
phabricator at mercurial-scm.org
Wed Nov 24 11:16:01 UTC 2021
marmoute created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.
REVISION SUMMARY
The current `set_clean` usage is racy (file might be modified between its
restoration and the `set_clean` call.
So we simply leave the file as ambiguous and the next status will fix that.
We still have to make sure the copy information is dropped, so we teach dirstate
how to do that.
The win32txt extension is confused after this because current logic is broken in
more location. However this series will ultimately fix that so we "ignore" it
for now. Fixing it now is complicated without some extra fix landing later.
REPOSITORY
rHG Mercurial
BRANCH
default
REVISION DETAIL
https://phab.mercurial-scm.org/D11788
AFFECTED FILES
mercurial/cmdutil.py
mercurial/dirstate.py
tests/test-revert.t
tests/test-win32text.t
CHANGE DETAILS
diff --git a/tests/test-win32text.t b/tests/test-win32text.t
--- a/tests/test-win32text.t
+++ b/tests/test-win32text.t
@@ -418,6 +418,7 @@
$ hg revert -a
reverting linefeed
$ hg st -q
+ M linefeed (known-bad-output !)
$ cat linefeed
% just linefeed\r (esc)
diff --git a/tests/test-revert.t b/tests/test-revert.t
--- a/tests/test-revert.t
+++ b/tests/test-revert.t
@@ -320,7 +320,7 @@
$ hg mv --force a b/b
$ hg revert b/b
- $ hg status a b/b
+ $ hg status a b/b --copies
$ cd ..
diff --git a/mercurial/dirstate.py b/mercurial/dirstate.py
--- a/mercurial/dirstate.py
+++ b/mercurial/dirstate.py
@@ -452,19 +452,24 @@
return self._map.copymap
@requires_no_parents_change
- def set_tracked(self, filename):
+ def set_tracked(self, filename, reset_copy=False):
"""a "public" method for generic code to mark a file as tracked
This function is to be called outside of "update/merge" case. For
example by a command like `hg add X`.
+ if reset_copy is set, any existing copy information will be dropped.
+
return True the file was previously untracked, False otherwise.
"""
self._dirty = True
entry = self._map.get(filename)
if entry is None or not entry.tracked:
self._check_new_tracked_filename(filename)
- return self._map.set_tracked(filename)
+ pre_tracked = self._map.set_tracked(filename)
+ if reset_copy:
+ self._map.copymap.pop(filename, None)
+ return pre_tracked
@requires_no_parents_change
def set_untracked(self, filename):
diff --git a/mercurial/cmdutil.py b/mercurial/cmdutil.py
--- a/mercurial/cmdutil.py
+++ b/mercurial/cmdutil.py
@@ -3628,15 +3628,14 @@
prntstatusmsg(b'drop', f)
repo.dirstate.set_untracked(f)
- normal = None
- if node == parent:
- # We're reverting to our parent. If possible, we'd like status
- # to report the file as clean. We have to use normallookup for
- # merges to avoid losing information about merged/dirty files.
- if p2 != repo.nullid:
- normal = repo.dirstate.set_tracked
- else:
- normal = repo.dirstate.set_clean
+ # We're reverting to our parent. If possible, we'd like status
+ # to report the file as clean. We have to be less agressive for merge
+ # merges to avoid losing information about copy introduced by the merge.
+ # This might comes with bugs ?
+ reset_copy = p2 == repo.nullid
+
+ def normal(filename):
+ return repo.dirstate.set_tracked(filename, reset_copy=reset_copy)
newlyaddedandmodifiedfiles = set()
if interactive:
@@ -3727,9 +3726,6 @@
checkout(f)
repo.dirstate.set_tracked(f)
- normal = repo.dirstate.set_tracked
- if node == parent and p2 == repo.nullid:
- normal = repo.dirstate.set_clean
for f in actions[b'undelete'][0]:
if interactive:
choice = repo.ui.promptchoice(
To: marmoute, #hg-reviewers
Cc: mercurial-patches, mercurial-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mercurial-scm.org/pipermail/mercurial-patches/attachments/20211124/df7ed122/attachment-0001.html>
More information about the Mercurial-patches
mailing list