Re: [PATCH 9/9] add has: query prefix to search for specific properties
[notmuch-archives.git] / 0f / a0bd0d5d916eaca5e4afd55cbe66f82ecd696a
1 Return-Path: <bremner@unb.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 9C726431FAF\r
6         for <notmuch@notmuchmail.org>; Wed, 26 Dec 2012 06:55:17 -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 tests=[none]\r
12         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 GjNZD3CmTbz4 for <notmuch@notmuchmail.org>;\r
16         Wed, 26 Dec 2012 06:55:17 -0800 (PST)\r
17 Received: from tesseract.cs.unb.ca (tesseract.cs.unb.ca [131.202.240.238])\r
18         (using TLSv1 with cipher AES256-SHA (256/256 bits))\r
19         (No client certificate requested)\r
20         by olra.theworths.org (Postfix) with ESMTPS id 17C2A431FAE\r
21         for <notmuch@notmuchmail.org>; Wed, 26 Dec 2012 06:55:17 -0800 (PST)\r
22 Received: from fctnnbsc30w-156034082078.dhcp-dynamic.fibreop.nb.bellaliant.net\r
23         ([156.34.82.78] helo=zancas.localnet)\r
24         by tesseract.cs.unb.ca with esmtpsa\r
25         (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.72)\r
26         (envelope-from <bremner@unb.ca>)\r
27         id 1TnsO0-0002X1-76; Wed, 26 Dec 2012 10:55:12 -0400\r
28 Received: from bremner by zancas.localnet with local (Exim 4.80)\r
29         (envelope-from <bremner@unb.ca>)\r
30         id 1TnsNu-0007TF-Jb; Wed, 26 Dec 2012 10:55:06 -0400\r
31 From: David Bremner <david@tethera.net>\r
32 To: Austin Clements <amdragon@MIT.EDU>, notmuch@notmuchmail.org\r
33 Subject: Re: [PATCH v2 0/5] Use Xapian query syntax for batch-tag dump/restore\r
34 In-Reply-To: <1356493723-11085-1-git-send-email-amdragon@mit.edu>\r
35 References: <1356493723-11085-1-git-send-email-amdragon@mit.edu>\r
36 User-Agent: Notmuch/0.14+213~g4af1ac6 (http://notmuchmail.org) Emacs/24.2.1\r
37         (x86_64-pc-linux-gnu)\r
38 Date: Wed, 26 Dec 2012 10:55:06 -0400\r
39 Message-ID: <87ehicrhh1.fsf@zancas.localnet>\r
40 MIME-Version: 1.0\r
41 Content-Type: text/plain\r
42 X-Spam_bar: -\r
43 X-BeenThere: notmuch@notmuchmail.org\r
44 X-Mailman-Version: 2.1.13\r
45 Precedence: list\r
46 List-Id: "Use and development of the notmuch mail system."\r
47         <notmuch.notmuchmail.org>\r
48 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
49         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
50 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
51 List-Post: <mailto:notmuch@notmuchmail.org>\r
52 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
53 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
54         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
55 X-List-Received-Date: Wed, 26 Dec 2012 14:55:17 -0000\r
56 \r
57 Austin Clements <amdragon@MIT.EDU> writes:\r
58 \r
59 > This obsoletes\r
60 >\r
61 >   id:1356415076-5692-1-git-send-email-amdragon@mit.edu\r
62 >\r
63 > In addition to incorporating all of David's suggestions, this reworks\r
64 > the boolean term parsing so it only handles the subset of quoting\r
65 > syntax used by make_boolean_term (which also happens to be all that we\r
66 > described in the man page for the format).  The diff from v1 is below.\r
67 \r
68 Implementation wise this series now looks OK to me; I think the dequote\r
69 routine could be slightly clearer, but it's a bit subjective at this\r
70 point.\r
71 \r
72 IMHO the real question is if we like (or can live with) the resulting UI\r
73 for batch tagging.  Mark ( id:87ehid5h64.fsf@qmul.ac.uk ) brings up the\r
74 question of whether tags with newlines should be somehow supported in a\r
75 batch-tagging context. The other UI issue I'm aware of is the fact that\r
76 tags with spaces needed to be quoted in two different ways for tag\r
77 operations and for queries.  A relatively smooth upgrade path would be\r
78 to support Xapian quoting for tag operations in the future.\r
79 \r
80 d\r
81 \r