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

[PATCHv5 5/9] submodule-config: introduce parse_generic_submodule_config

From
Stefan Beller <sbeller@google.com>
Date
Nov 25, 2015, 01:14 UTC
Message-ID
<1448414096-22021-6-git-send-email-sbeller@google.com>
In-Reply-To
<1448414096-22021-1-git-send-email-sbeller@google.com>

This rewrites parse_config to distinguish between configs specific to one submodule and configs which apply generically to all submodules. We do not have generic submodule configs yet, but the next patch will introduce "submodule.jobs".

Signed-off-by: Stefan Beller <sbeller@google.com>
---
 submodule-config.c | 41 ++++++++++++++++++++++++++++++++---------
 1 file changed, 32 insertions(+), 9 deletions(-)
diff --git a/submodule-config.c b/submodule-config.c
index b826841..29e21b2 100644
--- a/submodule-config.c
+++ b/submodule-config.c
@@ -234,17 +234,22 @@ struct parse_config_parameter {
 	int overwrite;
 };
 
-static int parse_config(const char *var, const char *value, void *data)
+static int parse_generic_submodule_config(const char *key,
+					  const char *var,
+					  const char *value,
+					  struct parse_config_parameter *me)
 {
-	struct parse_config_parameter *me = data;
-	struct submodule *submodule;
-	int subsection_len, ret = 0;
-	const char *subsection, *key;
-
-	if (parse_config_key(var, "submodule", &subsection,
-			     &subsection_len, &key) < 0 || !subsection_len)
-		return 0;
+	return 0;
+}
 
+static int parse_specific_submodule_config(const char *subsection, int subsection_len,
+					   const char *key,
+					   const char *var,
+					   const char *value,
+					   struct parse_config_parameter *me)
+{
+	int ret = 0;
+	struct submodule *submodule;
 	submodule = lookup_or_create_by_name(me->cache,
 					     me->gitmodules_sha1,
 					     subsection, subsection_len);
@@ -314,6 +319,24 @@ static int parse_config(const char *var, const char *value, void *data)
 	return ret;
 }
 
+static int parse_config(const char *var, const char *value, void *data)
+{
+	struct parse_config_parameter *me = data;
+	int subsection_len;
+	const char *subsection, *key;
+
+	if (parse_config_key(var, "submodule", &subsection,
+			     &subsection_len, &key) < 0)
+		return 0;
+
+	if (!subsection_len)
+		return parse_generic_submodule_config(key, var, value, me);
+	else
+		return parse_specific_submodule_config(subsection,
+						       subsection_len, key,
+						       var, value, me);
+}
+
 static int gitmodule_sha1_from_commit(const unsigned char *commit_sha1,
 				      unsigned char *gitmodules_sha1)
 {
-- 
2.6.1.261.g0d9c4c1
Previous: Stefan BellerNext: Stefan Beller
Message 6 of 10 in “[PATCHv5 0/9] Expose submodule parallelism to the user”
  1. Stefan BellerNov 25, 2015
  2. 1/9 run_processes_parallel: delimit intermixed task outputStefan Beller, Nov 25, 2015
  3. 2/9 submodule-config: keep update strategy aroundStefan Beller, Nov 25, 2015
  4. 3/9 submodule-config: drop check against NULLStefan Beller, Nov 25, 2015
  5. 4/9 submodule-config: remove name_and_item_from_varStefan Beller, Nov 25, 2015
  6. 5/9 submodule-config: introduce parse_generic_submodule_configStefan Beller, Nov 25, 2015
  7. 6/9 fetching submodules: respect `submodule.fetchJobs` config optionStefan Beller, Nov 25, 2015
  8. 7/9 git submodule update: have a dedicated helper for cloningStefan Beller, Nov 25, 2015
  9. 8/9 submodule update: expose parallelism to the userStefan Beller, Nov 25, 2015
  10. 9/9 clone: allow an explicit argument for parallel submodule clonesStefan Beller, Nov 25, 2015

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.