From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 21 Aug 2025 20:32:22 GMT
Subject: Re: [PATCH 3/3] bulk-checkin: wire repository variable
Message-ID: <xmqqms7sza95.fsf@gitster.g>
In-Reply-To: <rhkhmaozjaogm34xfwagagy2qifsq5p4fbtb23nhdu7jfg2627@qkrv57gg2k3u>

Justin Tobler <jltobler@gmail.com> writes:

> Thinking about this more, we should probably just require
> `index_blob_bulk_checkin()` be provided a transaction. Callers will need
> to ensure a transaction is running so that a `struct
> bulk_checkin_packfile` gets set up, but this shouldn't be a big deal.

Thanks for thinking this through.  I think reducing the number of
oddball callers-from-sideways leads us to good code hygiene.

> With this we could easily just propagate the transaction for all these
> function as you suggested.
>
> I'll do this in the next version. Thanks!
>
> -Justin