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

Re: [PATCH] git gui: Use git diff --submodule when available

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 10, 2009, 22:52 UTC
Message-ID
<7vskbi8mrd.fsf@alter.siamese.dyndns.org>
In-Reply-To
<4B2179E4.3070603@web.de>
Jens Lehmann <Jens.Lehmann@web.de> writes:
Show 21 quoted lines
> Changed the use of submodule summary to diff --submodule because the
> implementation in C is much faster than the submodule script. Also a test
> has been added to make sure that the underlying git supports the diff
> option --submodule (which was introduced in 1.6.6).
>
> Signed-off-by: Jens Lehmann <Jens.Lehmann@web.de>
> ---
> @@ -296,16 +304,6 @@ proc start_show_diff {cont_info {add_opts {}}} {
>  		lappend cmd $path
>  	}
>
> -	if {[string match {160000 *} [lindex $s 2]]
> -        || [string match {160000 *} [lindex $s 3]]} {
> -		set is_submodule_diff 1
> -		if {$w eq $ui_index} {
> -			set cmd [list submodule summary --cached -- $path]
> -		} else {
> -			set cmd [list submodule summary --files -- $path]
> -		}
> -	}
> -
Heh, I didn't even know that it originally came from git-gui ;-)
Previous: Jens Lehmann
Message 2 of 2 in “git gui: Use git diff --submodule when available”
  1. git gui: Use git diff --submodule when availableJens Lehmann, Dec 10, 2009
  2. Junio C HamanoDec 10, 2009

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.