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

Re: git cherry-pick conflict error message is deceptive when cherry-picking multiple commits

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Aug 18, 2016, 14:15 UTC
Message-ID
<alpine.DEB.2.20.1608181612340.4924@virtualbox>
In-Reply-To
<f58933df-352a-9d2b-a35a-9c48cb2d958e@nokia.com>
Hi Stephen,
On Wed, 17 Aug 2016, Stephen Morton wrote:
> > multiple_commits = (todo_list->next) != NULL;
> > Why not "last_commit" instead of "multiple_commits"?
> 
> Because it *isn't*.

Personally, I do prefer to have the simpler instruction if the commit happens to be the last one cherry-picked.

I do not follow your argument that it makes sense to decide based on the total number of commits that were to be cherry-picked whether to use the more cumbersome or the easier command.

Instead, I think the user should be given the simplest advice that gets them out of the fix most quickly. And if `git commit --amend` is good enough, we should not advise to use `git cherry-pick --continue`, and that is that.

Ciao, Johannes

Previous: Junio C HamanoNext: Remi Galan Alfonso
Message 6 of 8 in “Re: git cherry-pick conflict error message is deceptive when cherry-picking multiple commits”
  1. Stephen MortonAug 10, 2016
  2. Christian CouderAug 14, 2016
  3. Stephen MortonAug 17, 2016
  4. Remi Galan AlfonsoAug 17, 2016
  5. Junio C HamanoAug 17, 2016
  6. Johannes SchindelinAug 18, 2016
  7. Remi Galan AlfonsoAug 16, 2016
  8. Johannes SchindelinAug 17, 2016

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.