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

Re: [PATCH] Fix rebase -p --onto

From
Johannes Sixt <j.sixt@viscovery.net>
Date
Jul 17, 2009, 08:47 UTC
Message-ID
<4A603AA4.7020800@viscovery.net>
In-Reply-To
<4A6038E8.1090402@viscovery.net>
Johannes Sixt schrieb:
Show 20 quoted lines
> I have used rebase -i -p in the past to rewrite history that involves
> merges of topic branches like this:
> 
>   ---------Y--M--M--F     <-- master
>              /  /
>   ----a--a--a  /
>               /
>   --b--b--b--b
> 
> where F is a fixup that I want to insert between Y and M, and I thought
> rebase -i -p was intended for this use-case.
> 
> With this in mind, I do not see why should we distinguish the cases. I
> would even go so far to say that this is OK:
> 
>   X---a---o---o---M            X---a
>        \         /    ===>          \
>         x---x---x                    x---x---x
>                                               \
>   Y                            Y---a'--o'--o'--M'

because it is a user error: wrong tool for the task. The right tool would perhaps be git-sequencer.

-- Hannes (I hit the send button too early by accident, sorry for the excessive quoting in the previous mail.)

Previous: Johannes SixtNext: Greg Price
Message 5 of 9 in “Fix rebase -p --onto”
  1. Fix rebase -p --ontoGreg Price, Jul 16, 2009
  2. Johannes SixtJul 17, 2009
  3. Junio C HamanoJul 17, 2009
  4. Johannes SixtJul 17, 2009
  5. Johannes SixtJul 17, 2009
  6. Greg PriceJul 17, 2009
  7. Johannes SchindelinJul 17, 2009
  8. Greg PriceJul 17, 2009
  9. Greg PriceJul 17, 2009

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.