Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > This changes the remaining <non-empty?> special snowflake test modes > to <boolean> and gets rid of test_tristate() in favor of the now > standard "boolea" test. > > I'm replying to my "gc: run more pre-detach operations under lock" > thread because one of the things my WIP patches to make gc locking > less sucky depends on is new GIT_TEST_GC_* test modes to test its > racyness, which in turn depends on these cleanups. That sounds like the "gc: run more ..." depends on these (iow, that should be the reply to these, not the other way around)? I am asking because I see obvious value in these "uniformly require <boolean>" consistency change (which could be backward incompatible, but as long as these are GIT_TEST_*, we do not mind too much forcing developers to adjust), but not yet in the "gc: run more ..." one, and do not want these to be taken hostage. Thanks. > > Ævar Arnfjörð Bjarmason (6): > env--helper: new undocumented builtin wrapping git_env_*() > t6040 test: stop using global "script" variable > tests: make GIT_TEST_GETTEXT_POISON a boolean > tests README: re-flow a previously changed paragraph > tests: replace test_tristate with "git env--helper" > tests: make GIT_TEST_FAIL_PREREQS a boolean > > .gitignore | 1 + > Makefile | 1 + > builtin.h | 1 + > builtin/env--helper.c | 74 +++++++++++++++++++++++++++++++++++++++ > ci/lib.sh | 2 +- > gettext.c | 6 ++-- > git-sh-i18n.sh | 4 ++- > git.c | 1 + > po/README | 2 +- > t/README | 12 +++---- > t/lib-git-daemon.sh | 7 ++-- > t/lib-git-svn.sh | 11 +++--- > t/lib-httpd.sh | 15 ++++---- > t/t0000-basic.sh | 10 +++--- > t/t0016-env-helper.sh | 70 ++++++++++++++++++++++++++++++++++++ > t/t0205-gettext-poison.sh | 2 +- > t/t5512-ls-remote.sh | 3 +- > t/t6040-tracking-info.sh | 6 ++-- > t/t7201-co.sh | 2 +- > t/t9902-completion.sh | 2 +- > t/test-lib-functions.sh | 58 +++++------------------------- > t/test-lib.sh | 29 ++++++++++++--- > 22 files changed, 220 insertions(+), 99 deletions(-) > create mode 100644 builtin/env--helper.c > create mode 100755 t/t0016-env-helper.sh