Re: [PATCH v5 4/4] init: provide useful advice about init.defaultBranch
- From
SZEDER Gábor <szeder.dev@gmail.com>
- Date
- Feb 3, 2021, 05:20 UTC
- Message-ID
- <20210203052018.GE2091@szeder.dev>
- In-Reply-To
- <xmqqsg6eds5e.fsf@gitster.c.googlers.com>
On Tue, Feb 02, 2021 at 02:25:01PM -0800, Junio C Hamano wrote:
Show 16 quoted lines
> SZEDER Gábor <szeder.dev@gmail.com> writes:
>
> >> - "${GIT_TEST_INSTALLED:-$GIT_EXEC_PATH}/git$X" init \
> >> + "${GIT_TEST_INSTALLED:-$GIT_EXEC_PATH}/git$X" -c \
> >> + init.defaultBranch="${GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME-master}" \
> >> + init \
> >> "--template=$GIT_BUILD_DIR/templates/blt/" >&3 2>&4 ||
> >
> > This breaks GIT_TEST_INSTALLED when the given Git version doesn't yet
> > contain 8b1fa77867 (Allow passing of configuration parameters in the
> > command line, 2010-03-26):
>
> Yes, but test-installed is about running an old version of Git on
> the current Git test suite. In the tests we make liberal use of
> "git -c <var>=<val>", which such an old version of Git has no chance
> of passing. So...I've never run the whole test suite with GIT_TEST_INSTALLED for the exact reason you mentioned. However, I do regularly use it with custom tests using only commands and options that already existed as early as v1.6.0 (my go-to "old" version) to check when some interesting or puzzling behavior appeared, and did just that yesterday as well.