Re: [PATCH v2 0/4] Check .gitmodules when using packfile URIs
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Feb 22, 2021, 20:12 UTC
- Message-ID
- <xmqqft1n7tgn.fsf@gitster.g>
- In-Reply-To
- <cover.1614021092.git.jonathantanmy@google.com>
Jonathan Tan <jonathantanmy@google.com> writes:
Show 11 quoted lines
> Here's v2. I think I've addressed all the review comments, including > passing the index-pack args as separate arguments (to avoid the > necessity to somehow encode in order to get rid of spaces), and by using > a custom error function instead of a specific option in fsck. > > This applies on master. I mentioned earlier [1] that I was planning to > implement this on Ævar's fsck API improvements, but after looking at the > latest v2, I see that it omits patch 11 from v1 (which is the one I > need), so what I've done is to use a string check in the meantime. > > [1] https://lore.kernel.org/git/20210219004612.1181920-1-jonathantanmy@google.com/
I only looked at the difference between this round and what is in 'seen', but everything looked reasonable to me (including the code that is near NEEDSWORK comment, and what the comment said).
Will queue. Thanks.