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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.