Thread (4 messages) flat view 4 messages, 3 authors, 2016-06-15

Re: [PATCH] unpack-trees: don't update files flagged for deletion

From: Junio C Hamano <hidden>
Date: 2016-06-15 23:05:47

Possibly related (same subject, not in this thread)

David Turner [off-list ref] writes:
Don't update files in the worktree from cache entries which are
flagged with CE_WT_REMOVE.  This is fixes merges in sparse
checkouts.
s/This is/This/;

But more importantly, what is missing is "why" it is a good fix.
i.e. things like:

 - why is it wrong to update what is marked with CE_UPDATE bit to be
   updated in the working tree when CE_WT_REMOVE bit is on?

 - how that incorrect operation breaks which operation in what way?

 - how does the changed code fix that breakage?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help