This patch teaches "git rm" how to remove submodules from the
.gitmodules file. The .gitmodules update is handled by an undocumented
subcommand to "git submodule" named "rmconfig".
Signed-off-by: Peter Collingbourne <peter@xxxxxxxxx>
---
Documentation/git-rm.txt | 5 ++-
builtin/rm.c | 25 ++++++++++-
git-submodule.sh | 45 +++++++++++++++++++-
...09-submodule-mv.sh => t7409-submodule-mv-rm.sh} | 15 ++++++-
4 files changed, 85 insertions(+), 5 deletions(-)
rename t/{t7409-submodule-mv.sh => t7409-submodule-mv-rm.sh} (82%)
diff --git a/Documentation/git-rm.txt b/Documentation/git-rm.txt
index c21d19e..81c1bbd 100644
--- a/Documentation/git-rm.txt
+++ b/Documentation/git-rm.txt
@@ -7,7 +7,7 @@ git-rm - Remove files from the working tree and from the index
SYNOPSIS
--------
-'git rm' [-f | --force] [-n] [-r] [--cached] [--ignore-unmatch] [--quiet] [--] <file>...
+'git rm' [-f | --force] [-n] [-r] [-M] [--cached] [--ignore-unmatch] [--quiet] [--] <file>...
DESCRIPTION
-----------
@@ -49,6 +49,9 @@ OPTIONS
Allow recursive removal when a leading directory name is
given.
+-M::
+ Do not try to remove submodule entry in .gitmodules
+
\--::
This option can be used to separate command-line options from
the list of files, (useful when filenames might be mistaken
diff --git a/builtin/rm.c b/builtin/rm.c
index 02ee259..3c26a43 100644
--- a/builtin/rm.c
+++ b/builtin/rm.c
@@ -9,6 +9,7 @@
#include "cache-tree.h"
#include "tree-walk.h"
#include "parse-options.h"
+#include "run-command.h"
static const char * const builtin_rm_usage[] = {
"git rm [options] [--] <file>...",
@@ -139,7 +140,7 @@ static int check_local_mod(unsigned char *head, int index_only)
static struct lock_file lock_file;
static int show_only = 0, force = 0, index_only = 0, recursive = 0, quiet = 0;
-static int ignore_unmatch = 0;
+static int ignore_unmatch = 0, skip_module_update = 0;
static struct option builtin_rm_options[] = {
OPT__DRY_RUN(&show_only),
@@ -149,6 +150,8 @@ static struct option builtin_rm_options[] = {
OPT_BOOLEAN('r', NULL, &recursive, "allow recursive removal"),
OPT_BOOLEAN( 0 , "ignore-unmatch", &ignore_unmatch,
"exit with a zero status even if nothing matched"),
+ OPT_BOOLEAN('M', NULL, &skip_module_update,
+ "don't update submodule entries"),
OPT_END(),
};
@@ -276,5 +279,25 @@ int cmd_rm(int argc, const char **argv, const char *prefix)
die("Unable to write new index file");
}
+ if (!skip_module_update)
+ for (i = 0; i < list.nr; i++) {
+ if (S_ISGITLINK(list.mode[i])) {
+ const char *path = list.name[i];
+
+ const char *argv_submodule[] = {
+ "submodule", "rmconfig", NULL, NULL, NULL, NULL
+ };
+ int argc = 2;
+
+ if (index_only)
+ argv_submodule[argc++] = "--cached";
+
+ argv_submodule[argc++] = "--";
+ argv_submodule[argc++] = path;
+
+ run_command_v_opt(argv_submodule, RUN_GIT_CMD);
+ }
+ }
+
return 0;
}
diff --git a/git-submodule.sh b/git-submodule.sh
index 3fd067a..6d9c08b 100755
--- a/git-submodule.sh
+++ b/git-submodule.sh
@@ -876,6 +876,49 @@ cmd_mvconfig()
die "Could not update .gitmodules entry for $name"
git add .gitmodules || die "Could not add .gitmodules to index"
}
+#
+# Removes the entry in .gitmodules to remove a submodule.
+# This command is called by "git rm" for each submodule it removes.
+#
+cmd_rmconfig()
+{
+ while test $# -ne 0
+ do
+ case "$1" in
+ --cached)
+ index_only=1
+ shift
+ ;;
+ --)
+ shift
+ break
+ ;;
+ *)
+ break
+ ;;
+ esac
+ done
+ path="$1"
+
+ if test -z "$index_only"
+ then
+ name=$(module_name "$path") || exit
+ git config -f .gitmodules --remove-section submodule."$name" ||
+ die "Could not update .gitmodules entry for $name"
+ git add .gitmodules || die "Could not add .gitmodules to index"
+ else
+ git cat-file -p :0:.gitmodules > .git/gitmodules.index ||
+ { rm .git/gitmodules.index; die "Could not extract .gitmodules from index"; }
+ name=$(module_name "$path" .git/gitmodules.index) || { rm .git/gitmodules.index; exit; }
+ git config -f .git/gitmodules.index --remove-section submodule."$name" ||
+ { rm .git/gitmodules.index; die "Could not update .gitmodules entry for $name"; }
+ blob=$(git hash-object -w --stdin < .git/gitmodules.index) ||
+ { rm .git/gitmodules.index; die "Could not create blob for .gitmodules"; }
+ rm .git/gitmodules.index || die "Could not remove temporary .gitmodules file"
+ git update-index --cacheinfo 100644 "$blob" .gitmodules ||
+ die "Could not add .gitmodules to index"
+ fi
+}
# This loop parses the command line arguments to find the
# subcommand name to dispatch. Parsing of the subcommand specific
@@ -886,7 +929,7 @@ cmd_mvconfig()
while test $# != 0 && test -z "$command"
do
case "$1" in
- add | foreach | init | update | status | summary | sync | mvconfig)
+ add | foreach | init | update | status | summary | sync | mvconfig | rmconfig)
command=$1
;;
-q|--quiet)
diff --git a/t/t7409-submodule-mv.sh b/t/t7409-submodule-mv-rm.sh
similarity index 82%
rename from t/t7409-submodule-mv.sh
rename to t/t7409-submodule-mv-rm.sh
index 9eb3fb1..91b7866 100755
--- a/t/t7409-submodule-mv.sh
+++ b/t/t7409-submodule-mv-rm.sh
@@ -3,9 +3,9 @@
# Copyright (c) 2010 Peter Collingbourne
#
-test_description='git submodule mv
+test_description='git submodule mv, rm
-These tests exercise the "git mv" command for submodules.
+These tests exercise the "git mv" and "git rm" commands for submodules.
'
. ./test-lib.sh
@@ -91,4 +91,15 @@ test_expect_success 'move multiple submodules at once' '
)
'
+test_expect_success 'remove multiple submodules at once' '
+ (cd super &&
+ git rm -r test\ dir &&
+ test ! -d test\ dir/unreg &&
+ test -d test\ dir/reg2 &&
+ test -z "$(git ls-files test\ dir/unreg)" &&
+ test -z "$(git ls-files test\ dir/reg2)" &&
+ test -z "$(git config -f .gitmodules submodule.reg.path)"
+ )
+'
+
test_done
--
1.6.5
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html