Merge pull request #2977 from pks-t/submodule-sync-url-fix Fix git_submodule_sync writing URL to wrong key.
diff --git a/src/submodule.c b/src/submodule.c
index 567ab74..c3bb6fe 100644
--- a/src/submodule.c
+++ b/src/submodule.c
@@ -1091,10 +1091,10 @@ int git_submodule_sync(git_submodule *sm)
/* return error from reading submodule config */;
else if ((error = lookup_head_remote_key(&remote_name, smrepo)) < 0) {
giterr_clear();
- error = git_buf_sets(&key, "branch.origin.remote");
+ error = git_buf_sets(&key, "remote.origin.url");
} else {
error = git_buf_join3(
- &key, '.', "branch", remote_name.ptr, "remote");
+ &key, '.', "remote", remote_name.ptr, "url");
git_buf_free(&remote_name);
}
diff --git a/tests/submodule/modify.c b/tests/submodule/modify.c
index 3d7269b..fcb4c22 100644
--- a/tests/submodule/modify.c
+++ b/tests/submodule/modify.c
@@ -118,11 +118,11 @@ void test_submodule_modify__sync(void)
/* check that submodule config is updated */
assert_submodule_url_is_synced(
- sm1, "submodule."SM1".url", "branch.origin.remote");
+ sm1, "submodule."SM1".url", "remote.origin.url");
assert_submodule_url_is_synced(
- sm2, "submodule."SM2".url", "branch.origin.remote");
+ sm2, "submodule."SM2".url", "remote.origin.url");
assert_submodule_url_is_synced(
- sm3, "submodule."SM3".url", "branch.origin.remote");
+ sm3, "submodule."SM3".url", "remote.origin.url");
git_submodule_free(sm1);
git_submodule_free(sm2);