The old behavior of keeping config sections matching the new name caused
problems leading to warnings being emitted by git-remote when renaming
branches where information about tracked remote branches differed. To
fix this any config sections that will conflict with the new name are
removed from the config file. Update test to check for this.
Signed-off-by: Jonas Fonseca <fonseca@xxxxxxx>
---
config.c | 9 ++++++++-
t/t1300-repo-config.sh | 17 +++++++++++++++++
2 files changed, 25 insertions(+), 1 deletions(-)
This command sequence was causing problems for me:
git checkout -b test madcoder/next
git checkout -b test2 spearce/next
git branch -M test
On top of spearce/next ...
diff --git a/config.c b/config.c
index dc3148d..578849a 100644
--- a/config.c
+++ b/config.c
@@ -1013,6 +1013,14 @@ int git_config_rename_section(const char *old_name, const char *new_name)
; /* do nothing */
if (buf[i] == '[') {
/* it's a section */
+ remove = 0;
+ if (new_name != NULL
+ && section_name_match (&buf[i+1], new_name)) {
+ /* Remove any existing occurences of the
+ * new section. */
+ remove = 1;
+ continue;
+ }
if (section_name_match (&buf[i+1], old_name)) {
ret++;
if (new_name == NULL) {
@@ -1026,7 +1034,6 @@ int git_config_rename_section(const char *old_name, const char *new_name)
}
continue;
}
- remove = 0;
}
if (remove)
continue;
diff --git a/t/t1300-repo-config.sh b/t/t1300-repo-config.sh
index 1d2bf2c..63b969e 100755
--- a/t/t1300-repo-config.sh
+++ b/t/t1300-repo-config.sh
@@ -419,6 +419,23 @@ EOF
test_expect_success "section was removed properly" \
"git diff -u expect .git/config"
+cat > .git/config << EOF
+[branch "new-name"]
+ x = 1
+[branch "old-name"]
+ y = 1
+EOF
+
+test_expect_success "rename and remove old section" \
+ 'git config --rename-section branch.old-name branch.new-name'
+
+cat > expect << EOF
+[branch "new-name"]
+ y = 1
+EOF
+
+test_expect_success "rename and remove succeeded" "git diff expect .git/config"
+
rm .git/config
cat > expect << EOF
--
1.5.3.4.1206.g5f96-dirty
--
Jonas Fonseca
-
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
