Re: Applying patches directly from emails?
[notmuch-archives.git] / ee / 46258f954a3190e31301b017cbe24f40ae82bf
1 Return-Path: <david@tethera.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 3DF76431FBD\r
6         for <notmuch@notmuchmail.org>; Sun,  2 Mar 2014 11:11:27 -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 RKIpe95tZL65 for <notmuch@notmuchmail.org>;\r
16         Sun,  2 Mar 2014 11:11:22 -0800 (PST)\r
17 Received: from yantan.tethera.net (yantan.tethera.net [199.188.72.155])\r
18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
19         (No client certificate requested)\r
20         by olra.theworths.org (Postfix) with ESMTPS id E467A431FBC\r
21         for <notmuch@notmuchmail.org>; Sun,  2 Mar 2014 11:11:22 -0800 (PST)\r
22 Received: from remotemail by yantan.tethera.net with local (Exim 4.80)\r
23         (envelope-from <david@tethera.net>)\r
24         id 1WKBnD-0004oI-7p; Sun, 02 Mar 2014 15:11:19 -0400\r
25 Received: (nullmailer pid 17144 invoked by uid 1000); Sun, 02 Mar 2014\r
26         19:11:15 -0000\r
27 From: David Bremner <david@tethera.net>\r
28 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
29 Subject: Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix\r
30 In-Reply-To: <cover.1393105055.git.jani@nikula.org>\r
31 References: <cover.1393105055.git.jani@nikula.org>\r
32 User-Agent: Notmuch/0.17 (http://notmuchmail.org) Emacs/24.3.1\r
33         (x86_64-pc-linux-gnu)\r
34 Date: Sun, 02 Mar 2014 15:11:15 -0400\r
35 Message-ID: <87mwh8mn58.fsf@zancas.localnet>\r
36 MIME-Version: 1.0\r
37 Content-Type: text/plain\r
38 X-BeenThere: notmuch@notmuchmail.org\r
39 X-Mailman-Version: 2.1.13\r
40 Precedence: list\r
41 List-Id: "Use and development of the notmuch mail system."\r
42         <notmuch.notmuchmail.org>\r
43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
46 List-Post: <mailto:notmuch@notmuchmail.org>\r
47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
50 X-List-Received-Date: Sun, 02 Mar 2014 19:11:27 -0000\r
51 \r
52 Jani Nikula <jani@nikula.org> writes:\r
53 >\r
54 > I've dropped most of the content in patches 7 and 10 due to their\r
55 > size. The patches (and the whole series) are available in the\r
56 > boolean-folder-and-path-v2 branch at\r
57 > git://gitorious.org/jani/notmuch.git. Web interface at\r
58 > https://gitorious.org/jani/notmuch/commits/0b3dd2d1cc6c413ea07ea326883ac448499c0e79.\r
59 >\r
60 \r
61 Hi Jani;\r
62 \r
63 I put a some proposed changes at \r
64 \r
65   http://pivot.cs.unb.ca/git?p=notmuch.git;a=shortlog;h=refs/heads/boolean-folder-and-path-v2\r
66 \r
67 After playing with it a bit, I decided to \r
68 \r
69       - leave the source for the corpuses in the main tree. As we\r
70         discussed, we don't expect them to change much, and they are\r
71         text files.\r
72 \r
73       - store the binary database blobs externally. We could make the\r
74         same argument about infrequent changes here, except that even\r
75         the tiniest change will in practice require a whole new copy be\r
76         stored.\r
77 \r
78         - compute a checksum file\r
79         - test: add machinery to fetch and publish test databases\r
80         - test: commit database-v1.tar.xz checksum, ignore actual database\r
81 \r
82        - this is an independant bug fix that should probably be pushed anyway:\r
83 \r
84          - test: don't use $(dir) in recipes.\r
85 \r
86       - I also made some unrelated changes including switching to xz\r
87         compression. I intentionally didn't squash my patches into yours\r
88         so you see and decide\r
89 \r
90         - convert to XZ compression\r
91         - convert test to XZ\r
92 \r
93         Originally I thought metadata in the database tarball is\r
94         needed. It turns out not. I think it's harmless, but feel free\r
95         to drop it.\r
96 \r
97         - add some metadata to the database tarball\r
98         \r
99       - I struggled to find a way that would allow people to manage the\r
100         tarballs in a nice way while not requiring everyone to\r
101         install/learn new tools. My conclusion is in\r
102       \r
103         - test: add support for git annex managing test databases\r
104 \r
105 We still need to think about what to do for people installing from\r
106 tarballs. As it stands they will need to either have network access or\r
107 skip the relevant tests. Probably at least one followup patch is needed\r
108 here. The alternative would be to include the database tarballs in the\r
109 source; this will pretty quickly double the size of the source tarball.\r
110 \r
111 \r