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

[PATCH v2 0/2] fix cases of rebase -i ignoring core.commentchar

From
Eric Sunshine <sunshine@sunshineco.com>
Date
Aug 18, 2013, 17:12 UTC
Message-ID
<1376845970-31204-1-git-send-email-sunshine@sunshineco.com>

This is a reroll of [1] which fixes a couple cases where rebase -i ignores core.commentchar. One case was an oversight from the original patch which added core.commentchar support to rebase -i, and is already in 'maint'. The other is a regression introduced by [2] which is still in 'next'.

In this reroll, the single patch from v1 is split into two: patch 1/2 is suitable for 'maint'; and patch 2/2 is for 'next'. There are no other changes.

I wanted to add tests to t3404 for these bugs but couldn't figure out how to do it using the external interface of rebase -i. I was able to verify before and after behavior by adding temporary echo's to the code in order to observe the "internal" functioning.

[1]: http://git.661346.n2.nabble.com/PATCH-rebase-i-fix-cases-ignoring-core-commentchar-td7593965.html [2]: http://git.661346.n2.nabble.com/PATCH-0-3-fix-interactive-rebase-short-SHA-1-collision-bug-td7593673.html#a7593676

Eric Sunshine (2):
  rebase -i: fix skip_unnecessary_picks() to respect core.commentchar
  rebase -i: fix core.commentchar regression
 git-rebase--interactive.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
-- 
1.8.4.rc3.500.gc3113b0
Next: Eric Sunshine
Message 1 of 5 in “fix cases of rebase -i ignoring core.commentchar”
  1. 0/2 fix cases of rebase -i ignoring core.commentcharEric Sunshine, Aug 18, 2013
  2. 1/2 rebase -i: fix skip_unnecessary_picks() to respect core.commentcharEric Sunshine, Aug 18, 2013
  3. 2/2 rebase -i: fix core.commentchar regressionEric Sunshine, Aug 18, 2013
  4. Junio C HamanoAug 18, 2013
  5. Eric SunshineAug 19, 2013

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.