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 --]
next prev parent 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