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

Re: merge strategy request

From
Martin Langhoff <martin.langhoff@gmail.com>
Date
Mar 26, 2007, 00:11 UTC
Message-ID
<46a038f90703251711x6ecfa3e2k19b7d36717fad44e@mail.gmail.com>
In-Reply-To
<7vejnc28i3.fsf@assigned-by-dhcp.cox.net>
On 3/26/07, Junio C Hamano <junkio@cox.net> wrote:
> But a more interesting point is "under what condition".  If the
> condition is "when GIT_MERGE environment is defined", it is not
> much better than what Dscho suggested, as the external merger
> would be called for all paths.

Well well... no. Unless you mean all paths that need diff3 merging (that is, all paths that changed on both sides). And in that case, the script itself can have the "change policy depending on file extension or phase of the moon" machinery. I think it makes it more powerful because it can have specialised logic for this -- where the git machinery is going to be limited to simpler rules (like file extension).

In any case, it is a _different_ mechanism to saying "all changed paths on either side". I don't know if there is even an interesting case for that.

> it is rather an unfortunate norm.  I am not opposed to give
> git-merge-recursive a command line option to tell it to merge
> ignoring the whitespace-only changes, when it does the 3-way
> file-level merge internally.
That'd be great to have. How about both? ;-)
cheers,
m
Previous: Junio C Hamano
Message 10 of 10 in “merge strategy request”
  1. David LangMar 24, 2007
  2. Johannes SchindelinMar 25, 2007
  3. David LangMar 25, 2007
  4. Linus TorvaldsMar 25, 2007
  5. David LangMar 25, 2007
  6. Junio C HamanoMar 25, 2007
  7. Martin LanghoffMar 25, 2007
  8. David LangMar 25, 2007
  9. Junio C HamanoMar 25, 2007
  10. Martin LanghoffMar 26, 2007

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.