From: Junio C Hamano Date: Mon, 05 Dec 2022 09:32:12 GMT Subject: Re: [PATCH 2/1] t3920: support CR-eating grep Message-ID: In-Reply-To: <569714f6-a913-5f7f-855a-b303adbde3ee@web.de> René Scharfe writes: > Depends on the meaning of "fixed". If it stops removing CRs then this > line is unaffected -- .crlf-orig-$branch.txt contains no CRs. OK, that "fix" was what I was worried about and if there is no problem with the input we use, that is good ;-) > If it > starts adding CRs then we'd have a problem with all grep invocations, > which was addressed by 4d715ac05c (Windows: a test_cmp that is agnostic > to random LF <> CRLF conversions, 2013-10-26). Yeah, but I thought that the unspoken motivation behind recent changes are so that we do not have to rely on "the differences between CRLF and LF do not matter" version of test_cmp? Thanks.