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

Re: Request for adding a "clean" merge strategy for a double-commit merge to deal with conflicts separately

From
Michal Suchánek <msuchanek@suse.de>
Date
Jul 16, 2020, 17:25 UTC
Message-ID
<20200716172549.GJ32107@kitsune.suse.cz>
In-Reply-To
<CAD9n_qh0y84HC6sX1OxXWWv8dDMMA_tPv9zRknePVivQq_rfww@mail.gmail.com>
On Thu, Jul 16, 2020 at 09:45:24PM +0430, Alireza wrote:
Show 5 quoted lines
> Hi,
> 
> Even though the merge commit's message includes conflicted files by
> default, the *resolution* itself is lost, that is, it's hard or
> impossible to review how the author *resolved* said conflicts.

No, the merge commit includes the resolution. You can compare with all parents, and even perform the same merge locally using your strategy and tooling of choice and compare the result with what the merge author committed as resolution.

> 
> The proposal is that an option like `-X clean` would commit a clean
> merge and leave out any conflicting hunks in the tree for a follow-up
> commit to resolve conflicts.

I don't see any value above performing the same merge locally and comparing with the committed resolution. Can you please elaborate on the expected format of the merge that does add some value?

Thanks
Michal
Previous: AlirezaNext: Junio C Hamano
Message 2 of 8 in “Request for adding a "clean" merge strategy for a double-commit merge to deal with conflicts separately”
  1. AlirezaJul 16, 2020
  2. Michal SuchánekJul 16, 2020
  3. Junio C HamanoJul 16, 2020
  4. AlirezaJul 21, 2020
  5. Elijah NewrenJul 21, 2020
  6. AlirezaJul 21, 2020
  7. Elijah NewrenJul 21, 2020
  8. Junio C HamanoJul 21, 2020

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.