Re: [PATCH 2/3] rebase: Add tests for console output
- From
- Phillip Wood <phillip.wood@talktalk.net>
- Date
- Jun 14, 2017, 10:18 UTC
- Message-ID
- <699c6d43-4c1b-d3c2-3b01-8925f5c21511@talktalk.net>
- In-Reply-To
- <xmqqr2yt5d4g.fsf@gitster.mtv.corp.google.com>
On 09/06/17 17:39, Junio C Hamano wrote:
Show 9 quoted lines
> Phillip Wood <phillip.wood@talktalk.net> writes: > >> I'm confused by this as my script does not use the escape sequence "\r" >> out of portability concerns. It has a literal carriage return as you get >> from typing Ctrl-Q Ctrl-M in emacs or Ctrl-V Ctrl-M in bash ... > > I think the worry is that some implementations of sed may be unhappy > to see these raw control characters in the sed script. >
Ah, I'd misunderstood the problem, thanks for taking the time to point that out. I'll redo the tests as Johannes suggested
Thanks
Phillip