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

Re: What's cooking in git.git (Oct 2015, #01; Mon, 5)

From
Torsten Bögershausen <tboegi@web.de>
Date
Oct 6, 2015, 04:45 UTC
Message-ID
<561351FD.9020102@web.de>
In-Reply-To
<xmqqwpv0x6op.fsf@gitster.mtv.corp.google.com>
On 06.10.15 00:59, Junio C Hamano wrote:
Show 9 quoted lines
> * gr/rebase-i-drop-warn (2015-10-02) 2 commits
>  - rebase-i: loosen over-eager check_bad_cmd check
>  - rebase-i: explicitly accept tab as separator in commands
> 
>  "git rebase -i" had a minor regression recently, which stopped
>  considering a line that begins with an indented '#' in its insn
>  sheet not a comment, which is now fixed.
> 
>  Will merge to 'next'.

This is breaking using a non-gnu sed, which doesn't handle "\t"... The solution is to replace "\t" with a literal TAB.

diff --git a/t/t3404-rebase-interactive.sh b/t/t3404-rebase-interactive.sh
index 3da3ba3..3de0b1d 100755
--- a/t/t3404-rebase-interactive.sh
+++ b/t/t3404-rebase-interactive.sh
@@ -1232,7 +1232,7 @@ test_expect_success 'tabs and spaces are accepted in the todolist' '
 	write_script add-indent.sh <<-\EOF &&
 	(
 		# Turn single spaces into space/tab mix
-		sed "1s/ /\t/g; 2s/ /  /g; 3s/ / \t/g" "$1"
+		sed "1s/ /	/g; 2s/ /  /g; 3s/ / 	/g" "$1"
 		printf "\n\t# comment\n #more\n\t # comment\n"
 	) >$1.new
 	mv "$1.new" "$1"
Previous: Mike RappazzoNext: Junio C Hamano
Message 5 of 7 in “What's cooking in git.git (Oct 2015, #01; Mon, 5)”
  1. Junio C HamanoOct 5, 2015
  2. Torsten BögershausenOct 6, 2015
  3. Junio C HamanoOct 6, 2015
  4. Mike RappazzoOct 6, 2015
  5. Torsten BögershausenOct 6, 2015
  6. Junio C HamanoOct 6, 2015
  7. Matthieu MoyOct 6, 2015

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.