From: Taylor Blau <me@ttaylorr.com>
Date: Tue, 24 Aug 2021 21:24:45 GMT
Subject: Re: [PATCH v4 05/25] midx: clear auxiliary .rev after replacing the MIDX
Message-ID: <YSVjnSDaBXgXvT9W@nand.local>
In-Reply-To: <xmqqr1eimtrp.fsf@gitster.g>

On Tue, Aug 24, 2021 at 02:12:42PM -0700, Junio C Hamano wrote:
> Taylor Blau <me@ttaylorr.com> writes:
>
> >> This needs to take object_dir into account, no?
> >
> > Yes and no; clear_midx_files_ext() still takes a pointer to a 'struct
> > repository' until we pick up [1].
>
> I was hoping that [1] will become part of this series as a trivial
> clean-up and bugfix, perhaps in its early part.

Sure, that works even better. I'll send a reroll incorporating it as
soon as I finish re-testing.

Thanks,
Taylor