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

Re: Possible bug in "git rebase" (non-interactive) with regards to post-rewrite

From
Thomas Rast <trast@student.ethz.ch>
Date
Dec 26, 2010, 10:38 UTC
Message-ID
<201012261138.21059.trast@student.ethz.ch>
In-Reply-To
<7vd3ot48ke.fsf@alter.siamese.dyndns.org>
Junio C Hamano wrote:
Show 5 quoted lines
> When "rebase --skip" is used to skip the last patch in the series, the
> code to wrap up the rewrite by copying the notes from old to new commits
> and also by running the post-rewrite hook was bypassed.
> 
> Signed-off-by: Junio C Hamano <gitster@pobox.com>
[...]
Show 10 quoted lines
> -if test "$this" -gt "$last"
> -then
> -	say Nothing to do.
> -	rm -fr "$dotest"
> -	exit
> -fi
> -
>  while test "$this" -le "$last"
>  do
>  	msgnum=`printf "%0${prec}d" $this`
Ack, thanks for patching this.

(I think it's saner anyway not to say "nothing to do" when the user would expect "we're all done here"...)

-- 
Thomas Rast
trast@{inf,student}.ethz.ch
Previous: Junio C Hamano
Message 4 of 4 in “Possible bug in "git rebase" (non-interactive) with regards to post-rewrite”
  1. Mihai RusuOct 18, 2010
  2. Junio C HamanoDec 23, 2010
  3. Junio C HamanoDec 23, 2010
  4. Thomas RastDec 26, 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.