Difference between revisions of "Create non-email method of contributing kernel patches"

From eLinux.org
Jump to: navigation, search
(Outlook inline comments: add autocorrect fix for Outlook)
(add info about automatic replies and forward popout option)
Line 127: Line 127:
 
  * in the "format" section, select "Aa Plain Text"
 
  * in the "format" section, select "Aa Plain Text"
  
 +
If it starts your resonse composition in the main outlook window, there is an option to "Pop Out" the message into a standalone window.
 +
If you want to always start responses in a standalone window, set the option:
 +
* File
 +
* Options
 +
* Mail
 +
* Replies and Forwards sectino
 +
* check Open replies and forwards in a new window
 
----  
 
----  
 
There's an option to prevent Outlook from removing extraneous line feeds.  See
 
There's an option to prevent Outlook from removing extraneous line feeds.  See

Revision as of 13:08, 23 May 2016

Summary
Create non-email method of contributing kernel patches
Proposer
Tim Bird

Description

A consistent problem for "industry" developers is problems using their e-mail client for submitting patches to mainline. In particular, many industry developers work for companies whose IT policies dictate that they use Outlook as their email client and Exchange as their email server. These two programs are problematical to use for submitting patches to the kernel. They sometimes mess up the text formatting of the patches. Often, Exchange will be configured to automatically add confidentiality notices to outgoing email. These notices conflict with the intent of publishing the patch, and emails containing such notices will not be applied by upstream maintainers.

The purpose of this project is to create a method of submitting patches to the mainline Linux kernel community that is independent of any particular e-mail client or configuration, and that thus does not suffer from the problems mentioned.

One alternative to using one's corporate email account for submitting patches is to set up a separate email account (quite often a gmail account), and submit patches from that.

The primary motivations for not just using gmail accounts are:

  • it may be desirable to have the author of the patch attributed to the corporate email address
  • it may be difficult to reach gmail (or some other external email client) from inside the corporate firewall

Ideally, this feature would be integrated into the git command suite, which is the preferred tool for working with Linux kernel source. This may be as simple as making a "git-<command>" version of this, and putting it in the right place where git can find it. (It appears the standard git exec-path is /usr/lib/git-core). It might be possible to just add this functionality to git send-email (maybe add a flag to indicate sending the "e-mail" via a web proxy or something).

what about follow-on emails?

Note that this project is limited in scope to just the submission of patches to the community mailing lists and addresses. It does not address the further problem of responding to feedback on the patches, from those corporate-mandated email programs. That is, Outlook doesn't handle inline commenting properly, and either Outlook or Exchange may mangle the text for emails related to patch reviews. For the purpose of this project, this is considered a secondary consideration, as it is possible to use gmail or some other email account external to the company to respond to feedback.

discussion from kernel summit list

Darren Hart and others discussed this topic on the kernel summit discussion list prior to the 2015 kernel summit. Some attributes of this tooling would ease the job of maintainers, and some would ease the task for commercial developers.

Here are some features that were mentioned that would be nice to have:

  • feature available as a git command


[under construction]

Areas of research

Is it possible to specify the author of the patch when e-mailing from a different account than the author's e-mail account? That is, what does git use as the author for the commit meta-data?

List of things that go wrong

Here is a list of common problems encountered when someone sends an e-mail from a non-community-compliant e-mail client: (below each one is how to fix it)

  • patch is sent as an attachment
  • patch is sent as Content-Type: text/html
    • needs to be converted to text/plain
  • patch is sent with Content-Transfer-Encoding: base64
    • convert to plain text with uudecode or base64?

Here are problems I have:

  • mis-typing a person's e-mail, and having the mail go out with a bad CC: This is extremely embarrassing, because everyone who responds to the e-mail gets a bounce
  • sending to the wrong people. Twice I left off the actual maintainer of a driver I was sending a fix for, because I misread the output of get_maintainers.pl. This was also very embarrassing.

