On 24.01.16 17:27, Torsten Bögershausen wrote: > On 24.01.16 11:48, Johannes Schindelin wrote: > (I had the same reasoning about the CRLF in the working tree: > We don't need to look at core.autocrlf/attributes, so Ack from me) > >> +test_expect_success 'conflict markers match existing line endings' ' >> + append_cr <nolf-orig.txt >crlf-orig.txt && >> + append_cr <nolf-diff1.txt >crlf-diff1.txt && >> + append_cr <nolf-diff2.txt >crlf-diff2.txt && >> + test_must_fail git -c core.eol=crlf merge-file -p \ >> + crlf-diff1.txt crlf-orig.txt crlf-diff2.txt >crlf.txt && >> + test $(tr "\015" Q <crlf.txt | grep "\\.txtQ$" | wc -l) = 3 && >> + test_must_fail git -c core.eol=crlf merge-file -p \ >> + nolf-diff1.txt nolf-orig.txt nolf-diff2.txt >nolf.txt && >> + test $(tr "\015" Q <nolf.txt | grep "\\.txtQ$" | wc -l) = 0 >> +' >> + > > Minor remark: > > Ramsay suggested a test that doesn't use grep or wc and looks like this: (That looked really garbled in my mail pgm. The suggestion is here:) <http://article.gmane.org/gmane.comp.version-control.git/283261> -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html