Re: [PATCH v4 3/5] Use the S-Expression structured printer in notmuch-show, notmuch...
[notmuch-archives.git] / 41 / 574ce3ab043f89de1f7469e010a0cddf5332eb
1 Return-Path: <Sebastian@SSpaeth.de>\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 C87B5431FBC\r
6         for <notmuch@notmuchmail.org>; Wed, 27 Jan 2010 01:15:51 -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.404\r
10 X-Spam-Level: \r
11 X-Spam-Status: No, score=-0.404 tagged_above=-999 required=5\r
12         tests=[AWL=-0.405, BAYES_50=0.001] autolearn=ham\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 ZIeV0txg3RgA for <notmuch@notmuchmail.org>;\r
16         Wed, 27 Jan 2010 01:15:51 -0800 (PST)\r
17 Received: from homiemail-a25.g.dreamhost.com (caiajhbdcagg.dreamhost.com\r
18         [208.97.132.66])\r
19         by olra.theworths.org (Postfix) with ESMTP id E9A39431FAE\r
20         for <notmuch@notmuchmail.org>; Wed, 27 Jan 2010 01:15:50 -0800 (PST)\r
21 Received: from sspaeth.de (unknown [84.55.198.58])\r
22         by homiemail-a25.g.dreamhost.com (Postfix) with ESMTPA id 11C77678063; \r
23         Wed, 27 Jan 2010 01:15:48 -0800 (PST)\r
24 Received: by sspaeth.de (sSMTP sendmail emulation);\r
25         Wed, 27 Jan 2010 10:15:46 +0100\r
26 From: "Sebastian Spaeth" <Sebastian@SSpaeth.de>\r
27 To: notmuch@notmuchmail.org\r
28 In-Reply-To: <yunockgoiob.fsf@aiko.keithp.com>\r
29 References: <1264173971-11879-1-git-send-email-Sebastian@SSpaeth.de>\r
30         <yunpr4xnzik.fsf@aiko.keithp.com> <874om99jam.fsf@SSpaeth.de>\r
31         <yunockgoiob.fsf@aiko.keithp.com>\r
32 Date: Wed, 27 Jan 2010 10:15:46 +0100\r
33 Message-ID: <87eilbewn1.fsf@SSpaeth.de>\r
34 MIME-Version: 1.0\r
35 Content-Type: text/plain; charset=us-ascii\r
36 Subject: Re: [notmuch] [PATCH] Make the date parser nicer\r
37 X-BeenThere: notmuch@notmuchmail.org\r
38 X-Mailman-Version: 2.1.13\r
39 Precedence: list\r
40 List-Id: "Use and development of the notmuch mail system."\r
41         <notmuch.notmuchmail.org>\r
42 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
43         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
44 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
45 List-Post: <mailto:notmuch@notmuchmail.org>\r
46 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
47 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
48         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
49 X-List-Received-Date: Wed, 27 Jan 2010 09:15:51 -0000\r
50 \r
51 On Tue, 26 Jan 2010 09:55:00 -0800, Keith Packard <keithp@keithp.com> wrote:\r
52 > Very cool. Oh, if you've got commits that don't compile on their own,\r
53 > you should squash them together (or fix it in some other way). Makes\r
54 > bisecting easier in the future.\r
55 \r
56 Makes sense. I am still quite new to git, so excuse those beginner's\r
57 lapses. Perhaps Carl could squash commits ec3c79a and 2565fc6 when (if?)\r
58 pulling, that would make every step compile IMHO.\r
59 \r
60 > Also, cworth is on vacation this week, so we won't be seeing any\r
61 > merging to master...\r
62 \r
63 No hurry :-). cworth will have to do quite some catching up when he\r
64 returns.\r
65 \r
66 The one "disadvantage" my integration has over your original approach,\r
67 is that we now always require "date:XXX..YYY". A 'date:lastmonth' won't\r
68 work, it will need to be "date:lastmonth..today". The reason is that\r
69 xapian only seems to invoke the RangeParser when something of the format\r
70 'A..B' is passed as a parameter. So while we could get "date:..2005" to\r
71 work, "date:2005.." is not passed to the RangeParser handler, it seems.\r
72  \r
73 We could ditch the "date:" prefix, but imho it is more consistent with\r
74 the other keywords to use it. I have no strong feelings about this.\r
75 \r
76 It also still has the same limitation, in that it will not find emails with\r
77 a future timestamp (I use date:lastweek..5000 to get all mails with a\r
78 future stamp).\r
79 \r
80 Sebastian\r