git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v4 6/9] submodule: use the 'submodule--helper config' command

From
Antonio Ospite <ao2@ao2.it>
Date
Aug 24, 2018, 13:29 UTC
Message-ID
<20180824132951.8000-7-ao2@ao2.it>
In-Reply-To
<20180824132951.8000-1-ao2@ao2.it>

Use the 'submodule--helper config' command in git-submodules.sh to avoid referring explicitly to .gitmodules by the hardcoded file path.

This makes it possible to access the submodules configuration in a more controlled way.

Signed-off-by: Antonio Ospite <ao2@ao2.it>
---
 git-submodule.sh | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/git-submodule.sh b/git-submodule.sh
index f7fd80345c..9e47dc9574 100755
--- a/git-submodule.sh
+++ b/git-submodule.sh
@@ -72,7 +72,7 @@ get_submodule_config () {
 	value=$(git config submodule."$name"."$option")
 	if test -z "$value"
 	then
-		value=$(git config -f .gitmodules submodule."$name"."$option")
+		value=$(git submodule--helper config submodule."$name"."$option")
 	fi
 	printf '%s' "${value:-$default}"
 }
@@ -283,11 +283,11 @@ or you are unsure what this means choose another name with the '--name' option."
 	git add --no-warn-embedded-repo $force "$sm_path" ||
 	die "$(eval_gettext "Failed to add submodule '\$sm_path'")"
 
-	git config -f .gitmodules submodule."$sm_name".path "$sm_path" &&
-	git config -f .gitmodules submodule."$sm_name".url "$repo" &&
+	git submodule--helper config submodule."$sm_name".path "$sm_path" &&
+	git submodule--helper config submodule."$sm_name".url "$repo" &&
 	if test -n "$branch"
 	then
-		git config -f .gitmodules submodule."$sm_name".branch "$branch"
+		git submodule--helper config submodule."$sm_name".branch "$branch"
 	fi &&
 	git add --force .gitmodules ||
 	die "$(eval_gettext "Failed to register submodule '\$sm_path'")"
-- 
2.18.0
Previous: Antonio OspiteNext: Antonio Ospite
Message 10 of 14 in “Make submodules work if .gitmodules is not checked out”
  1. 0/9 Make submodules work if .gitmodules is not checked outAntonio Ospite, Aug 24, 2018
  2. 4/9 t7411: be nicer to future tests and really clean things upAntonio Ospite, Aug 24, 2018
  3. 1/9 submodule: add a print_config_from_gitmodules() helperAntonio Ospite, Aug 24, 2018
  4. Ævar Arnfjörð BjarmasonAug 24, 2018
  5. Antonio OspiteAug 24, 2018
  6. Antonio OspiteSep 4, 2018
  7. 9/9 submodule: support reading .gitmodules when it's not in the working treeAntonio Ospite, Aug 24, 2018
  8. 2/9 submodule: factor out a config_set_in_gitmodules_file_gently functionAntonio Ospite, Aug 24, 2018
  9. 3/9 t7411: merge tests 5 and 6Antonio Ospite, Aug 24, 2018
  10. 6/9 submodule: use the 'submodule--helper config' commandAntonio Ospite, Aug 24, 2018
  11. 8/9 submodule: add a helper to check if it is safe to write to .gitmodulesAntonio Ospite, Aug 24, 2018
  12. 7/9 t7506: clean up .gitmodules properly before setting up new scenarioAntonio Ospite, Aug 24, 2018
  13. 5/9 submodule--helper: add a new 'config' subcommandAntonio Ospite, Aug 24, 2018
  14. Ævar Arnfjörð BjarmasonAug 24, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.