averello/git-mirror
1
0
Fork 0
mirror of https://github.com/git/git.git synced 2025-12-12 20:36:24 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'jc/submodule-add' into jch

Browse Source 
"git submodule add" to add a submodule under <name> segfaulted,
when a submodule.<name>.something is already in .gitmodules file
without defining where its submodule.<name>.path is, which has been
corrected.

Comments?

* jc/submodule-add:
  submodule add: sanity check existing .gitmodules
This commit is contained in:
Junio C Hamano
2025-12-12 15:53:06 +09:00
parent 5b838e9a7d dd8e8c786e
commit a21836a42a
2 changed files with 29 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
builtin/submodule--helper.c
View File

@@ -1898,6 +1898,13 @@ static int determine_submodule_update_strategy(struct repository *r,
const char *val;
int ret;
/*
* NEEDSWORK: audit and ensure that update_submodule() has right
* to assume that submodule_from_path() above will always succeed.
*/
if (!sub)
BUG("update_submodule assumes a submodule exists at path (%s)",
path);
key = xstrfmt("submodule.%s.update", sub->name);
if (update) {
@@ -3523,14 +3530,15 @@ static int module_add(int argc, const char **argv, const char *prefix,
}
}
if(!add_data.sm_name)
if (!add_data.sm_name)
add_data.sm_name = add_data.sm_path;
existing = submodule_from_name(the_repository,
null_oid(the_hash_algo),
add_data.sm_name);
if (existing && strcmp(existing->path, add_data.sm_path)) {
if (existing && existing->path &&
strcmp(existing->path, add_data.sm_path)) {
if (!force) {
die(_("submodule name '%s' already used for path '%s'"),
add_data.sm_name, existing->path);

19
t/t7400-submodule-basic.sh
View File

@@ -48,6 +48,25 @@ test_expect_success 'submodule deinit works on empty repository' '
git submodule deinit --all
'
test_expect_success 'submodule add with incomplete .gitmodules' '
test_when_finished "rm -f expect actual" &&
test_when_finished "git config remove-section submodule.one" &&
test_when_finished "git rm -f one .gitmodules" &&
git init one &&
git -C one commit --allow-empty -m one-initial &&
git config -f .gitmodules submodule.one.ignore all &&
git submodule add ./one &&
for var in ignore path url
do
git config -f .gitmodules --get "submodule.one.$var" ||
return 1
done >actual &&
test_write_lines all one ./one >expect &&
test_cmp expect actual
'
test_expect_success 'setup - initial commit' '
>t &&
git add t &&