[mythtv] Re: [patch] patch policy question

[Michael T. Dean]

On 02/04/2004 06:08 AM, J. Donavan Stanley wrote:

> Andrew M. Bishop wrote:
>
>> I am not sure that I know why attachments are better than inlining the
>> patch though.  Unless someone somewhere is mangling the e-mail it
>> should be perfectly readable and I am not the only person sending them
>> this way.  It has always worked for me before, Isaac has had no
>> problem receiving them, so why I want to attach them?
>
> Many helpful mail clients will append the patch to the bottom of the 
> message as though it had been typed in there to begin with, just like 
> mine did with yours.
>  Which would you rather do 1) Cut and past out sections of a message 
> to create a patch file to apply or 2) Save an attached patch

Not to mention the fact that a properly configured e-mail client should 
be putting line breaks in the messages somewhere around 72 or 78 
characters per line before sending to this list (because you should be 
sending them as plain text).  When you have a line of code that's 
greater than this length, or the header lines inserted by diff -u 
labelling each file to change, extra line breaks get put in, and the 
developers have to manually remove them to make a valid patch file.

Mike