Re: [PATCH 9/9] add has: query prefix to search for specific properties
[notmuch-archives.git] / dc / 1d0a3530a67d849efbd0a3e5c939a84c72eb30
1 Return-Path: <ethan.glasser.camp@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 8411B431FBC\r
6         for <notmuch@notmuchmail.org>; Sun,  1 Jul 2012 09:40:19 -0700 (PDT)\r
7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
8 X-Spam-Flag: NO\r
9 X-Spam-Score: 1.061\r
10 X-Spam-Level: *\r
11 X-Spam-Status: No, score=1.061 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, RCVD_IN_BL_SPAMCOP_NET=1.246,\r
14         RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_SORBS_WEB=0.614] 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 dU4Y8dfgucAE for <notmuch@notmuchmail.org>;\r
18         Sun,  1 Jul 2012 09:40:19 -0700 (PDT)\r
19 Received: from mail-wg0-f45.google.com (mail-wg0-f45.google.com\r
20  [74.125.82.45])        (using TLSv1 with cipher RC4-SHA (128/128 bits))        (No client\r
21  certificate requested) by olra.theworths.org (Postfix) with ESMTPS id\r
22  C206D431FAE    for <notmuch@notmuchmail.org>; Sun,  1 Jul 2012 09:40:18 -0700\r
23  (PDT)\r
24 Received: by wgbdt14 with SMTP id dt14so3718702wgb.2\r
25         for <notmuch@notmuchmail.org>; Sun, 01 Jul 2012 09:40:16 -0700 (PDT)\r
26 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;\r
27         h=from:to:subject:date:message-id:x-mailer:in-reply-to:references;\r
28         bh=4DpI0eDxkxX4K2lpcUiJxHAhLd/N98dUJ1d1DmIysLw=;\r
29         b=Ct/uxqo6z5a5jWtyZA8WaXMVvf6TV9UL9FkrABSbjrs4jJMe9Zk5om5uYhsuwq5IWc\r
30         O0OP90SjBCze0f/J30YNyIp7MBaKFRjTklQbo/+2SybVhFdfFSx+366fWWiAzZ6mw7Ky\r
31         d/qSrg28GHCLAwD8Q0eD5MXcw1NDptgvZLfZVZEEX0yn+kr/DAz/R/gFygfu1u0Nn3V3\r
32         2wVe71BEko7lzk3v1n6tup20q9a5UpeVDb/SGhaxkJxZGnAMf7RViCWSyz1VocTB+qs5\r
33         6jlRDlYNKPM932iKU4pGaipKguaGMmK02QtzQ96aVTi+mqbnh2hL6RMH9TbucHZBFL4r\r
34         X3UQ==\r
35 Received: by 10.180.103.4 with SMTP id fs4mr11193008wib.16.1341160816022;\r
36         Sun, 01 Jul 2012 09:40:16 -0700 (PDT)\r
37 Received: from localhost ([195.24.209.21])\r
38         by mx.google.com with ESMTPS id fu8sm19686504wib.5.2012.07.01.09.40.06\r
39         (version=TLSv1/SSLv3 cipher=OTHER);\r
40         Sun, 01 Jul 2012 09:40:15 -0700 (PDT)\r
41 From: Ethan Glasser-Camp <ethan.glasser.camp@gmail.com>\r
42 To: notmuch@notmuchmail.org\r
43 Subject: [PATCH v2 0/8] URI-based modular mail stores\r
44 Date: Sun,  1 Jul 2012 12:39:42 -0400\r
45 Message-Id: <1341160790-14525-1-git-send-email-ethan@betacantrips.com>\r
46 X-Mailer: git-send-email 1.7.9.5\r
47 In-Reply-To:\r
48  <CAOJ+Ob0MSOez2MvD2fCgF7t32kFPk4g2+xCud88QmBLt_b5pOA@mail.gmail.com>\r
49 References:\r
50  <CAOJ+Ob0MSOez2MvD2fCgF7t32kFPk4g2+xCud88QmBLt_b5pOA@mail.gmail.com>\r
51 X-BeenThere: notmuch@notmuchmail.org\r
52 X-Mailman-Version: 2.1.13\r
53 Precedence: list\r
54 List-Id: "Use and development of the notmuch mail system."\r
55         <notmuch.notmuchmail.org>\r
56 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
57         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
58 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
59 List-Post: <mailto:notmuch@notmuchmail.org>\r
60 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
61 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
62         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
63 X-List-Received-Date: Sun, 01 Jul 2012 16:40:19 -0000\r
64 \r
65 Hi guys,\r
66 \r
67 Based on the feedback from the last patch series (thanks a lot to Mark\r
68 Walters and Jani Nikula for taking the time to look at it!) I revised\r
69 this patch series to not change the handling of maildir filenames at\r
70 all. This meant not having to give up relocatability, and also\r
71 simplified the patch series a great deal (down to 8 patches instead of\r
72 14). I also noticed that there is only one use of notmuch_sha1_of_file\r
73 and it is only used on messages, so instead of splitting the function\r
74 as in the last patch series, I just renamed it.\r
75 \r
76 As a reminder, this patch series proposes a mailstore.c module, which\r
77 is responsible for accessing "filenames" which, in addition to\r
78 ordinary filenames, can also be URIs.\r
79 \r
80 - Patches 1-5 add the mailstore module and complementary interfaces,\r
81   such as a new.scan option that tell notmuch-new where to look for\r
82   other mail.\r
83 \r
84 - Patches 6-8 prove the validity of this interface by using it to add\r
85   mbox support.\r
86 \r
87 I know this is still pretty big and does two different things, but\r
88 mbox support relies on URI support, and URI support is meaningless\r
89 without some new kind of mail store.\r
90 \r
91 There are a few missing features in this patch series, but I was\r
92 hesitant to add them because I didn't want to make the series any more\r
93 complex. To wit, future series could include:\r
94 \r
95 - notmuch-new.c: some code is shared between add_files_recursive and\r
96   add_files_mbox{,_file}, specifically the code that scans directories\r
97   for subdirectories and then recurses, and the code that scans\r
98   directories for regular files and then does something else. Some\r
99   guidance on how best to refactor that, or whether to even bother,\r
100   would be appreciated.\r
101 \r
102 - notmuch-config.c: I think instead of one option for new.scan, it\r
103   would be good to have a section called [urls], for which each entry\r
104   is name=url. This would also allow relocatability for mail not\r
105   located in the central maildir (store the name in Xapian, and\r
106   translate it to the URL on the way in/out).\r
107 \r
108 - notmuch-new.c: code could be added to notice when we "walked past" a\r
109   message in an mbox file and mark it as deleted, as is currently done\r
110   with maildir.\r
111 \r
112 - notmuch-new.c: no support for count_files kinds of operations\r
113   exists.\r
114 \r
115 Unlike the previous patch series, I would appreciate it if you\r
116 reviewed this one :)\r
117 \r
118 Thanks for your time!\r
119 \r
120 Ethan\r
121 \r