Re: [PATCH v8 00/17] Upstreaming the Scalar command

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Nov 22, 2021, 12:21 UTC
Message-ID: <nycvar.QRO.7.76.6.2111221317390.63@tvgsbejvaqbjf.bet>
In-Reply-To: <CABPp-BG=fcKq2Ng2gan3HbBGcT7WCMhtZCP6m2xjA5BSuTekOg@mail.gmail.com>

Hi Elijah,

On Sat, 20 Nov 2021, Elijah Newren wrote:

Show 12 quoted lines

> On Fri, Nov 19, 2021 at 3:03 PM Johannes Schindelin via GitGitGadget
> <gitgitgadget@gmail.com> wrote:
> >
> > tl;dr: This series contributes the core part of the Scalar command to
> > the Git project. This command provides an opinionated way to create
> > and configure Git repositories with a focus on very large
> > repositories.
>
> I thought after
> https://lore.kernel.org/git/nycvar.QRO.7.76.6.2110062241150.395@tvgsbejvaqbjf.bet/
> that you'd update merge.renames to true on what is now patch 7.  Did
> you end up changing your mind, or was this overlooked?

Oops! Thank you so much for the reminder.

Will fix. I do not plan on sending out a new iteration for a few more days because I do not want to send lots of patches to the list right now, reviewer bandwidth seems to be stretched quite a bit already.

> Other than that, this round looks good to me.  (I have no opinion on
> the build system integration, other than that I like it being optional
> and not installed by default.)

Yes, I very much wanted to keep this optional and as well-encapsulated as possible for the moment. (Hence the way it integrates with Git's build process.)

Thank you for chiming in!

Ciao, Dscho