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

Re: git diff-tree ignores --textconv

From
Jeff King <peff@peff.net>
Date
Sep 24, 2018, 01:23 UTC
Message-ID
<20180924012354.GA27439@sigill.intra.peff.net>
In-Reply-To
<13dec8d3-b164-dc24-45e0-8e1bbd550524@stason.org>
On Sun, Sep 23, 2018 at 05:56:03PM -0700, Stas Bekman wrote:
Show 5 quoted lines
> > You probably want "--ext-diff", not "--textconv".
> [...]
> Would it be safe to ask the maintainer of the application to include
> both --textconv and --ext-diff in that 'git diff-tree' call? I only need
> the latter, but someone needed --textconv there as it's in the code.

I think so. The main reason that they are not the default for plumbing commands such as diff-tree is that the output may be quite surprising to anything trying to parse the output. Using --textconv will always produce a diff, but one that may not be applied to the original content. Using --ext-diff may produce output that doesn't even look like a diff, though in practice they often do.

> This is for this package:
> https://github.com/rsmmr/git-notifier

It looks like the output is meant to be read by humans, so yeah, I think it would be fine (and preferred) to enable both.

-Peff
Previous: Stas BekmanNext: Stas Bekman
Message 4 of 5 in “git diff-tree ignores --textconv”
  1. Stas BekmanSep 23, 2018
  2. Jeff KingSep 24, 2018
  3. Stas BekmanSep 24, 2018
  4. Jeff KingSep 24, 2018
  5. Stas BekmanSep 24, 2018

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.