proposed features

  • check that destination addresses are valid (list names spelled correctly, individuals are in MAINTAINERS file or in a git commit, or in the senders address book)
    • check the list against the addresses listed by get_maintainers.pl
  • run scripts/checkpatch.pl against the patch
  • use the mail threading options appropriate for the primary target of the patch
    • each list or maintainer has their own preferences for some submission methods - read those from MAINTAINERS and implement as desired
  • allow user to specify the author e-mail address of the patch
    • Needs some validation that they own the address, lots of abuse potential here...
  • Check that the patch is against a sensible kernel version.

Related work

Resources

Outlook inline comments

Here is some information on how to configure outlook to use "> " to preface replied-to text in inline responses.

These instructions worked for me on Outlook in Office Professinoal Plus 2013

* click on File on the menu bar
* select Options on pane that appears
* select Mail on left pane
* under Compose Messages, select change the setting for "Compose messages in this format" to "Plain text"
* Under Replies and Forwards
  * Set "When replying to a message" to "Prefix each line of the original message"
  * Set "When forwarding a message" to "Prefix each line of the original message:
  * Set "Preface each line in a plain-text message with:" to "> "  (greater-than sign followed by a space)

The above instructions work when the message is received in plain text. When a message is received in HTML format, it appears to be impossible to do inline or post-message comments.


From Andrew Ducker at http://superuser.com/a/587558/597172

You can cause Outlook to format all inbound mail at plan text as follows:

* File
* Options
* Trust Center
* Trust Center Settings
* Email Security
* Read As Plain Text.

This means they all appear as plain text, so replies go out as plain text, which means that you get proper indenting (using ">") if you've set that up.

You can still display the HTML original for messages where you want to, by clicking on the "This message was converted to plain text" message on the header and switching to a different format.


Tim - Note that once you select the option to "View as HTML", you cannot switch back to plain text again. (At least, I couldn't find an option to do it. The info box that appears no longer has any options for changing the view formatting.)


In Outlook, if you reply, you may be put in a composer pain in the main outlook window, or it may create a new window to compose your response. In the standalone window, you can control the output format by selecting:

* Format Text
* in the "format" section, select "Aa Plain Text"

If it starts your resonse composition in the main outlook window, there is an option to "Pop Out" the message into a standalone window. If you want to always start responses in a standalone window, set the option:

* File
* Options
* Mail
* Replies and Forwards sectino
* check Open replies and forwards in a new window

There's an option to prevent Outlook from removing extraneous line feeds. See

* File
* Options
* Mail
* Message Format
* Remove extra line breaks in plain text messages

Note that in that same section of options, there is control over the column to wrap plain text at:

* Automatically wrap text at character [   ]

There's an way to prevent the first letter of a new line from being automatically capitalized. Change the autocorrect setting:

* File
* Options
* Mail in left-side menu
* Compose Messages (section)
* Spelling and Autocorrect button
* Proofing in left-side menu
* AutoCorrect options section
* AutoCorrect Options button
* AutoCorrect tab
* uncheck Capitalize first letter of sentences

Scope

Describe the expected amount of work (in person-weeks, please)

Contractor Candidates

List possible contractors for this work (can be yourself)

Comments

Some random ideas:

  • is it possible (or advisable) to use the list of e-mail addresses already in the kernel git tree to validate new incoming e-mails
    • idea is to encourage more newcomers. Maybe if the from address is already valid, then eliminate some validation tests?
  • need to prevent spamming the lists. Do some kind of challenge response, where an e-mail is sent to the from address, and wait

for a response to send the email forward to the lists?

    • this is the kind of thing that should be automate-able if the person already has a patch in the kernel tree. Otherwise, use some other method of spam avoidance.
    • If we just accept things for already existing e-mails then people will just abuse that...

stuff about outlook and exchange

From this LWN.net comment

Thunderbird+Lightning+EWS Provider appears to be a decent Outlook alternative if you're stuck with Exchange on the backend. It's far more stable and performant than Evolution