Re: [PATCH v6 00/39] SHA-256, part 3/3
- From
Jeff King <peff@peff.net>
- Date
- Mar 10, 2021, 17:08 UTC
- Message-ID
- <YEj9J5I7eInl3BAx@coredump.intra.peff.net>
- In-Reply-To
- <CABPp-BFFLichkBY6Jh2qzE-=Md2eU5tZzHm4aVQVnVpJFD7b0w@mail.gmail.com>
On Wed, Mar 10, 2021 at 08:36:15AM -0800, Elijah Newren wrote:
Show 6 quoted lines
> Honestly not sure it's worth bothering fixing filter-branch at this > point, given the strong WARNINGs we added to it. If anything, I'd > just add another warning to the pile for this issue. If folks really > still want/need filter-branch once SHA-256 is the only option, I'm > sure they'll be motivated to make the necessary fixes (or just use/fix > filter-repo's filter-branch-ish).
You're probably right that it's not a good use of time, but I just sent a series fixing it. It was one of those cases where I was like "oh, it's probably just X". And then "no, wait...", and before I knew it I had wasted an hour fixing filter-branch. ;)
-Peff