public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jaco Kroon <jaco@kroon.co.za>
To: Lee Revell <rlrevell@joe-job.com>
Cc: jason@stdbev.com, rostedt@goodmis.org,
	linux-kernel@vger.kernel.org, pavel@ucw.cz,
	s0348365@sms.ed.ac.uk
Subject: Re: recommended mail clients
Date: Mon, 26 Dec 2005 20:28:40 +0200	[thread overview]
Message-ID: <43B03658.9040108@kroon.co.za> (raw)
In-Reply-To: <1135621183.8293.64.camel@mindpipe>

[-- Attachment #1: Type: text/plain, Size: 3511 bytes --]

Lee Revell wrote:
> On Mon, 2005-12-26 at 12:09 -0600, Jason Munro wrote:
> 
>>On 11:54:00 am 26 Dec 2005 Lee Revell <rlrevell@joe-job.com> wrote:
>>
>><snip>
>>
>>>> Dare I say it, KMail has also been doing the Right Thing for a
>>>> long time. It will only line wrap things that you insert by
>>>> typing; pastes are left untouched.
>>>
>>>It seems that of all the popular mail clients only Thunderbird has
>>>this problem.  AFAICT it's impossible to make it DTRT with inline
>>>patches and even if it is the fact that most users get it wrong
>>>points to a serious usability/UI issue.
>>>
>>>Would a patch to add "Don't use Thunderbird/Mozilla Mail" to
>>>SubmittingPatches be accepted?  Then we can point the Mozilla
>>>developers at it (they have shown zero interest in fixing the problem
>>>so far) and hopefully this will light a fire under someone.

I would second that patch.

>>Maybe this is a stupid question but in terms of inline patches what exactly
>>would be ideal behavior from a mail client for LKML patch submitters? What
>>line lengths are expected to be maintained, preferred encodings, tabs vs.
>>spaces, etc? I have noticed that some patch submitters append an EOF after
>>the patch, while others do not. Would the ability to pull the patch from
>>the message body (assuming there was an agreed upon patch termination
>>string) as a separate file/download be useful? Though my client is web
>>based it is quite speedy and can handle large folders as well as many
>>desktop clients IMHO. I would gladly implement specific features to make
>>patch submission for LKML compliant.
> 
> 
> The specifics do not matter.  It does not even have to do what we want
> by default when you paste or insert text.  There just has to be SOME way
> (well, some reasonable way - a global config option is not reasonable)
> to insert a text file and paste from the clipboard as-is, no tab->space
> conversion, no line wrapping, nothing.

And mozilla only does the line-wrapping (with no way that I can find to
switch it off).  It doesn't do tab->space conversion, that usually (in
my experience) results from c&p'ing from an [axe]term which outputs
spaces instead of tabs to begin with (well, it does represent a
character matrix so I don't really see another way).

Ideally (imho) one would like the 'changelog' part to be line-wrapped
(to keep it from running into oblivion) but the patch part to be left
"as is".  There is also the [PATCH] subject prefix and signed-of-by
requirements.  The only other recommendation (that I recall) is that the
changelog and the patch be seperated by '---' - but since this is part
of the initial output of the diff command this is done implicitly.

I've looked at a few clients and it seems I'm stuck with mozilla for at
least a while.  Whilst probably the buggiest client there is it does
look like it's the best suited for what I want.  I might switch to
FireFox (which iirc does have an "insert file" feature - which might
also solve this problem).

For the moment though I'm quickly hacking together a bash script that
wraps the sendmail binary that can be used specifically for submitting
patches (the intent is to perform certain checks for Signed-of-by lines,
correct [PATCH] subject and so forth).  If anybody else is interrested
I'd be more than happy to share (albeit I suspect the usefullness will
be seriously limited).

Jaco
-- 
There are only 10 kinds of people in this world,
  those that understand binary and those that don't.
http://www.kroon.co.za/

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/x-pkcs7-signature, Size: 3166 bytes --]

  reply	other threads:[~2005-12-26 18:30 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-12-26  4:52 [PATCH] ati-agp suspend/resume support (try 2) Jaco Kroon
2005-12-26  8:29 ` Pavel Machek
2005-12-26  8:55   ` recommended mail clients [was] " Jaco Kroon
2005-12-26 14:38     ` Steven Rostedt
2005-12-26 15:35       ` Alistair John Strachan
2005-12-26 17:54         ` Lee Revell
2005-12-26 18:09           ` Jason Munro
2005-12-26 18:19             ` Lee Revell
2005-12-26 18:28               ` Jaco Kroon [this message]
2005-12-26 18:43                 ` recommended mail clients Lee Revell
2005-12-26 18:47                   ` Jaco Kroon
2005-12-26 18:48                 ` Lee Revell
2005-12-26 18:47                   ` Christoph Hellwig
2005-12-26 18:54                     ` Jaco Kroon
2005-12-29  9:49                   ` Coywolf Qi Hunt
2005-12-26 18:58                 ` Randy.Dunlap
2005-12-26 19:04                   ` Jaco Kroon
2005-12-26 19:12                   ` Jaco Kroon
2005-12-27 21:55                   ` Ryan Anderson
2005-12-26 19:18                 ` Jeff Garzik
2005-12-26 19:32                   ` Lee Revell
2005-12-26 20:32                     ` Pavel Machek
2005-12-26 18:24             ` recommended mail clients [was] [PATCH] ati-agp suspend/resume support (try 2) Lee Revell
2005-12-26 22:33             ` Jan Engelhardt
2005-12-26 20:03           ` Alistair John Strachan
2005-12-26 21:00             ` Lee Revell
2005-12-28  0:05               ` Michael Clark
2005-12-28  0:33                 ` Lee Revell
2005-12-28  1:09                 ` Peter Williams
2005-12-28  2:01                   ` Michael Clark
2005-12-28  2:10                     ` Steven Rostedt
2005-12-28  2:14                       ` Randy.Dunlap
2005-12-28 12:44                         ` Paolo Ornati
2005-12-28  2:12                     ` Randy.Dunlap
2005-12-28  3:22                       ` Peter Williams
2005-12-28  3:28                         ` Michael Clark
2005-12-28  3:33                           ` Randy.Dunlap
2005-12-28  3:39                             ` Randy.Dunlap
2005-12-28  7:22                               ` Stefan Smietanowski
2005-12-26 22:26         ` Jan Engelhardt
2005-12-27 15:20           ` Jason Munro
2005-12-27 15:45             ` Pavel Machek
2005-12-26 15:05     ` Lee Revell
2005-12-26 18:50       ` Randy.Dunlap
2005-12-27 15:48         ` Bob Copeland
     [not found]   ` <mailman.1135587661.17617.linux-kernel2news@redhat.com>
2005-12-31  3:28     ` Pete Zaitcev
2006-01-02 21:27       ` Tomasz Torcz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=43B03658.9040108@kroon.co.za \
    --to=jaco@kroon.co.za \
    --cc=jason@stdbev.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@ucw.cz \
    --cc=rlrevell@joe-job.com \
    --cc=rostedt@goodmis.org \
    --cc=s0348365@sms.ed.ac.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox