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

Re: [PATCH] Make 'diff C^!' show the same diff as 'show C'

From
Jeff King <peff@peff.net>
Date
Aug 20, 2009, 22:34 UTC
Message-ID
<20090820223424.GA5843@coredump.intra.peff.net>
In-Reply-To
<7vab1up1gz.fsf@alter.siamese.dyndns.org>
On Thu, Aug 20, 2009 at 03:25:16PM -0700, Junio C Hamano wrote:
Show 10 quoted lines
> Thomas Rast <trast@student.ethz.ch> writes:
> 
> > Ideally, we'd like 'git diff C^!' to show the same diff that 'git show C'
> > does (with log.showroot enabled).  This gives easy access to a readable
> > diff for the commit, irrespective of how many parents it has and without
> > any trickery to remove the commit message from the git-show output.
> 
> Not interested yet, as the "git show" discussion is not convincing at all.
> 
> Is the message annoying enough to warrant this change?

I thought the same thing when I saw his message, but reading further, the current output is nonsensical. So if not this patch, we should probably at least complain about bogus input and die (though if it is easy to make it work, why not...).

-Peff
Previous: Junio C HamanoNext: Junio C Hamano
Message 3 of 8 in “Make 'diff C^!' show the same diff as 'show C'”
  1. Make 'diff C^!' show the same diff as 'show C'Thomas Rast, Aug 20, 2009
  2. Junio C HamanoAug 20, 2009
  3. Jeff KingAug 20, 2009
  4. Junio C HamanoAug 20, 2009
  5. Thomas RastAug 20, 2009
  6. Junio C HamanoAug 20, 2009
  7. Junio C HamanoAug 20, 2009
  8. Junio C HamanoAug 20, 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.