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

[PATCHv5 0/9] Expose submodule parallelism to the user

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

This is also available at github/stefanbeller/git submodule-parallel-update It applies on top of a merge of sb/submodule-parallel-fetch (including the "run-command: detect finished children by closed pipe rather than waitpid" sent yesterday) with "Merge branch 'rs/daemon-plug-child-leak' into sb/submodule-parallel-update" (d075d2604c0f9204)

The diff to v4 is below, just a change in braces.

My main motivation for resending this patch series is have it available to reviewers of the series I am going to send next, which builds on top of this series.

Thanks, Stefan

Stefan Beller (9):
  run_processes_parallel: delimit intermixed task output
  submodule-config: keep update strategy around
  submodule-config: drop check against NULL
  submodule-config: remove name_and_item_from_var
  submodule-config: introduce parse_generic_submodule_config
  fetching submodules: respect `submodule.fetchJobs` config option
  git submodule update: have a dedicated helper for cloning
  submodule update: expose parallelism to the user
  clone: allow an explicit argument for parallel submodule clones
 Documentation/config.txt        |   7 ++
 Documentation/git-clone.txt     |   6 +-
 Documentation/git-submodule.txt |   7 +-
 builtin/clone.c                 |  19 +++-
 builtin/fetch.c                 |   2 +-
 builtin/submodule--helper.c     | 239 ++++++++++++++++++++++++++++++++++++++++
 git-submodule.sh                |  54 ++++-----
 run-command.c                   |   4 +
 submodule-config.c              | 109 +++++++++++-------
 submodule-config.h              |   3 +
 submodule.c                     |   5 +
 t/t5526-fetch-submodules.sh     |  14 +++
 t/t7400-submodule-basic.sh      |   4 +-
 t/t7406-submodule-update.sh     |  27 +++++
 14 files changed, 417 insertions(+), 83 deletions(-)
 
Interdiff to series v4: (ignoring the merge base as that changed, too)
diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c
index 662d329..254824a 100644
--- a/builtin/submodule--helper.c
+++ b/builtin/submodule--helper.c
@@ -487,9 +487,9 @@ static int update_clone(int argc, const char **argv, const char *prefix)
                return 1;
        }
 
-       for_each_string_list_item(item, &pp.projectlines) {
+       for_each_string_list_item(item, &pp.projectlines)
                utf8_fprintf(stdout, "%s", item->string);
-       }
+
        return 0;
 }
 
-- 
2.6.1.261.g0d9c4c1
Next: Stefan Beller
Message 1 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.