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

Re: [PATCH] merge-file: warn for implicit 'myers' algorithm

From
Antonin Delpeuch <antonin@delpeuch.eu>
Date
Jul 5, 2024, 20:14 UTC
Message-ID
<1fd4be07-47ba-4db4-b8aa-860c23b5dd39@delpeuch.eu>
In-Reply-To
<CABPp-BEspjHqNXSAwptgxP059qOFU6MzwAd23-893Nw99ft_Ew@mail.gmail.com>
Thanks both.

It sounds like there is a strong case for at least making "merge-file" honour the `diff.algorithm` config variable, possibly with more fine-grained config variables as suggested by Junio. I'll try to make a patch for that.

Elijah, if any of your work on improving the histogram diff can be shared (for instance by writing down a description of the issues you'd like to address), I would be interested to have a look and perhaps help out if doable. But as far as I can tell there seems to be a lot of fans of the existing algorithm already, and such heuristics can never be perfect, so hopefully we can make the switch in a not too distant future.

Best,
Antonin
Previous: Elijah NewrenNext: Junio C Hamano
Message 6 of 9 in “merge-file: warn for implicit 'myers' algorithm”
  1. merge-file: warn for implicit 'myers' algorithmAntonin Delpeuch via GitGitGadget, Jul 3, 2024
  2. Junio C HamanoJul 3, 2024
  3. Antonin DelpeuchJul 3, 2024
  4. Junio C HamanoJul 3, 2024
  5. Elijah NewrenJul 5, 2024
  6. Antonin DelpeuchJul 5, 2024
  7. Junio C HamanoJul 6, 2024
  8. Antonin DelpeuchJul 6, 2024
  9. Junio C HamanoJul 6, 2024

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.