Re: [PATCH v2 12/12] testsvn: respect `init.defaultBranch`

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Jun 23, 2020, 21:07 UTC
Message-ID: <nycvar.QRO.7.76.6.2006232305160.54@tvgsbejvaqbjf.bet>
In-Reply-To: <20200616135121.GJ666057@coredump.intra.peff.net>

Hi Peff,

On Tue, 16 Jun 2020, Jeff King wrote:

Show 10 quoted lines

> On Mon, Jun 15, 2020 at 12:50:16PM +0000, Johannes Schindelin via GitGitGadget wrote:
>
> > From: Johannes Schindelin <johannes.schindelin@gmx.de>
> >
> > The default name of the main branch in new repositories can now be
> > configured. The `testsvn` remote helper translates the remote Subversion
> > repository's branch name `trunk` to the hard-coded name `master`.
> > Clearly, the intention was to make the name align with Git's detaults.
>
> s/detaults/defaults/ :)

:-)

Will fix.

> I'd agree that moving this to Git's default name makes sense.

Okay.

Show 5 quoted lines

> Though my overall preference is still to delete this whole testsvn thing
> entirely (I have some other pending tree-wide changes that are being
> held up by it, too). After getting "would you mind holding off until..."
> from Jonathan in [1], I've been waiting almost 2 years. Maybe now is the
> time?

I wouldn't mind dropping `testsvn`, seeing as there are fewer and fewer users of `git svn` (and even those are unlikely to switch to `testsvn`, should that ever become production-ready).

Having said that, this is an orthogonal issue to the purpose of this patch series. And I would really like to get this patch series into a shape that can be merged down to `next` soon.

Thank you, Dscho

Show 5 quoted lines

>
> -Peff
>
> [1] https://lore.kernel.org/git/20180818052605.GA241538@aiede.svl.corp.google.com/
>