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

Re: diff.ignoreSubmoudles config setting broken?

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 8, 2017, 20:54 UTC
Message-ID
<xmqq8tof32x9.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<CAGZ79kbwMhL-ZnL-iYwPH=tWa8cNQbEGOYYQBw6OzFCMhOWE-w@mail.gmail.com>
Stefan Beller <sbeller@google.com> writes:
Show 10 quoted lines
> On Wed, Mar 8, 2017 at 7:07 AM, Sebastian Schuberth
> <sschuberth@gmail.com> wrote:
>>
>> + Jens
>>
>
> + Jacob Keller, who touched submodule diff display code last.
> (I am thinking of fd47ae6a, diff: teach diff to display submodule
> difference with an inline diff, 2016-08-31), which is first release as
> part of v2.11.0 (that would fit your observance)
Between these two:
	git -c diff.ignoresubmodules=all diff
	git diff --ignore-submodules=all

one difference is that the latter disables reading extra config from .gitmodules (!) while the former makes the command honor it.

This comes from aee9c7d6 ("Submodules: Add the new "ignore" config option for diff and status", 2010-08-06), which is ancient and predates even v2.0, so if you see problems with v2.12 or v2.11 but not with older ones, that would rule out this theory.

Previous: Stefan BellerNext: Stefan Beller
Message 7 of 12 in “diff.ignoreSubmoudles config setting broken?”
  1. Sebastian SchuberthMar 8, 2017
  2. Jeff KingMar 8, 2017
  3. Sebastian SchuberthMar 8, 2017
  4. Jeff KingMar 8, 2017
  5. Sebastian SchuberthMar 8, 2017
  6. Stefan BellerMar 8, 2017
  7. Junio C HamanoMar 8, 2017
  8. Stefan BellerMar 8, 2017
  9. Junio C HamanoMar 8, 2017
  10. Stefan BellerMar 9, 2017
  11. Sebastian SchuberthMar 9, 2017
  12. Jacob KellerMar 8, 2017

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.