From: Taylor Blau <me@ttaylorr.com>
Date: Fri, 29 Jan 2021 01:15:20 GMT
Subject: Re: [PATCH v3 03/10] builtin/index-pack.c: allow stripping arbitrary extensions
Message-ID: <YBNhqDTG5dEcIbWP@nand.local>
In-Reply-To: <YBNWsJBDUpNiRXP4@coredump.intra.peff.net>

On Thu, Jan 28, 2021 at 07:28:32PM -0500, Jeff King wrote:
> On Mon, Jan 25, 2021 at 06:37:22PM -0500, Taylor Blau wrote:
> > Prepare for stripping off suffixes other than '.pack' by making the
> > suffix to strip a parameter of derive_filename(). In order to make this
> > consistent with the "suffix" parameter which does not begin with a ".",
> > an additional check in derive_filename.
>
> Maybe "add" missing from the final line?

Oops, yeah. I'm happy to send a replacement to help with queuing, if the
maintainer thinks that would be helpful.

Thanks,
Taylor