Re: [PATCH v4 0/3] indicate length of omitted body content
[notmuch-archives.git] / fd / 7fe6010595b5ef483f587e03210889b27d3461
1 Return-Path: <m.walters@qmul.ac.uk>\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 E29E5431FAF\r
6         for <notmuch@notmuchmail.org>; Sun,  9 Dec 2012 11:21:20 -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.401\r
10 X-Spam-Level: *\r
11 X-Spam-Status: No, score=1.401 tagged_above=-999 required=5\r
12         tests=[DKIM_ADSP_CUSTOM_MED=0.001, FREEMAIL_FROM=0.001,\r
13         FREEMAIL_REPLY=2.499, NML_ADSP_CUSTOM_MED=1.2, RCVD_IN_DNSWL_MED=-2.3]\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 zeRtsn9dOzHz for <notmuch@notmuchmail.org>;\r
18         Sun,  9 Dec 2012 11:21:19 -0800 (PST)\r
19 Received: from mail2.qmul.ac.uk (mail2.qmul.ac.uk [138.37.6.6])\r
20         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
21         (No client certificate requested)\r
22         by olra.theworths.org (Postfix) with ESMTPS id 20568431FB6\r
23         for <notmuch@notmuchmail.org>; Sun,  9 Dec 2012 11:21:19 -0800 (PST)\r
24 Received: from smtp.qmul.ac.uk ([138.37.6.40])\r
25         by mail2.qmul.ac.uk with esmtp (Exim 4.71)\r
26         (envelope-from <m.walters@qmul.ac.uk>)\r
27         id 1ThmR9-0008Hi-EQ; Sun, 09 Dec 2012 19:21:15 +0000\r
28 Received: from 93-97-24-31.zone5.bethere.co.uk ([93.97.24.31] helo=localhost)\r
29         by smtp.qmul.ac.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.69)\r
30         (envelope-from <m.walters@qmul.ac.uk>)\r
31         id 1ThmR9-0007NF-4e; Sun, 09 Dec 2012 19:21:15 +0000\r
32 From: Mark Walters <markwalters1009@gmail.com>\r
33 To: notmuch@notmuchmail.org, Peter Wang <novalazy@gmail.com>\r
34 Subject: Re: [PATCH v4 0/3] indicate length of omitted body content\r
35 In-Reply-To: <1355057796-19260-1-git-send-email-markwalters1009@gmail.com>\r
36 References: <1355057796-19260-1-git-send-email-markwalters1009@gmail.com>\r
37 User-Agent: Notmuch/0.14+155~g7edfdc3 (http://notmuchmail.org) Emacs/23.4.1\r
38         (x86_64-pc-linux-gnu)\r
39 Date: Sun, 09 Dec 2012 19:21:18 +0000\r
40 Message-ID: <87vccb2frl.fsf@qmul.ac.uk>\r
41 MIME-Version: 1.0\r
42 Content-Type: text/plain; charset=us-ascii\r
43 X-Sender-Host-Address: 93.97.24.31\r
44 X-QM-SPAM-Info: Sender has good ham record.  :)\r
45 X-QM-Body-MD5: 6e2b44589f881d169c562ca1d29c7913 (of first 20000 bytes)\r
46 X-SpamAssassin-Score: -1.2\r
47 X-SpamAssassin-SpamBar: -\r
48 X-SpamAssassin-Report: The QM spam filters have analysed this message to\r
49         determine if it is\r
50         spam. We require at least 5.0 points to mark a message as spam.\r
51         This message scored -1.2 points.\r
52         Summary of the scoring: \r
53         * -2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/,\r
54         *      medium trust\r
55         *      [138.37.6.40 listed in list.dnswl.org]\r
56         * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\r
57         provider *      (markwalters1009[at]gmail.com)\r
58         *  1.0 FREEMAIL_REPLY From and body contain different freemails\r
59         *  0.1 AWL AWL: From: address is in the auto white-list\r
60 X-QM-Scan-Virus: ClamAV says the message is clean\r
61 X-BeenThere: notmuch@notmuchmail.org\r
62 X-Mailman-Version: 2.1.13\r
63 Precedence: list\r
64 List-Id: "Use and development of the notmuch mail system."\r
65         <notmuch.notmuchmail.org>\r
66 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
67         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
68 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
69 List-Post: <mailto:notmuch@notmuchmail.org>\r
70 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
71 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
72         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
73 X-List-Received-Date: Sun, 09 Dec 2012 19:21:21 -0000\r
74 \r
75 \r
76 On Sun, 09 Dec 2012, Mark Walters <markwalters1009@gmail.com> wrote:\r
77 > This is a (trivially) rebased version of\r
78 > id:1344428872-12374-1-git-send-email-novalazy@gmail.com (There was one\r
79 > bit of context clash, and a small change to one of the sexp tests to\r
80 > mirror the same change for the json test)\r
81 >\r
82 >\r
83 > The patch is Peter's not mine so as a reviewer +1 from me.\r
84 >\r
85 \r
86 Hi (sorry Peter I had assumed git send-email would have cc'd you on my\r
87 rebase earlier today)\r
88 \r
89 There was some discussion about this patch series on irc and the general\r
90 view was favourable but that, for consistency, all omitted parts should\r
91 received a content-length (including text/html). This is a little messy\r
92 to code into the if else stuff: Jani suggested putting\r
93 content-length/content-encoding into a small function.\r
94 \r
95 Anyway I had hope it was just a trivial rebase and could go in, but the\r
96 above will take a bit longer to do neatly so I will leave it for now.\r
97 \r
98 I think it could go in relatively easily if resubmitted with this\r
99 change but I will mark this version `moreinfo'.\r
100 \r
101 \r
102 Best wishes\r
103 \r
104 Mark\r
105 \r
106 > Best wishes\r
107 >\r
108 > Mark\r
109 >\r
110 >\r
111 > Peter Wang (3):\r
112 >   test: normalize only message filenames in show json\r
113 >   show: indicate length, encoding of omitted body content\r
114 >   test: conform to content length, encoding fields\r
115 >\r
116 >  devel/schemata   |    9 ++++++++-\r
117 >  notmuch-show.c   |   14 ++++++++++++++\r
118 >  test/crypto      |   30 +++++++++++++++++++++---------\r
119 >  test/json        |    4 +++-\r
120 >  test/multipart   |   11 ++++++-----\r
121 >  test/sexp        |    2 +-\r
122 >  test/test-lib.sh |    2 +-\r
123 >  7 files changed, 54 insertions(+), 18 deletions(-)\r
124 >\r
125 > -- \r
126 > 1.7.9.1\r