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

Re: [PATCH] A more sensible error message why 'git rebase --continue' failed

From
Steven Grimm <koreth@midwinter.com>
Date
Aug 20, 2007, 13:59 UTC
Message-ID
<46C99E26.2020009@midwinter.com>
In-Reply-To
<2D8BF484-7990-4DDC-9E52-69143D4BA413@fallingsnow.net>
Evan Phoenix wrote:
> How about something like
>
> "Please verify all merge conflicts in all modified files, then run
> 'git add' to resolve them."

Since we're nitpicking language here... "git add" doesn't resolve anything. *You* resolve the conflicts (by editing the file and removing the conflict markers and editing the code, by running git mergetool, etc.) then you run "git add" to tell git that you're done resolving.

If I literally followed the above, I would run "less" on a file, say, "Yep, git flagged all the merge conflicts correctly," then run "git add" figuring it would pop me into some kind of conflict resolution UI. Actually it's worse than that -- the above implies that all modified files will have merge conflicts, which is certainly not true in many cases.

How about:

"Please resolve all merge conflicts, then run 'git add' to tell git you've finished resolving each file."

-Steve
Previous: Evan PhoenixNext: Johannes Sixt
Message 4 of 5 in “A more sensible error message why 'git rebase --continue' failed”
  1. A more sensible error message why 'git rebase --continue' failedEvan Phoenix, Aug 20, 2007
  2. Junio C HamanoAug 20, 2007
  3. Evan PhoenixAug 20, 2007
  4. Steven GrimmAug 20, 2007
  5. Johannes SixtAug 20, 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.