2014-12-01 22:12:08

by Richard Leitner

[permalink] [raw]
Subject: [PATCH] Documentation/email-clients.txt: add info about Claws Mail

Claws Mail GUI client works when "Auto wrapping" is disabled.

Signed-off-by: Richard Leitner <[email protected]>
---
Documentation/email-clients.txt | 9 +++++++++
1 file changed, 9 insertions(+)

diff --git a/Documentation/email-clients.txt b/Documentation/email-clients.txt
index 9af538be..1f8929e 100644
--- a/Documentation/email-clients.txt
+++ b/Documentation/email-clients.txt
@@ -77,6 +77,15 @@ should appear, and then pressing CTRL-R let you specify the patch file
to insert into the message.

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+Claws Mail (GUI)
+
+Some people use this successfully for patches.
+
+Just turn off Auto wrapping in Configuration->Preferences->Compose->Wrapping
+
+To insert a patch use Message->Insert File (CTRL+i) or an external editor.
+
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Evolution (GUI)

Some people use this successfully for patches.
--
2.1.3


2014-12-01 22:34:09

by Jonathan Corbet

[permalink] [raw]
Subject: Re: [PATCH] Documentation/email-clients.txt: add info about Claws Mail

On Mon, 1 Dec 2014 23:02:08 +0100
Richard Leitner <[email protected]> wrote:

> Claws Mail GUI client works when "Auto wrapping" is disabled.

Interesting, I've sent an awful lot of patches with claws and I have
wrapping (a useful feature) enabled. I can't see it being a problem
unless you're writing the patch itself in the claws composition
window...and that would lead to other problems...?

jon

2014-12-02 07:10:26

by Richard Leitner

[permalink] [raw]
Subject: Re: [PATCH] Documentation/email-clients.txt: add info about Claws Mail

Hi jon,
thanks for your reply.

On Mon, 1 Dec 2014 17:34:05 -0500
Jonathan Corbet <[email protected]> wrote:

> Interesting, I've sent an awful lot of patches with claws and I have
> wrapping (a useful feature) enabled. I can't see it being a problem
> unless you're writing the patch itself in the claws composition
> window...and that would lead to other problems...?

You are right. I had the Auto wrapping disabled because it once messed
up a patch I was editing in the claws composition window. But in
general it works with Auto wrapping enabled. Sorry for that.

Should I change the patch to something like:

To insert a patch use Message->Insert File (CTRL+i) or an external
editor.
If the inserted patch has to be edited in the Claws composition window
Auto wrapping in Configuration->Preferences->Compose->Wrapping should
be disabled.

Or should I resend the patch without any hint to the auto wrapping?

best regards,
richard

2014-12-02 13:57:41

by Jonathan Corbet

[permalink] [raw]
Subject: Re: [PATCH] Documentation/email-clients.txt: add info about Claws Mail

On Tue, 2 Dec 2014 08:10:48 +0100
Richard Leitner <[email protected]> wrote:

> Should I change the patch to something like:
>
> To insert a patch use Message->Insert File (CTRL+i) or an external
> editor.
> If the inserted patch has to be edited in the Claws composition window
> Auto wrapping in Configuration->Preferences->Compose->Wrapping should
> be disabled.
>
> Or should I resend the patch without any hint to the auto wrapping?

The former looks good; it's worth a mention in case somebody does decide
to edit their patches on the way out. Send a new patch (soon for 3.19)
and I'll apply it.

Thanks,

jon