Mailing-patches

From LinuxMIPS
Revision as of 16:55, 24 May 2006 by Ralf (Talk | contribs)

(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to: navigation, search

Documentation/SubmittingPatches in the kernel sources asks patch submitters to

6) No MIME, no links, no compression, no attachments.  Just plain text.

Linus and other kernel developers need to be able to read and comment
on the changes you are submitting.  It is important for a kernel
developer to be able to "quote" your changes, using standard e-mail
tools, so that they may comment on specific portions of your code.

But due to brokenness of the most commonly used mail clients became necessary which generally scrable patches by:

  • chopping of trailing whitespace
  • converting tab characters to whitespace
  • break long lines into multiple lines.

that has led to the problem that a very significant fraction of patches had to be rejected simply because they didn't survive email transport. This page is trying to help you to fullfill the apparent contradiction. Keep in mind that the author of this page is a mutt user and would only touch a GUI mail client with a blowtorch.

Evolution

  • Set the formatting to "Preformat"
  • Hit Insert/Text File (Alt-I X)
    Then at the ghastly gnome dialog hit ^L (secret handshake for the 'developers cant stand it either' shortcut box) and type in the file name.

Thunderbird receipe #1

Yep, the odd part is not to disable html email. Then when composing a message, there is a drop-down box for a format selection. While the cursor is in the body area, change the format from default "Body text" to reformat, and then copy-n-paste the patch.

  • Before starting to write the email, do Edit->Preferences, Composition tab, change "Wrap plain text messages at __ characters" to 0
  • Begin writing your email
  • Start up an editor in which you can select the text, for example:
  • # gvim patch.diff
  • Edit->Select All
  • Edit->Copy

Switch back to the open Compose window, then Edit->Paste

Thunderbird receipe #2

If you have Enigmail configured, when sending the email, you might get a box asking if you want to change the line wrapping to 68 characters, so just say No if that happens.

Others have suggested using an external editor which will show up in the compose editor and allow to include the patch file inline.

See also