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

[PATCH 3/3] log: show merge commit when --cc is given

From
Junio C Hamano <gitster@pobox.com>
Date
Aug 20, 2015, 22:43 UTC
Message-ID
<1440110591-12941-4-git-send-email-gitster@pobox.com>
In-Reply-To
<1440110591-12941-1-git-send-email-gitster@pobox.com>

We defaulted to ignoring merge diffs because long long ago, in a galaxy far away, we didn't have a great way to show the diffs. The whole "--cc" option goes back to January '06 and commit d8f4790e6fe7 ("diff-tree --cc: denser combined diff output for a merge commit"). And before that option - so for about 8 months - we had no good way to show the diffs of merges in a good dense way. So the whole "don't show diffs for merges by default" actually made a lot of sense originally, because our merge diffs were not very useful.

And this was carried forward to this day. "git log --cc" still ignores merge commits, and you need to say "git log -m --cc" to view a sensible rendition of merge and non-merge commits, even with the previous change to make "--cc" imply "-p".

Teach "git log" that "--cc" means the user wants to see interesting changes in merge commits by turning "-m" on.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 builtin/log.c | 4 ++++
 1 file changed, 4 insertions(+)
diff --git a/builtin/log.c b/builtin/log.c
index e37c27a..0cdd889 100644
--- a/builtin/log.c
+++ b/builtin/log.c
@@ -637,6 +637,10 @@ static void log_setup_revisions_tweak(struct rev_info *rev,
 	/* Turn --cc/-c into -p --cc/-c when -p was not given */
 	if (!rev->diffopt.output_format && rev->combine_merges)
 		rev->diffopt.output_format = DIFF_FORMAT_PATCH;
+
+	/* Turn -m on when --cc/-c was given */
+	if (rev->combine_merges)
+		rev->ignore_merges = 0;
 }
 
 int cmd_log(int argc, const char **argv, const char *prefix)
-- 
2.5.0-546-gb1bbc0d
Previous: Junio C HamanoNext: Junio C Hamano
Message 4 of 6 in “Showing merges easier with "git log"”
  1. 0/3 Showing merges easier with "git log"Junio C Hamano, Aug 20, 2015
  2. 1/3 log: rename "tweak" helpersJunio C Hamano, Aug 20, 2015
  3. 2/3 log: when --cc is given, default to -p unless told otherwiseJunio C Hamano, Aug 20, 2015
  4. 3/3 log: show merge commit when --cc is givenJunio C Hamano, Aug 20, 2015
  5. 4/3 builtin/log.c: minor reformatJunio C Hamano, Aug 21, 2015
  6. log: make '-p' imply '-m --cc'Junio C Hamano, Aug 21, 2015

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.