]> git.ipfire.org Git - thirdparty/git.git/commit
env-helper: move this built-in to "test-tool env-helper"
authorÆvar Arnfjörð Bjarmason <avarab@gmail.com>
Thu, 12 Jan 2023 16:03:21 +0000 (17:03 +0100)
committerJunio C Hamano <gitster@pobox.com>
Sun, 15 Jan 2023 02:07:11 +0000 (18:07 -0800)
commit4a1baacd4680e27226892706c7a3fd1e22504572
tree431628d69eecafb7465102df826c91714749cb54
parentc48035d29b4e524aed3a32f0403676f0d9128863
env-helper: move this built-in to "test-tool env-helper"

Since [1] there has been no reason for keeping "git env--helper" a
built-in. The reason it was a built-in to begin with was to support
the GIT_TEST_GETTEXT_POISON mode removed in that commit. I.e. unlike
the rest of "test-tool" it would potentially be called by the
installed git via "git-sh-i18n.sh".

As none of that applies since [1] we should stop carrying this
technical debt, and move it to t/helper/*. As this mostly move-only
change shows this has the nice bonus that we'll stop wasting time
translating the internal-only strings it emits.

Even though this was a built-in, it was intentionally never
documented, see its introduction in [2]. It never saw use outside of
the test suite, except for the "GIT_TEST_GETTEXT_POISON" use-case
noted above.

1. d162b25f956 (tests: remove support for GIT_TEST_GETTEXT_POISON,
   2021-01-20)
2. b4f207f3394 (env--helper: new undocumented builtin wrapping
   git_env_*(), 2019-06-21)

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
.gitignore
Makefile
git.c
t/helper/test-env-helper.c [moved from builtin/env--helper.c with 71% similarity]
t/helper/test-tool.c
t/helper/test-tool.h
t/t0017-env-helper.sh
t/test-lib-functions.sh
t/test-lib.sh