Re: [PATCH v4 13/16] add indexopts to notmuch python bindings.
[notmuch-archives.git] / 86 / 405146fd137da8b038f692339acf12b9f2534d
1 Return-Path: <awg@lagos.xvx.ca>\r
2 X-Original-To: notmuch@notmuchmail.org\r
3 Delivered-To: notmuch@notmuchmail.org\r
4 Received: from localhost (localhost [127.0.0.1])\r
5         by olra.theworths.org (Postfix) with ESMTP id 9BFEB429E29\r
6         for <notmuch@notmuchmail.org>; Wed, 30 Nov 2011 08:50:22 -0800 (PST)\r
7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
8 X-Spam-Flag: NO\r
9 X-Spam-Score: 0\r
10 X-Spam-Level: \r
11 X-Spam-Status: No, score=0 tagged_above=-999 required=5\r
12         tests=[RCVD_IN_DNSWL_NONE=-0.0001] autolearn=disabled\r
13 Received: from olra.theworths.org ([127.0.0.1])\r
14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
15         with ESMTP id eRUJ0EoVb72q for <notmuch@notmuchmail.org>;\r
16         Wed, 30 Nov 2011 08:50:20 -0800 (PST)\r
17 Received: from idcmail-mo2no.shaw.ca (idcmail-mo2no.shaw.ca [64.59.134.9])\r
18         by olra.theworths.org (Postfix) with ESMTP id CBA11429E39\r
19         for <notmuch@notmuchmail.org>; Wed, 30 Nov 2011 08:50:16 -0800 (PST)\r
20 Received: from lb7f8hsrpno-svcs.dcs.int.inet (HELO pd5ml2no-ssvc.prod.shaw.ca)\r
21         ([10.0.144.222])\r
22         by pd7mo1no-svcs.prod.shaw.ca with ESMTP; 30 Nov 2011 09:40:42 -0700\r
23 X-Cloudmark-SP-Filtered: true\r
24 X-Cloudmark-SP-Result: v=1.1 cv=1i/nnIW0nMC5uWcUJ59KMVUkrnWI9M+zreV2y9Ww/qI=\r
25         c=1 sm=1\r
26         a=7GgN6RdxzlQA:10 a=BLceEmwcHowA:10 a=yQp6g8lIsgqumF79BAsFDg==:17\r
27         a=2LoIg5_kSO0TY0mPySQA:9 a=DK_ML55IPaTeQFSTg7YA:7\r
28         a=HpAAvcLHHh0Zw7uRqdWCyQ==:117\r
29 Received: from unknown (HELO lagos.xvx.ca) ([96.52.216.56])\r
30         by pd5ml2no-dmz.prod.shaw.ca with ESMTP; 30 Nov 2011 09:40:42 -0700\r
31 Received: by lagos.xvx.ca (Postfix, from userid 1000)\r
32         id AE8DA800027D; Wed, 30 Nov 2011 09:40:41 -0700 (MST)\r
33 From: Adam Wolfe Gordon <awg+notmuch@xvx.ca>\r
34 To: notmuch@notmuchmail.org,\r
35         awg@xvx.ca\r
36 Subject: [RFC PATCH 0/3] Quoting HTML-only emails in replies\r
37 Date: Wed, 30 Nov 2011 09:40:38 -0700\r
38 Message-Id: <1322671241-23438-1-git-send-email-awg+notmuch@xvx.ca>\r
39 X-Mailer: git-send-email 1.7.5.4\r
40 X-BeenThere: notmuch@notmuchmail.org\r
41 X-Mailman-Version: 2.1.13\r
42 Precedence: list\r
43 List-Id: "Use and development of the notmuch mail system."\r
44         <notmuch.notmuchmail.org>\r
45 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
46         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
47 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
48 List-Post: <mailto:notmuch@notmuchmail.org>\r
49 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
50 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
51         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
52 X-List-Received-Date: Wed, 30 Nov 2011 16:50:22 -0000\r
53 \r
54 Hi everyone,\r
55 \r
56 I use notmuch at work, where the email server is Exchange and the majority\r
57 of my colleagues use Outlook. This means that I get a fair bit of\r
58 HTML-only email. When using notmuch reply to create replies to such\r
59 emails, the quoted body ends up containing only the line "Non-text part:\r
60 text/html", and if I want to quote them I have to do it manually.\r
61 \r
62 This series of patches fixes this issue. It adds a --format=html option to\r
63 notmuch reply, which acts like --format=default except that it includes\r
64 HTML parts as well as text parts. The emacs interface render the quoted\r
65 HTML into text using w3m. I've added a customize option,\r
66 notmuch-mua-quote-html-parts to control this behavior. I've included a\r
67 test for the notmuch reply part, though it's not a great one.\r
68 \r
69 This code is working well for me, but I'm marking this as RFC because\r
70 there are a few things that could be better:\r
71 \r
72 1. If an email has a text part and an HTML part they'll both end up\r
73    quoted, and emacs might make the text part look ugly by trying to\r
74    render it as HTML.  Ideally it should omit HTML parts from the reply\r
75    when a text part exists, but it looks like this would take some\r
76    non-trivial refactoring of the notmuch reply code.\r
77 \r
78 2. The easiest way to implement --format=html in notmuch reply was to use\r
79    a global variable controlling HTML output, which feels clumsy to me.\r
80    Perhaps someone who is more familiar with the notmuch code can suggest\r
81    a cleaner way of doing it.\r
82 \r
83 3. As I mentioned, the test I've included isn't great.  A test that uses\r
84    an HTML-only email would be nice, and if (1) can be resolved, then the\r
85    existing test will have different results.\r
86 \r
87 I hope someone else will find these changes useful - please do let me know\r
88 if you have suggestions.\r
89 \r
90 Cheers.\r
91 \r
92 Adam Wolfe Gordon (3):\r
93   test: replies that quote HTML parts (expected to fail)\r
94   reply: New format 'html' that quotes text/html parts\r
95   emacs: Optionally quote HTML parts when replying.\r
96 \r
97  emacs/notmuch-mua.el |   23 +++++++++++++++++++++++\r
98  notmuch-reply.c      |    8 +++++++-\r
99  notmuch.c            |    6 +++++-\r
100  test/multipart       |   24 ++++++++++++++++++++++++\r
101  4 files changed, 59 insertions(+), 2 deletions(-)\r
102 \r
103 -- \r
104 1.7.5.4\r
105 \r