Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix
authorJani Nikula <jani@nikula.org>
Tue, 4 Mar 2014 19:37:35 +0000 (21:37 +0200)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 18:00:20 +0000 (10:00 -0800)
b4/04401229a56ef4f3bf789679165639f8f41612 [new file with mode: 0644]

diff --git a/b4/04401229a56ef4f3bf789679165639f8f41612 b/b4/04401229a56ef4f3bf789679165639f8f41612
new file mode 100644 (file)
index 0000000..dce0360
--- /dev/null
@@ -0,0 +1,143 @@
+Return-Path: <jani@nikula.org>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id 36D7D431FBF\r
+       for <notmuch@notmuchmail.org>; Tue,  4 Mar 2014 11:37:44 -0800 (PST)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.7\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
+       tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id Rw6ZEeH76ckI for <notmuch@notmuchmail.org>;\r
+       Tue,  4 Mar 2014 11:37:40 -0800 (PST)\r
+Received: from mail-ee0-f54.google.com (mail-ee0-f54.google.com\r
+ [74.125.83.54])       (using TLSv1 with cipher RC4-SHA (128/128 bits))        (No client\r
+ certificate requested)        by olra.theworths.org (Postfix) with ESMTPS id\r
+ BA2BF431FBD   for <notmuch@notmuchmail.org>; Tue,  4 Mar 2014 11:37:39 -0800\r
+ (PST)\r
+Received: by mail-ee0-f54.google.com with SMTP id d49so1698533eek.13\r
+       for <notmuch@notmuchmail.org>; Tue, 04 Mar 2014 11:37:38 -0800 (PST)\r
+X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
+       d=1e100.net; s=20130820;\r
+       h=x-gm-message-state:from:to:subject:in-reply-to:references\r
+       :user-agent:date:message-id:mime-version:content-type;\r
+       bh=o0+Yv1XiVA2mOZOpDPK6MSpxNo95NTOKqdTCp1SPUZo=;\r
+       b=Po0jPcl8NQsyb3HKr0NU/bXm/C4Xre8OBbOVPKcOfZsrwh0STuBhDvzhLt/plYKg6e\r
+       UXLQ4RcTkwHn511UaCYuhIKyTZJtuZB5prB9B7lVcnuSDzZHnklrwfRR5V5gqOsSLpNO\r
+       6DuV7cNv5AhIjeQ6/9Bq6o3efLuYeFgYv6x3ncTYEDoTlC1Eiwc6hKseb22v0K6peLE/\r
+       EkyNkAWyz9hgU3VKLONtmG7+6/Pxjt57nWsCiGW1Bz9ZX/EVB4ZP83KccaQC8ZKmp28H\r
+       0tvclLAQhQpDgBOJbZEkqfYrekfC4wLDrJ6VLIr8IulnA92lUuTxytO0QghdcSUhSRD4\r
+       YiYA==\r
+X-Gm-Message-State:\r
+ ALoCoQlGtACDEBUf0+UgsnM28rxX5wShYdNzUbgXRVJy46uELBuv6gMpJanIjbdZQSw/d+lu5IgS\r
+X-Received: by 10.14.175.129 with SMTP id z1mr1116153eel.97.1393961858437;\r
+       Tue, 04 Mar 2014 11:37:38 -0800 (PST)\r
+Received: from localhost (dsl-hkibrasgw2-58c36f-91.dhcp.inet.fi.\r
+       [88.195.111.91]) by mx.google.com with ESMTPSA id\r
+       o43sm66900569eef.12.2014.03.04.11.37.36 for <multiple recipients>\r
+       (version=TLSv1.2 cipher=RC4-SHA bits=128/128);\r
+       Tue, 04 Mar 2014 11:37:37 -0800 (PST)\r
+From: Jani Nikula <jani@nikula.org>\r
+To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org\r
+Subject: Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix\r
+In-Reply-To: <87mwh8mn58.fsf@zancas.localnet>\r
+References: <cover.1393105055.git.jani@nikula.org>\r
+       <87mwh8mn58.fsf@zancas.localnet>\r
+User-Agent: Notmuch/0.17+101~g2c5a6ae9e6e9 (http://notmuchmail.org)\r
+       Emacs/24.3.1 (x86_64-pc-linux-gnu)\r
+Date: Tue, 04 Mar 2014 21:37:35 +0200\r
+Message-ID: <87siqxiwlc.fsf@nikula.org>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Tue, 04 Mar 2014 19:37:44 -0000\r
+\r
+On Sun, 02 Mar 2014, David Bremner <david@tethera.net> wrote:\r
+> Jani Nikula <jani@nikula.org> writes:\r
+>>\r
+>> I've dropped most of the content in patches 7 and 10 due to their\r
+>> size. The patches (and the whole series) are available in the\r
+>> boolean-folder-and-path-v2 branch at\r
+>> git://gitorious.org/jani/notmuch.git. Web interface at\r
+>> https://gitorious.org/jani/notmuch/commits/0b3dd2d1cc6c413ea07ea326883ac448499c0e79.\r
+>>\r
+>\r
+> Hi Jani;\r
+>\r
+> I put a some proposed changes at \r
+>\r
+>   http://pivot.cs.unb.ca/git?p=notmuch.git;a=shortlog;h=refs/heads/boolean-folder-and-path-v2\r
+>\r
+> After playing with it a bit, I decided to \r
+>\r
+>       - leave the source for the corpuses in the main tree. As we\r
+>         discussed, we don't expect them to change much, and they are\r
+>         text files.\r
+>\r
+>       - store the binary database blobs externally. We could make the\r
+>         same argument about infrequent changes here, except that even\r
+>         the tiniest change will in practice require a whole new copy be\r
+>         stored.\r
+>\r
+>         - compute a checksum file\r
+>         - test: add machinery to fetch and publish test databases\r
+>         - test: commit database-v1.tar.xz checksum, ignore actual database\r
+>\r
+>        - this is an independant bug fix that should probably be pushed anyway:\r
+>\r
+>          - test: don't use $(dir) in recipes.\r
+>\r
+>       - I also made some unrelated changes including switching to xz\r
+>         compression. I intentionally didn't squash my patches into yours\r
+>         so you see and decide\r
+>\r
+>         - convert to XZ compression\r
+>         - convert test to XZ\r
+>\r
+>         Originally I thought metadata in the database tarball is\r
+>         needed. It turns out not. I think it's harmless, but feel free\r
+>         to drop it.\r
+>\r
+>         - add some metadata to the database tarball\r
+\r
+I'm fine with all of the above.\r
+\r
+>       - I struggled to find a way that would allow people to manage the\r
+>         tarballs in a nice way while not requiring everyone to\r
+>         install/learn new tools. My conclusion is in\r
+>       \r
+>         - test: add support for git annex managing test databases\r
+>\r
+> We still need to think about what to do for people installing from\r
+> tarballs. As it stands they will need to either have network access or\r
+> skip the relevant tests. Probably at least one followup patch is needed\r
+> here. The alternative would be to include the database tarballs in the\r
+> source; this will pretty quickly double the size of the source tarball.\r
+\r
+I think we should use the prereq mechanism in the tests to check if a\r
+required test database is in place, skipping the test if not. This\r
+should be fairly straightforward to do, and does not care *how* the test\r
+database is to be fetched. (I'm undecided whether the test should verify\r
+the checksum, or the part that fetches the database. Minor detail.)\r
+\r
+I know git annex in principle, but don't have any practical experience\r
+with it. I don't really have any ideas for that.\r
+\r
+\r
+BR,\r
+Jani.\r