]> git.ipfire.org Git - thirdparty/git.git/commitdiff
t6423-merge-rename-directories.sh: use the $(...) construct
authorElia Pinto <gitter.spiros@gmail.com>
Sun, 13 Mar 2022 17:28:29 +0000 (17:28 +0000)
committerJunio C Hamano <gitster@pobox.com>
Sun, 13 Mar 2022 19:03:32 +0000 (19:03 +0000)
The Git CodingGuidelines prefer the $(...) construct for command
substitution instead of using the backquotes `...`.

The backquoted form is the traditional method for command
substitution, and is supported by POSIX.  However, all but the
simplest uses become complicated quickly.  In particular, embedded
command substitutions and/or the use of double quotes require
careful escaping with the backslash character.

The patch was generated by:

for _f in $(find . -name "*.sh")
do
shellcheck -i SC2006 -f diff ${_f} | ifne git apply -p2
done

and then carefully proof-read.

Signed-off-by: Elia Pinto <gitter.spiros@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t6423-merge-rename-directories.sh

index 5b81a130e9c45086a8bb618d8b8eeed16324ad20..479db32cd629c1b118eb892ee541700fb9100afd 100755 (executable)
@@ -4421,14 +4421,14 @@ test_setup_12c1 () {
 
                git checkout A &&
                git mv node2/ node1/ &&
-               for i in `git ls-files`; do echo side A >>$i; done &&
+               for i in $(git ls-files); do echo side A >>$i; done &&
                git add -u &&
                test_tick &&
                git commit -m "A" &&
 
                git checkout B &&
                git mv node1/ node2/ &&
-               for i in `git ls-files`; do echo side B >>$i; done &&
+               for i in $(git ls-files); do echo side B >>$i; done &&
                git add -u &&
                test_tick &&
                git commit -m "B"
@@ -4511,7 +4511,7 @@ test_setup_12c2 () {
 
                git checkout A &&
                git mv node2/ node1/ &&
-               for i in `git ls-files`; do echo side A >>$i; done &&
+               for i in $(git ls-files); do echo side A >>$i; done &&
                git add -u &&
                echo leaf5 >node1/leaf5 &&
                git add node1/leaf5 &&
@@ -4520,7 +4520,7 @@ test_setup_12c2 () {
 
                git checkout B &&
                git mv node1/ node2/ &&
-               for i in `git ls-files`; do echo side B >>$i; done &&
+               for i in $(git ls-files); do echo side B >>$i; done &&
                git add -u &&
                echo leaf6 >node2/leaf6 &&
                git add node2/leaf6 &&
@@ -4759,7 +4759,7 @@ test_setup_12f () {
                echo g >dir/subdir/tweaked/g &&
                echo h >dir/subdir/tweaked/h &&
                test_seq 20 30 >dir/subdir/tweaked/Makefile &&
-               for i in `test_seq 1 88`; do
+               for i in $(test_seq 1 88); do
                        echo content $i >dir/unchanged/file_$i
                done &&
                git add . &&