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

Re: git cherry-pick --continue?

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 10, 2010, 22:01 UTC
Message-ID
<7v63644uoq.fsf@alter.siamese.dyndns.org>
In-Reply-To
<20100210212408.GB7728@coredump.intra.peff.net>
Jeff King <peff@peff.net> writes:
> Hmm. I was thinking "am" was the odd man out, but really there are only
> two sequencer commands that I noted: rebase and am. So you could perhaps
> argue that rebase should also learn "--resolved". Or am I forgetting
> one?

I don't think you are forgetting anything, except that "am" came first with "resolved".

The focus of the verb is "I declare I am finished marking the resolution". Taking that declaration and continuing is ultimately "am"'s decision. IOW the user is not telling "am" to continue---the difference is subtle, but it was a conscious design decision.

"rebase --continue" came later, and I think its focus is placed more heavily on the instruction side ("Please continue"), and not on the declaration side ("I now have marked the resolution for all paths").

This causes people sometimes to want to see it "continue", even when then haven't marked the resolved paths as resolved. I personally think the focus is misplaced.

But that is just a philosophical difference ;-).
Previous: Sverre RabbelierNext: Junio C Hamano
Message 5 of 22 in “git cherry-pick --continue?”
  1. Sverre RabbelierFeb 10, 2010
  2. Jeff KingFeb 10, 2010
  3. Jeff KingFeb 10, 2010
  4. Sverre RabbelierFeb 10, 2010
  5. Junio C HamanoFeb 10, 2010
  6. Junio C HamanoFeb 10, 2010
  7. Sverre RabbelierFeb 10, 2010
  8. Junio C HamanoFeb 10, 2010
  9. Sverre RabbelierFeb 10, 2010
  10. Jeff KingFeb 11, 2010
  11. 1/4 cherry-pick: rewrap advice messageJeff King, Feb 11, 2010
  12. 2/4 cherry-pick: refactor commit parsing codeJeff King, Feb 11, 2010
  13. 3/4 cherry-pick: format help message as strbufJeff King, Feb 11, 2010
  14. 4/4 cherry-pick: show commit name instead of sha1Jeff King, Feb 11, 2010
  15. Jeff KingFeb 11, 2010
  16. Jay SoffianFeb 11, 2010
  17. Junio C HamanoFeb 11, 2010
  18. Jeff KingFeb 11, 2010
  19. Jeff KingFeb 11, 2010
  20. Junio C HamanoFeb 11, 2010
  21. Jeff KingFeb 11, 2010
  22. SZEDER GáborFeb 12, 2010

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.