On 12 February 2010 22:56, Pavel Roskin <proski@xxxxxxx> wrote:
> On Fri, 2010-02-12 at 16:11 +0000, Catalin Marinas wrote:
>> The original implementation was asking for the SMTP password on every
>> patch sent. This patch only asks the password once before sending or
>> even editing the cover message and patches.
>
> I don't have time to investigate at the moment, but "make test" fails in
> t1900-mail.sh on the "proposed" branch, which includes this patch:
The patch was converting the message headers to Header objects rather
than strings so that the long-line folding is done using "\n " rather
than "\n\t" (Outlook cannot handle the latter correctly). But it looks
like the Python library assumes that they are strings and getting
"content-type" fails (I would say Python bug).
Anyway, I change the patch so that it only converts the the Subject
header which is the one that bothers me most:
commit 1b0c0113861681974b8905dbe10a57f6831ecb87
Author: Catalin Marinas <catalin.marinas@xxxxxxxxx>
Date: Fri Feb 12 15:36:37 2010 +0000
mail: Use space rather than tab for long subject header folding
The default Python implementation (at least 2.5 and earlier) fold long
e-mail header lines by inserting "\n\t". This causes issues with some
e-mail clients that remove both "\n\t". The RFC2822 shows that folding
should be done with "\n ". The Python workaround is to use a Header
object instead of a string when setting the message headers.
Signed-off-by: Catalin Marinas <catalin.marinas@xxxxxxxxx>
diff --git a/stgit/commands/mail.py b/stgit/commands/mail.py
index d0334b4..ed55fd9 100644
--- a/stgit/commands/mail.py
+++ b/stgit/commands/mail.py
@@ -426,6 +426,13 @@ def __encode_message(msg):
new_val = ' '.join(words)
msg.replace_header(header, new_val)
+ # replace the Subject string with a Header() object otherwise the long
+ # line folding is done using "\n\t" rather than "\n ", causing issues with
+ # some e-mail clients
+ subject = msg.get('subject', '')
+ msg.replace_header('subject',
+ email.Header.Header(subject, header_name = 'subject'))
+
# encode the body and set the MIME and encoding headers
if msg.is_multipart():
for p in msg.get_payload():
--
Catalin
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html