Re: notmuch-search-toggle-order and notmuch-tree
[notmuch-archives.git] / 0f / 99a5897f28c520da767e7d6a6f65a23a8c025f
1 Return-Path: <jrollins@finestructure.net>\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 BE529431FB6\r
6         for <notmuch@notmuchmail.org>; Thu,  8 Mar 2012 13:33:11 -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: -2.29\r
10 X-Spam-Level: \r
11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] 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 4tJfoOv3XNYZ for <notmuch@notmuchmail.org>;\r
16         Thu,  8 Mar 2012 13:33:11 -0800 (PST)\r
17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
18         [131.215.239.19])\r
19         by olra.theworths.org (Postfix) with ESMTP id ED61A431FAE\r
20         for <notmuch@notmuchmail.org>; Thu,  8 Mar 2012 13:33:10 -0800 (PST)\r
21 Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
22         by earth-doxen-postvirus (Postfix) with ESMTP id 6FFC066E01EE;\r
23         Thu,  8 Mar 2012 13:33:10 -0800 (PST)\r
24 X-Spam-Scanned: at Caltech-IMSS on earth-doxen by amavisd-new\r
25 Received: from finestructure.net (DHCP-123-180.caltech.edu [131.215.123.180])\r
26         (Authenticated sender: jrollins)\r
27         by earth-doxen-submit (Postfix) with ESMTP id 292F566E00DA;\r
28         Thu,  8 Mar 2012 13:32:47 -0800 (PST)\r
29 Received: by finestructure.net (Postfix, from userid 1000)\r
30         id CAA149DE; Thu,  8 Mar 2012 13:32:46 -0800 (PST)\r
31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
32 To: Daniel Kahn Gillmor <dkg@fifthhorseman.net>,\r
33         David Bremner <david@tethera.net>,\r
34         Austin Clements <amdragon@MIT.EDU>, notmuch@notmuchmail.org\r
35 Subject: Re: Parsing regression with gmime-2.6?\r
36 In-Reply-To: <87vcme3kf6.fsf@pip.fifthhorseman.net>\r
37 References: <87d38w2e7h.fsf@zancas.localnet>\r
38         <1331058417-13776-1-git-send-email-amdragon@mit.edu>\r
39         <87wr6xmlml.fsf@zancas.localnet>\r
40         <87vcme3kf6.fsf@pip.fifthhorseman.net>\r
41 User-Agent: Notmuch/0.11.1+266~gadb05cc (http://notmuchmail.org) Emacs/23.3.1\r
42         (x86_64-pc-linux-gnu)\r
43 Date: Thu, 08 Mar 2012 13:32:44 -0800\r
44 Message-ID: <874ntybwxv.fsf@servo.finestructure.net>\r
45 MIME-Version: 1.0\r
46 Content-Type: multipart/signed; boundary="=-=-=";\r
47         micalg=pgp-sha256; protocol="application/pgp-signature"\r
48 X-BeenThere: notmuch@notmuchmail.org\r
49 X-Mailman-Version: 2.1.13\r
50 Precedence: list\r
51 List-Id: "Use and development of the notmuch mail system."\r
52         <notmuch.notmuchmail.org>\r
53 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
54         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
55 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
56 List-Post: <mailto:notmuch@notmuchmail.org>\r
57 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
58 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
59         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
60 X-List-Received-Date: Thu, 08 Mar 2012 21:33:11 -0000\r
61 \r
62 --=-=-=\r
63 Content-Transfer-Encoding: quoted-printable\r
64 \r
65 On Thu, 08 Mar 2012 15:30:21 -0500, Daniel Kahn Gillmor <dkg@fifthhorseman.=\r
66 net> wrote:\r
67 > This does seem to be a regression in gmime 2.6.  I've reported the bug\r
68 > upstream, along with a simplified (non-notmuch) demonstration:\r
69 >=20\r
70 >  https://bugzilla.gnome.org/show_bug.cgi?id=3D671680\r
71 >=20\r
72 > We'll see what gmime's upstream has to say about it.\r
73 \r
74 Thanks so much for following up on this, Daniel.\r
75 \r
76 > As a devil's advocate, i could argue that a message in a maildir that\r
77 > starts with a "From " line isn't a proper e-mail message in the first\r
78 > place, and therefore gmime 2.6 is being more rigorously correct about\r
79 > what it accepts.  In particular, if a user were to place a multi-message\r
80 > mbox file in their notmuch message store, i think that notmuch linked\r
81 > against 2.4 would happily index only the first message of it, and the\r
82 > rest of the message would be "hidden", whereas gmime 2.6 allows us to\r
83 > detect these failures and avoid indexing them directly.\r
84 \r
85 I actually agree with this position.  mbox files are not proper email\r
86 messages, so if gmime does not explicitly support parsing them then we\r
87 really can't expect it to parse them.  We *can* expect gmime to return\r
88 some sort of proper error message/return code/etc and not fail in a bad\r
89 way, but beyond that I think the burden is on us.\r
90 \r
91 > That said, i understand that this is probably not an entirely rare\r
92 > situation, and i lean toward the idea that gmime 2.4's behavior was\r
93 > actually the Right Thing.  Also, I haven't been able to find any\r
94 > explicit documentation to indicate that the behavior change was\r
95 > a deliberate one.\r
96 \r
97 And I think I don't agree with this one: I not think that handling mbox\r
98 files in the way you've outlined above is the Right Thing.  Only\r
99 partially parsing an mbox file, by indexing only the first message for\r
100 instance, seems like a bad solution to me, and one that's likely to lead\r
101 to a lot of confusion (e.g. "where are the rest of the messages that\r
102 were in my mbox?").\r
103 \r
104 If notmuch encounters an mbox file in the store it should just skip the\r
105 message and continue, while indicating that it is skipping the message\r
106 because it's not a proper email message, as it does with other non-email\r
107 files.\r
108 \r
109 jamie.\r
110 \r
111 --=-=-=\r
112 Content-Type: application/pgp-signature\r
113 \r
114 -----BEGIN PGP SIGNATURE-----\r
115 Version: GnuPG v1.4.11 (GNU/Linux)\r
116 \r
117 iQIcBAEBCAAGBQJPWSV8AAoJEO00zqvie6q86N0P/RT0X702Jhcnl+53tFyUjFUH\r
118 UkAH+Myqp+6di7YVvoFoDrATmXVGj4S0Z2QeO+MEjSbfmuY2Xx9V4V2Sh5QQ08ZW\r
119 TeZPQ8qYkWI0FQsG6nQjZhAbxf6QEJ0DRBdDYsuNJK1a6FwayE2bqDvvSw9xf7fE\r
120 D/C0Dleucr3L10utZuSmxNcJLB/flpPI3A6GcUgwf81OF5N2paqsKUW1rZdYtHkJ\r
121 y8Go7rHMmsaw82ABQfG/eoEKTotQNGgNpciWsOub7/CT/PC9yF+yQBr44VsVIhTn\r
122 w0tYpiy1pNFQ1eWLDe1QWDW+dv1HNYDqBeThURN3A90IgV49IVEjfJHLMaxtSKuh\r
123 KCgniEOB0V40LY01Bt3ot1yHcMbY58slH15NFbVINQifs2egtRuESwBgeO+EcbGU\r
124 sfqP3nrAS2aDD/bhnEeQlEidGD4xsXLVk1bT7pj+2oSImj7rXd9ph/XJCrX0KCyI\r
125 3HtOkAnsxFer2x6UYAaLQ3D4CDKppwvrcwMH1pHckR6J+Foz5GC7Eq3AxGj9fUGJ\r
126 +SAsRFucJ50LnumziD3nQf+n+PBLfsp22ZVahT0waJJaxIiqMGdko/8mI9A4CSlA\r
127 uflptkIfkiSanbt7xkk+MI7MGonvRfT4m1G/5gHqdwhsihoeHI6fIfiTXq30ozBg\r
128 vCgiuNNbMG6lhfXq6o9X\r
129 =Flnt\r
130 -----END PGP SIGNATURE-----\r
131 --=-=-=--\r