Thread (61 messages) 61 messages, 5 authors, 2018-02-16

Re: [PATCH v7 31/31] merge-recursive: ensure we write updates for directory-renamed file

From: Stefan Beller <hidden>
Date: 2018-02-05 21:58:41

On Tue, Jan 30, 2018 at 3:25 PM, Elijah Newren [off-list ref] wrote:
When a file is present in HEAD before the merge and the other side of the
merge does not modify that file, we try to avoid re-writing the file and
making it stat-dirty.  However, when a file is present in HEAD before the
merge and was in a directory that was renamed by the other side of the
merge, we have to move the file to a new location and re-write it.
Update the code that checks whether we can skip the update to also work in
the presence of directory renames.

Signed-off-by: Elijah Newren <redacted>
---
Now I did not just review the tests, but also the code of this series.
(though I think I was more detail oriented on Friday)
All patches that had no comments are fine with me,
I left some comments on others.

Thanks,
Stefan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help