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

[PATCHv3 0/2] Persistent submodule pathspec specification

From
Stefan Beller <sbeller@google.com>
Date
May 26, 2016, 20:47 UTC
Message-ID
<20160526204730.20309-1-sbeller@google.com>
Changes since v2:
 * I replaced one 0 by NULL as pointed out by Ramsay, and reformatted the line
   to stay within 80 characters:
--- a/builtin/clone.c
+++ b/builtin/clone.c
@@ -908,8 +908,10 @@ int cmd_clone(int argc, const char **argv, const char *prefix)
                struct string_list_item *item;
                struct strbuf sb = STRBUF_INIT;
                for_each_string_list_item(item, &init_submodules) {
-                       strbuf_addf(&sb, "submodule.defaultUpdatePath=%s", item->string);
-                       string_list_append(&option_config, strbuf_detach(&sb, 0));
+                       strbuf_addf(&sb, "submodule.defaultUpdatePath=%s",
+                                   item->string);
+                       string_list_append(&option_config,
+                                          strbuf_detach(&sb, NULL));
                }
        }
 

Changes since v1:
 * fixed a broken && chain in a subshell for testing, as pointed out by Eric!

This was part of the former series 'submodule groups'.
However the labeling was ripped out and goes in its own series
sb/pathspec-label.

First we introduce a switch `--init-default-path` for `git submodule update`
which will read the pathspec to initialize the submodules not from the command
line but from `submodule.defaultUpdatePath`, which can be configured permanently.

The second patch utilizes this by having `clone` set that config option
and using that new option when calling to update the submodules.

Thanks,
Stefan

Stefan Beller (2):
  submodule update: add `--init-default-path` switch
  clone: add --init-submodule=<pathspec> switch

 Documentation/config.txt        |   5 ++
 Documentation/git-clone.txt     |  25 +++++---
 Documentation/git-submodule.txt |  11 +++-
 builtin/clone.c                 |  36 ++++++++++-
 git-submodule.sh                |  21 ++++++-
 t/t7400-submodule-basic.sh      | 134 ++++++++++++++++++++++++++++++++++++++++
 6 files changed, 218 insertions(+), 14 deletions(-)
-- 
2.9.0.rc0.2.g145fc64

base-commit: 3a0f269e7c82aa3a87323cb7ae04ac5f129f036b
Next: Stefan Beller
Message 1 of 4 in “[PATCHv3 0/2] Persistent submodule pathspec specification”
  1. Stefan BellerMay 26, 2016
  2. 1/2 submodule update: add `--init-default-path` switchStefan Beller, May 26, 2016
  3. 2/2 clone: add --init-submodule=<pathspec> switchStefan Beller, May 26, 2016
  4. Junio C HamanoMay 26, 2016

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.