]> git.phdru.name Git - git-scripts.git/commitdiff
Refactor(submodules/remove): Get `GIT_DIR` once
authorOleg Broytman <phd@phdru.name>
Sat, 16 Nov 2024 21:32:36 +0000 (00:32 +0300)
committerOleg Broytman <phd@phdru.name>
Sat, 16 Nov 2024 21:32:36 +0000 (00:32 +0300)
Call `git rev-parse --git-dir` once.
Also replace `$()` with backticks.

submodules/remove

index f9436cead70e12fdd84b414fb746d3c89aff7e83..4906a0be6443ad59351c6d555af41316071e3b67 100755 (executable)
@@ -18,7 +18,7 @@ if [ \! -f .gitmodules ]; then
 fi
 
 name="$1"
-path="$(git config -f .gitmodules --get "submodule.$name.path")"
+path="`git config -f .gitmodules --get "submodule.$name.path"`"
 
 if [ -z "$path" ]; then
     echo "Error: submodule $name does not exist" >&2
@@ -26,13 +26,14 @@ if [ -z "$path" ]; then
 fi
 
 git rm $cached "$path" &&
-rm -rf "`git rev-parse --git-dir`"/modules/"$name" &&
+GIT_DIR=`git rev-parse --git-dir` &&
+rm -rf "$GIT_DIR"/modules/"$name" &&
 git config --remove-section submodule."$name" &&
 
 if [ \! -s .gitmodules ]; then
     git rm -f .gitmodules
 fi &&
 
-if [ -z $(ls -A "`git rev-parse --git-dir`/modules") ]; then
-    exec rmdir "`git rev-parse --git-dir`/modules"
+if [ -z `ls -A "$GIT_DIR/modules"` ]; then
+    exec rmdir "$GIT_DIR/modules"
 fi