Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH] Fixed Cygwin CR-munging problem in mailsplit

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:42:27

Junio C Hamano [off-list ref] writes:
I personally think this is a right change.  Provided if MTAs on
the path between patch originator and you are not broken and
your MUA saved the message with CR/LF distinction in the
contents intact, this should do more right thing.

I see broken patches every once in a while, but when they are
mangled by the mailpath, CRLF is the least of the problem; they
have other whitespace breakage that makes them unapplicable
anyway.

Having said that, however, that historically used to be a big IF
with capital letters.


I have a feeling that Linus did this on purpose.  For the
Heh, I had a trailing CR after the "with capital letters." and
one blank line between paragraphs, but I now see two blank lines
there.  So even in this modern day, preserving CRLF is not
something that happens by default -- you would need to make sure
that everybody on your mailpath to the recipient is set up the
right way.
So I am slightly in favor of the change, but just barely.
So now I am less in favor of the change than when I wrote that
response.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help