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

Re: [PATCH] Don't add To: recipients to the Cc: header

From: Sergei Organov <hidden>
Date: 2016-06-15 22:43:53

Junio C Hamano [off-list ref] writes:
Sergei Organov [off-list ref] writes:
quoted
Junio C Hamano [off-list ref] writes:
[...]
quoted
Oops, forgot to say "no need to resend".  I asked only because I
wanted an independent datapoint for Emacs diff mode breakage.
I bet I can damage any patch using any editor ;)

More interesting is what version of Emacs it was?
To be fair and honest, I do not think there is a simple fix for
this, although it probably is possible to fix it.

What is causing the "breakage" is the fact that format-patch
output ends with the signature delimiter line "^-- $" that
immediately follows the patch text.
Exactly. What causes breakage is the fact that the '-' character (as
well as '+', ' ', '!', '#', and '\'), being the first symbol of a line
has special meaning in the diff format.

Therefore it seems that format-patch should better put one empty line
after the last diff hunk and before the signature. Any objections?

-- 
Sergei.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help