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

Re: [PATCH v3 2/5] read-cache: submodule add need --force given ignore=all configuration

From
Claus Schneider <claus.schneider@eficode.com>
Date
Jan 16, 2026, 16:07 UTC
Message-ID
<CA+GP4bpwMui5RT9ixo8iWyZ6XZDw3GnYhvgiqVmRq8dW7yo5RQ@mail.gmail.com>
In-Reply-To
<f021269b-c89c-41c8-aa08-2ec2443ac7e8@app.fastmail.com>
Thanks for feedback. Well noted and it will be reworded like this:

.... A message is printed (like ignored files) guiding the user to use the --force flag if the user explicitly wants to update the submodule reference.

The reason for the change is to support branch tracking in submodules
 with configuration `submdule.<name>.branch` or similar workflows where the
user is not interested in tracking each update of the sha1 in the submdule.
You can additionally set `submodule.<name>.ignore=all` and the
`git status`will state nothing and, with this patch, the `git add`
does not either - as
the default behaviour. This patch changes the workflow to a more logical
behaviour and similar to workflow for ignored files.

The patch gives more scenarios for submodules to be used effectively with less friction similar to the "repo" tool. A submodule can be added for many different reasons than a hard dependency. It can be added as loosely coupled dependencies whereas the user wants the latest based on the configuration `submoule.<name>.branch`, but are not interested to track each commit in the `super-repo`. Currently it gives friction of handling conflicts between branches even the sha1's are fast-forward and the user just wants the latest in any way. A developer can still add a sha1 explicitly to track updates. ...

// Claus

On Thu, Jan 15, 2026 at 4:43 PM Kristoffer Haugsbakk <kristofferhaugsbakk@fastmail.com> wrote:

