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

Re: improved diff tool

From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date
Aug 31, 2018, 09:28 UTC
Message-ID
<875zzqdgwg.fsf@evledraar.gmail.com>
In-Reply-To
<CAJmvCd1BwPNgZM+g7c6An-EESMHDOSGQe7RjkAAz36eNY3fvGA@mail.gmail.com>
On Thu, Aug 30 2018, Piers Titus van der Torren wrote:
> Is there interest to incorporate this algorithm in the main git
> codebase? And if so, any hints on how to proceed?

This looks very nice, it would be great to have it in git. I think it's more useful to focus on getting it into the git C codebase, the user base of gitk/git-gui is tiny by comparison.

Aside from what others have mentioned, maybe this commit helps to get an idea of how to do stuff like this: 3cde4e02ee ("diff: retire "compaction" heuristics", 2016-12-23)

I.e. it's one of our previous diff algorithms being ripped out. Grepping for the various --diff-algorithm=* options in "man git-diff" in the source/history should also give good ideas of where to start hooking in.

I see from your name / some brief spying on you via Google that you might be in the Amsterdam area. I'm not very familiar with the diff part of the codebase, but if you'd like I'd be happy to meet in person sometime to help you get started on various other stuff in git.git if you'd like.

Previous: Christian Couder
Message 5 of 5 in “improved diff tool”
  1. Piers Titus van der TorrenAug 30, 2018
  2. Johannes SchindelinAug 30, 2018
  3. Stefan BellerAug 30, 2018
  4. Christian CouderAug 31, 2018
  5. Ævar Arnfjörð BjarmasonAug 31, 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.