Re: [PATCH v3 08/25] sequencer: strip CR from the todo script

2016-10-12 Thread Johannes Schindelin
Hi Junio, On Tue, 11 Oct 2016, Junio C Hamano wrote: > Johannes Schindelin writes: > > > diff --git a/sequencer.c b/sequencer.c > > index 678fdf3..cee7e50 100644 > > --- a/sequencer.c > > +++ b/sequencer.c > > @@ -774,6 +774,9 @@ static int parse_insn_buffer(char *buf, struct > > todo_list *to

Re: [PATCH v3 08/25] sequencer: strip CR from the todo script

2016-10-11 Thread Junio C Hamano
Johannes Schindelin writes: > It is not unheard of that editors on Windows write CR/LF even if the > file originally had only LF. This is particularly awkward for exec lines > of a rebase -i todo sheet. Take for example the insn "exec echo": The > shell script parser splits at the LF and leaves t