Re: [PATCH v3 00/14] Remove more index compatibility macros

From: Derrick Stolee <stolee@gmail.com>
Date: Jan 25, 2021, 13:04 UTC
Message-ID: <a6906459-3a93-ecad-c460-b6c763819389@gmail.com>
In-Reply-To: <xmqq1retclzr.fsf@gitster.c.googlers.com>

On 1/10/2021 2:03 AM, Junio C Hamano wrote:

Show 13 quoted lines

> "Derrick Stolee via GitGitGadget" <gitgitgadget@gmail.com> writes:
> 
>> 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