[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#66475] (was 66436)] [PATCH 2/2] git-download: Add support for Git L
From: |
Maxim Cournoyer |
Subject: |
[bug#66475] (was 66436)] [PATCH 2/2] git-download: Add support for Git Large File Storage (LFS). |
Date: |
Tue, 31 Oct 2023 16:30:07 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Hi,
Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:
> Hi,
>
> Simon Tournier <zimon.toutoune@gmail.com> writes:
>
>> Hi,
>>
>> I reorder Ludo’s answers for clarity.
>>
>> On Sat, 14 Oct 2023 at 19:12, Ludovic Courtès <ludo@gnu.org> wrote:
>>
>>> Also, it is crucial for the “builtin:git-download” semantics to be the
>>> same across all installations and to be very stable.
>>
>> I agree and that’s one strong argument for me.
>>
>>> Overall, I feel like LFS support, if needed, should be “on the side”,
>>> with a custom ‘git-fetch/lfs’ or something along these lines (just like
>>> we have ‘url-fetch/tarbomb’).
>>
>> I agree with this direction. Most of the fetching methods should not be
>> implemented with “builtin“ but the converse, git-fetch/lfs.
>>
>> ( Even, I think the current git-fetch using out-of-band builtin
>> recently introduced should be a ’git-fetch/bootstrap’ method, and
>> git-fetch using in-band should be the default method. )
>
> Thanks for tipping in, I'll look into adding a whole new 'git-fetch/lfs'
> procedure.
Done, see v2. The 'orcus' package makes use of it -- its test suite is
broken without it.
--
Thanks,
Maxim