Show 35 quoted lines
>
> On Wed, Jan 14, 2026, at 08:47, Claus Schneider(Eficode) via GitGitGadget wrote:
> > From: "Claus Schneider(Eficode)" <claus.schneider@eficode.com>
> >
> > Submodules configured with ignore=all are now skipped during add operations
> > unless overridden by --force and the submodule path is explicitly specified.
> >
> > A message is printed (like ignored files) guiding the user to use the
> > --force flag if the user has explicitely want to update the submodule
> > reference.
> >
> > The reason for the change is support submodule branch tracking or
> > similar and git status states nothing and git add should not add either
> > as a default behaviour. The workflow is more logic and similar to regular
> > ignored files even the submodule is already tracked.
> >
> > The change opens up a lot of possibilities for submodules to be used
> > more freely and simular to the repo tool. A submodule can be added for many
>
> s/simular/similar/ ? Although the sentence doesn’t quite make sense to
> me. Well I’m unfamiliar with the domain anyway. ;)
>
> > more reason and loosely coupled dependencies to the super repo which often
>
> s/more reason/more reasons/ ?
>
> > gives the friction of handle the explicit commits and updates without
>
> s/handle/handling/
>
> > the need for tracking the submodule sha1 by sha1.
> >
> > Signed-off-by: Claus Schneider(Eficode) <claus.schneider@eficode.com>
> > ---
> >[snip]
Previous: Claus SchneiderNext: Claus Schneider
Message 29 of 43 in “git-add : Respect submodule ignore=all and only add changes with --force”
  1. 0/5 git-add : Respect submodule ignore=all and only add changes with --forceClaus Schneider via GitGitGadget, Oct 18, 2025
  2. 1/5 read-cache: update add_files_to_cache to take param ignored_too(--force)Claus Schneider(Eficode) via GitGitGadget, Oct 18, 2025
  3. 2/5 read-cache: let read-cache respect submodule ignore=all and --forceClaus Schneider(Eficode) via GitGitGadget, Oct 18, 2025
  4. 3/5 tests: add new t2206-add-submodule-ignored.sh to test ignore=all scenarioClaus Schneider(Eficode) via GitGitGadget, Oct 18, 2025
  5. 4/5 tests: fix existing tests when add an ignore=all submoduleClaus Schneider(Eficode) via GitGitGadget, Oct 18, 2025
  6. 5/5 Documentation: update add --force and submodule ignore=all configClaus Schneider(Eficode) via GitGitGadget, Oct 18, 2025
  7. Phillip WoodOct 19, 2025
  8. Claus SchneiderOct 20, 2025
  9. Phillip WoodOct 24, 2025
  10. Claus SchneiderNov 13, 2025
  11. 0/5 git-add : Respect submodule ignore=all and only add changes with --forceClaus Schneider via GitGitGadget, Nov 13, 2025
  12. 1/5 read-cache: update add_files_to_cache take param include_ignored_submodulesClaus Schneider(Eficode) via GitGitGadget, Nov 13, 2025
  13. 2/5 read-cache: add/read-cache respect submodule ignore=allClaus Schneider(Eficode) via GitGitGadget, Nov 13, 2025
  14. 3/5 tests: add new t2206-add-submodule-ignored.sh to test ignore=all scenarioClaus Schneider(Eficode) via GitGitGadget, Nov 13, 2025
  15. 4/5 tests: fix existing tests when add an ignore=all submoduleClaus Schneider(Eficode) via GitGitGadget, Nov 13, 2025
  16. 5/5 Documentation: add --include_ignored_submodules + ignore=all configClaus Schneider(Eficode) via GitGitGadget, Nov 13, 2025
  17. Junio C HamanoNov 13, 2025
  18. Junio C HamanoNov 13, 2025
  19. Claus SchneiderNov 14, 2025
  20. 0/5 git-add: Skip submodules with ignore=all unless --force and explicit path usedClaus Schneider via GitGitGadget, Jan 14, 2026
  21. 1/5 read-cache: update add_files_to_cache take param ignored_tooClaus Schneider(Eficode) via GitGitGadget, Jan 14, 2026
  22. 2/5 read-cache: submodule add need --force given ignore=all configurationClaus Schneider(Eficode) via GitGitGadget, Jan 14, 2026
  23. 3/5 tests: t2206-add-submodule-ignored: ignore=all and add --force testsClaus Schneider(Eficode) via GitGitGadget, Jan 14, 2026
  24. 4/5 tests: fix existing tests when add an ignore=all submoduleClaus Schneider(Eficode) via GitGitGadget, Jan 14, 2026
  25. 5/5 Documentation: update add --force option + ignore=all configClaus Schneider(Eficode) via GitGitGadget, Jan 14, 2026
  26. Ben KnobleJan 14, 2026
  27. Kristoffer HaugsbakkJan 15, 2026
  28. Claus SchneiderJan 16, 2026
  29. Claus SchneiderJan 16, 2026
  30. Claus SchneiderFeb 5, 2026
  31. Junio C HamanoFeb 5, 2026
  32. 0/5 git-add: Skip submodules with ignore=all unless --force and explicit path usedClaus Schneider via GitGitGadget, Feb 6, 2026
  33. 1/5 read-cache: update add_files_to_cache take param ignored_tooClaus Schneider(Eficode) via GitGitGadget, Feb 6, 2026
  34. 2/5 read-cache: submodule add need --force given ignore=all configurationClaus Schneider(Eficode) via GitGitGadget, Feb 6, 2026
  35. 3/5 tests: t2206-add-submodule-ignored: ignore=all and add --force testsClaus Schneider(Eficode) via GitGitGadget, Feb 6, 2026
  36. 4/5 tests: fix existing tests when add an ignore=all submoduleClaus Schneider(Eficode) via GitGitGadget, Feb 6, 2026
  37. 5/5 Documentation: update add --force option + ignore=all configClaus Schneider(Eficode) via GitGitGadget, Feb 6, 2026
  38. Junio C HamanoFeb 9, 2026
  39. Junio C HamanoFeb 9, 2026
  40. Junio C HamanoFeb 20, 2026
  41. Kristoffer HaugsbakkMar 4, 2026
  42. Claus SchneiderMar 10, 2026
  43. Claus SchneiderMar 10, 2026

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.