path: root/t/
diff options
authorJunio C Hamano <>2016-06-01 20:56:08 (GMT)
committerJunio C Hamano <>2016-06-01 21:00:33 (GMT)
commite256eec79d375649ecda7cc21b527386898794a4 (patch)
treea91775ad696e4e812fe90d9231bca29922f27525 /t/
parentd2554c7207896136ad2033776efd29578592a3fb (diff)
t5500 & t7403: lose bash-ism "local"
In t5500::check_prot_host_port_path(), diagport is not a variable used elsewhere and the function is not recursively called so this can simply lose the "local", which may not be supported by shell (besides, the function liberally clobbers other variables without making them "local"). t7403::reset_submodule_urls() overrides the "root" variable used in the test framework for no good reason; its use is not about temporarily relocating where the test repositories are created. This assignment can be made not to clobber the variable by moving them into the subshells it already uses. Its value is always $TRASH_DIRECTORY, so we could use it instead there, and this function that is called only once and its two subshells may not be necessary (instead, the caller can use "git -C $there config" and set a value that is derived from $TRASH_DIRECTORY), but this is a minimum fix that is needed to lose "local". Helped-by: John Keeping <> Helped-by: Jeff King <> Signed-off-by: Junio C Hamano <>
Diffstat (limited to 't/')
1 files changed, 2 insertions, 2 deletions
diff --git a/t/ b/t/
index 79bc135..5503ec0 100755
--- a/t/
+++ b/t/
@@ -62,13 +62,13 @@ test_expect_success 'change submodule' '
reset_submodule_urls () {
- local root
- root=$(pwd) &&
+ root=$(pwd) &&
cd super-clone/submodule &&
git config remote.origin.url "$root/submodule"
) &&
+ root=$(pwd) &&
cd super-clone/submodule/sub-submodule &&
git config remote.origin.url "$root/submodule"