Re: [PATCH v4 08/16] reorganize indexing of multipart/signed and multipart/encrypted
[notmuch-archives.git] / 88 / e7301a9d37f88b45b587d459c04c228dd1947b
1 Return-Path: <dmitry.kurochkin@gmail.com>\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 31D9F429E25\r
6         for <notmuch@notmuchmail.org>; Thu, 12 Jan 2012 06:18:40 -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: 1.7\r
10 X-Spam-Level: *\r
11 X-Spam-Status: No, score=1.7 tagged_above=-999 required=5\r
12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
13         FREEMAIL_FROM=0.001, FREEMAIL_REPLY=2.499, RCVD_IN_DNSWL_LOW=-0.7]\r
14         autolearn=disabled\r
15 Received: from olra.theworths.org ([127.0.0.1])\r
16         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
17         with ESMTP id yPXqemVr53vO for <notmuch@notmuchmail.org>;\r
18         Thu, 12 Jan 2012 06:18:39 -0800 (PST)\r
19 Received: from mail-ey0-f181.google.com (mail-ey0-f181.google.com\r
20         [209.85.215.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
21         (No client certificate requested)\r
22         by olra.theworths.org (Postfix) with ESMTPS id 5F78E431FB6\r
23         for <notmuch@notmuchmail.org>; Thu, 12 Jan 2012 06:18:39 -0800 (PST)\r
24 Received: by eaah10 with SMTP id h10so897202eaa.26\r
25         for <notmuch@notmuchmail.org>; Thu, 12 Jan 2012 06:18:36 -0800 (PST)\r
26 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
27         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
28         :mime-version:content-type;\r
29         bh=YRQK9yAiu4M9ghPjdStmf04+nTu7CZ3LML8qQk5Pj60=;\r
30         b=qzAXDwkP/1idbQHt4y2EoQ6MmB1r7yx8BU578J8jRx7gwLqt8tRQCxbRF/k+tbcos1\r
31         IxbgDyCrulNkQlfylocTbi8hFtlotrlSaSVKFbxmb+rnbf5XfDdITuN8l9DPRbTcfkEY\r
32         aqSM8Y5qJpUBULATuwIsjNZYxdhs9XXxMc1NY=\r
33 Received: by 10.205.127.14 with SMTP id gy14mr1138058bkc.103.1326377916481;\r
34         Thu, 12 Jan 2012 06:18:36 -0800 (PST)\r
35 Received: from localhost ([91.144.186.21])\r
36         by mx.google.com with ESMTPS id t23sm10993060bkv.10.2012.01.12.06.18.35\r
37         (version=TLSv1/SSLv3 cipher=OTHER);\r
38         Thu, 12 Jan 2012 06:18:35 -0800 (PST)\r
39 From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
40 To: David Edmondson <dme@dme.org>, notmuch@notmuchmail.org\r
41 Subject: Re: [PATCH] notmuch/emacs: Observe the charset of encoded parts,\r
42         where known.\r
43 In-Reply-To: <cun8vldm8q9.fsf@hotblack-desiato.hh.sledj.net>\r
44 References: <1326279001-28427-1-git-send-email-dme@dme.org>\r
45         <87k44ym6ka.fsf@gmail.com>\r
46         <cun8vldm8q9.fsf@hotblack-desiato.hh.sledj.net>\r
47 User-Agent: Notmuch/0.10.2+135~gb811a3c (http://notmuchmail.org) Emacs/23.3.1\r
48         (x86_64-pc-linux-gnu)\r
49 Date: Thu, 12 Jan 2012 18:17:44 +0400\r
50 Message-ID: <878vldgg3b.fsf@gmail.com>\r
51 MIME-Version: 1.0\r
52 Content-Type: text/plain; charset=us-ascii\r
53 X-BeenThere: notmuch@notmuchmail.org\r
54 X-Mailman-Version: 2.1.13\r
55 Precedence: list\r
56 List-Id: "Use and development of the notmuch mail system."\r
57         <notmuch.notmuchmail.org>\r
58 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
59         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
60 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
61 List-Post: <mailto:notmuch@notmuchmail.org>\r
62 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
63 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
64         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
65 X-List-Received-Date: Thu, 12 Jan 2012 14:18:40 -0000\r
66 \r
67 Hi David.\r
68 \r
69 On Thu, 12 Jan 2012 12:00:14 +0000, David Edmondson <dme@dme.org> wrote:\r
70 > On Wed, 11 Jan 2012 22:34:45 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
71 > > -1\r
72\r
73 > Which puts me back to 0 :-(\r
74\r
75 > > One particular issue with your patch is that it adds (incorrect) charset\r
76 > > parameter to plain/text parts which are converted to UTF-8 in JSON\r
77 > > output.\r
78\r
79 > Patches that break things should obviously not be accepted...\r
80\r
81 > > I already tried to solve the above problem using a more general approach\r
82 > > (output all content-type parameters, not just charset) [1].  There was a\r
83 > > lengthy discussion on IRC about it and it was rejected.  The consensus\r
84 > > was that we need to make some more substantial changes to JSON and raw\r
85 > > output formats to properly fix the issue (and the issue is more general\r
86 > > than what this patch fixes, other content-type parameters are useful for\r
87 > > renderers as well and should be included in the output).\r
88\r
89 > ...but a useful point fix should not be blocked by theoretical future\r
90 > work.\r
91\r
92 \r
93 I think there is a record of useful features and fixes that were not\r
94 accepted to notmuch because of some implementation issues.  And\r
95 interested people were using them in private repos for years.  (I do not\r
96 say that it is always the right thing to do, or that it is the right\r
97 thing in this particular case.)\r
98 \r
99 > > I am planning to work on a proper fix for this issue, but decided to\r
100 > > postpone it until Austin's rewrite of notmuch show is complete.\r
101\r
102 > If the UTF8 text/plain part issue can be resolved, would you be happier\r
103 > to accept this as an interim fix whilst we wait for the more complete\r
104 > solution?\r
105\r
106 \r
107 I would like to see the following changes:\r
108 \r
109 * Properly handle charset with parameters in Emacs UI.  Currently it is\r
110   broken by your patch in one place at least:\r
111   `notmuch-show-handlers-for' would produce incorrect results for\r
112   content-type string with parameters.  In my patch [1] I did parse the\r
113   charset at top level and then changed all usages of it accordingly.\r
114   Making `notmuch-show-handlers-for' smarter about parameters may be\r
115   sufficient, but I would like to see some more details on why adding\r
116   parameters to content-type string does not break Emacs UI code in\r
117   other places.\r
118 \r
119 * Add charset parameter for text/html parts only.\r
120 \r
121 * Use `mail-header-parse-content-type' to parse content-type instead of\r
122   contructing the list for `mm-make-handle' manually.\r
123 \r
124 * Add a proper XXX comment to notmuch-show code.\r
125 \r
126 I cannot say I would be happy about this patch after these changes.  It\r
127 would be a temporary hack anyway.  But I would withdraw my -1 and let\r
128 others decide.  I would like to see Jameson and Austin input on this\r
129 one.\r
130 \r
131 Regards,\r
132   Dmitry\r
133 \r
134 [1] id:"1321659905-24367-1-git-send-email-dmitry.kurochkin@gmail.com"\r
135 \r
136 > Nothing in the patch (so far) should make your proposed changes any\r
137 > harder, so I'm not sure what the problem would be.\r