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

Re: [PATCH] RFC: commit: add a commit.all-ignore-submodules config option

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Jan 7, 2020, 00:05 UTC
Message-ID
<20200107000551.GE92456@google.com>
In-Reply-To
<CAMxuvayT8FtovVnWU4bjQCP26drN37yuPG2+G2jAUsm0Ns_AYA@mail.gmail.com>
Marc-André Lureau wrote:
> On Sat, Jan 4, 2020 at 4:45 AM Jonathan Nieder <jrnieder@gmail.com> wrote:
>> Marc-André Lureau wrote:
Show 10 quoted lines
>>> One of my most frequent mistake is to commit undesired submodules
>>> changes when doing "commit -a", and I have seen a number of people doing
>>> the same mistake in various projects. I wish there would be a config to
>>> change this default behaviour.
>>
>> Can you say more about the overall workflow this is part of?  What
>> causes the submodules to change state in the first place here?
>
> The most common case is, I guess, when you work on different branches
> that have different (compatible) versions of the submodules.

Ah! This is because "git checkout" defaults to --no-recurse-submodules, which is a terrible default.

Does "git config submodule.recurse true" help? If so, we can look into which it would take to flip that default.

Thanks, Jonathan

Previous: Marc-André LureauNext: Marc-André Lureau
Message 4 of 5 in “RFC: commit: add a commit.all-ignore-submodules config option”
  1. RFC: commit: add a commit.all-ignore-submodules config optionmarcandre.lureau@redhat.com, Jan 3, 2020
  2. Jonathan NiederJan 4, 2020
  3. Marc-André LureauJan 4, 2020
  4. Jonathan NiederJan 7, 2020
  5. Marc-André LureauJan 7, 2020

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.