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

Re: Improving auto conflict resolving while merge

From
Jeff King <peff@peff.net>
Date
Sep 8, 2015, 22:03 UTC
Message-ID
<20150908220357.GD24159@sigill.intra.peff.net>
In-Reply-To
<1059528292.20150908210809@yandex.ru>
On Tue, Sep 08, 2015 at 09:08:09PM -0300, Eugen Konkov wrote:
Show 8 quoted lines
> JK> you _could_ argue that those changes are independent But it's close 
> JK> enough that there's a good chance the two need to be reconciled, 
> JK> and a human should at least take a look.
> 
> You are right and your words make sense. But this thought may apply
> for this: We have one method/function about 200 lines. One author make
> change at line 1 of this method and other on 199 line. Both changes
> are done in one method so **human should at least take a look**

Right, there is definitely a concept of "close enough" here, and git cannot catch everything. Semantic changes may even happen across files (e.g., function interface changes). So you do need to rely on things like testing and compilation to verify a merge result.

But I would agree there is room for being able to tune the "closeness" of changes that cause a conflict. Right now that isn't implemented, and I'm not familiar enough with the xdiff merge code to even point you in the right direction.

-Peff
Previous: Eugen Konkov
Message 4 of 4 in “Improving auto conflict resolving while merge”
  1. KESSep 7, 2015
  2. Jeff KingSep 8, 2015
  3. Re[2]: Improving auto conflict resolving while mergeEugen Konkov, Sep 9, 2015
  4. Jeff KingSep 8, 2015

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.