From: Junio C Hamano Date: Thu, 24 Feb 2022 20:05:25 GMT Subject: Re: [PATCH v3 3/4] test-lib: make $GIT_BUILD_DIR an absolute path Message-ID: In-Reply-To: <220224.86bkywaakn.gmgdl@evledraar.gmail.com> Ævar Arnfjörð Bjarmason writes: > So you're OK with the assumption/method being used here, but would > prefer if we also added an explicit check/"exit 1"? E.g.: > > if test "$TEST_DIRECTORY" = "${TEST_DIRECTORY%/t}" > then > echo "PANIC: Running in a $TEST_DIRECTORY that doesn't end in '/t'?" >&2 > exit 1 > fi Exactly.