Re: [PATCH v3 00/14] Remove more index compatibility macros
From: Derrick Stolee <hidden>
Date: 2021-01-26 20:54:08
On 1/10/2021 2:03 AM, Junio C Hamano wrote:
"Derrick Stolee via GitGitGadget" [off-list ref] writes:quoted
UPDATE: this is now based on ag/merge-strategies-in-c to avoid conflicts in 'seen'. The changes in builtin/rm.c still conflict with mt/rm-sparse-checkout, but that branch seems to be waiting for a clearer plan on some corner cases. I thought about ejecting it, but 'rm' still uses ce_match_stat(), so just dropping the patch gives less of a final stake at the end of the series. (I'm still open to it, if necessary.)I haven't read this latest iteration myself yet beyond the cover letter, but tonight's 'seen' has this queued near its tip. I expect it would either stay there or occasionally ejected, until the base topic solidifies a bit more.
Junio, Please drop this series for now. I'll be introducing a new series soon that will collide with it and this is a lower priority. I'll probably come back to revisit removing these macros, but I'll do so one builtin at a time when others are not modifying them at the same time. Thanks, -Stolee