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

[RFC/H] "git diff --submodule" showing submodule work tree dirtiness

From
Junio C Hamano <gitster@pobox.com>
Date
Jan 25, 2010, 18:05 UTC
Message-ID
<7vwrz62h7g.fsf_-_@alter.siamese.dyndns.org>
In-Reply-To
<4B5C547C.7000604@web.de>
Jens Lehmann <Jens.Lehmann@web.de> writes:
Show 16 quoted lines
> Right now "git diff --submodule" doesn't show the dirty status of a
> submodule at all (like it does when using it without that option and
> having paid the cost to get the necessary information). So IMHO something
> like the patch below should go into 1.7.0 to fix that. When applied the
> output looks like this:
>
> Submodule sub 3f35670..3f35670-dirty:
>
> which is now consistent with the output of "git diff" without that option:
>
> diff --git a/sub b/sub
> --- a/sub
> +++ b/sub
> @@ -1 +1 @@
> -Subproject commit 3f356705649b5d566d97ff843cf193359229a453
> +Subproject commit 3f356705649b5d566d97ff843cf193359229a453-dirty

I think this is sensible; I queued it to 'pu' to give chance to submodule users comment on it, but I am inclined to say it should be part of 1.7.0 for consistency.

Previous: Jens Lehmann
Message 8 of 8 in “What's cooking in git.git (Jan 2010, #07; Fri, 22)”
  1. Junio C HamanoJan 23, 2010
  2. Jakub NarebskiJan 23, 2010
  3. Jens LehmannJan 23, 2010
  4. Junio C HamanoJan 23, 2010
  5. Junio C HamanoJan 24, 2010
  6. Junio C HamanoJan 24, 2010
  7. Jens LehmannJan 24, 2010
  8. [RFC/H] "git diff --submodule" showing submodule work tree dirtinessJunio C Hamano, Jan 25, 2010

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.