b4/04401229a56ef4f3bf789679165639f8f41612

   1 Return-Path: <jani@nikula.org>\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 36D7D431FBF\r
   6         for <notmuch@notmuchmail.org>; Tue,  4 Mar 2014 11:37:44 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 Rw6ZEeH76ckI for <notmuch@notmuchmail.org>;\r
  16         Tue,  4 Mar 2014 11:37:40 -0800 (PST)\r
  17 Received: from mail-ee0-f54.google.com (mail-ee0-f54.google.com\r
  18  [74.125.83.54])        (using TLSv1 with cipher RC4-SHA (128/128 bits))        (No client\r
  19  certificate requested) by olra.theworths.org (Postfix) with ESMTPS id\r
  20  BA2BF431FBD    for <notmuch@notmuchmail.org>; Tue,  4 Mar 2014 11:37:39 -0800\r
  21  (PST)\r
  22 Received: by mail-ee0-f54.google.com with SMTP id d49so1698533eek.13\r
  23         for <notmuch@notmuchmail.org>; Tue, 04 Mar 2014 11:37:38 -0800 (PST)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=1e100.net; s=20130820;\r
  26         h=x-gm-message-state:from:to:subject:in-reply-to:references\r
  27         :user-agent:date:message-id:mime-version:content-type;\r
  28         bh=o0+Yv1XiVA2mOZOpDPK6MSpxNo95NTOKqdTCp1SPUZo=;\r
  29         b=Po0jPcl8NQsyb3HKr0NU/bXm/C4Xre8OBbOVPKcOfZsrwh0STuBhDvzhLt/plYKg6e\r
  30         UXLQ4RcTkwHn511UaCYuhIKyTZJtuZB5prB9B7lVcnuSDzZHnklrwfRR5V5gqOsSLpNO\r
  31         6DuV7cNv5AhIjeQ6/9Bq6o3efLuYeFgYv6x3ncTYEDoTlC1Eiwc6hKseb22v0K6peLE/\r
  32         EkyNkAWyz9hgU3VKLONtmG7+6/Pxjt57nWsCiGW1Bz9ZX/EVB4ZP83KccaQC8ZKmp28H\r
  33         0tvclLAQhQpDgBOJbZEkqfYrekfC4wLDrJ6VLIr8IulnA92lUuTxytO0QghdcSUhSRD4\r
  34         YiYA==\r
  35 X-Gm-Message-State:\r
  36  ALoCoQlGtACDEBUf0+UgsnM28rxX5wShYdNzUbgXRVJy46uELBuv6gMpJanIjbdZQSw/d+lu5IgS\r
  37 X-Received: by 10.14.175.129 with SMTP id z1mr1116153eel.97.1393961858437;\r
  38         Tue, 04 Mar 2014 11:37:38 -0800 (PST)\r
  39 Received: from localhost (dsl-hkibrasgw2-58c36f-91.dhcp.inet.fi.\r
  40         [88.195.111.91]) by mx.google.com with ESMTPSA id\r
  41         o43sm66900569eef.12.2014.03.04.11.37.36 for <multiple recipients>\r
  42         (version=TLSv1.2 cipher=RC4-SHA bits=128/128);\r
  43         Tue, 04 Mar 2014 11:37:37 -0800 (PST)\r
  44 From: Jani Nikula <jani@nikula.org>\r
  45 To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org\r
  46 Subject: Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix\r
  47 In-Reply-To: <87mwh8mn58.fsf@zancas.localnet>\r
  48 References: <cover.1393105055.git.jani@nikula.org>\r
  49         <87mwh8mn58.fsf@zancas.localnet>\r
  50 User-Agent: Notmuch/0.17+101~g2c5a6ae9e6e9 (http://notmuchmail.org)\r
  51         Emacs/24.3.1 (x86_64-pc-linux-gnu)\r
  52 Date: Tue, 04 Mar 2014 21:37:35 +0200\r
  53 Message-ID: <87siqxiwlc.fsf@nikula.org>\r
  54 MIME-Version: 1.0\r
  55 Content-Type: text/plain\r
  56 X-BeenThere: notmuch@notmuchmail.org\r
  57 X-Mailman-Version: 2.1.13\r
  58 Precedence: list\r
  59 List-Id: "Use and development of the notmuch mail system."\r
  60         <notmuch.notmuchmail.org>\r
  61 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  63 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  64 List-Post: <mailto:notmuch@notmuchmail.org>\r
  65 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  66 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  67         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  68 X-List-Received-Date: Tue, 04 Mar 2014 19:37:44 -0000\r
  69 \r
  70 On Sun, 02 Mar 2014, David Bremner <david@tethera.net> wrote:\r
  71 > Jani Nikula <jani@nikula.org> writes:\r
  72 >>\r
  73 >> I've dropped most of the content in patches 7 and 10 due to their\r
  74 >> size. The patches (and the whole series) are available in the\r
  75 >> boolean-folder-and-path-v2 branch at\r
  76 >> git://gitorious.org/jani/notmuch.git. Web interface at\r
  77 >> https://gitorious.org/jani/notmuch/commits/0b3dd2d1cc6c413ea07ea326883ac448499c0e79.\r
  78 >>\r
  79 >\r
  80 > Hi Jani;\r
  81 >\r
  82 > I put a some proposed changes at \r
  83 >\r
  84 >   http://pivot.cs.unb.ca/git?p=notmuch.git;a=shortlog;h=refs/heads/boolean-folder-and-path-v2\r
  85 >\r
  86 > After playing with it a bit, I decided to \r
  87 >\r
  88 >       - leave the source for the corpuses in the main tree. As we\r
  89 >         discussed, we don't expect them to change much, and they are\r
  90 >         text files.\r
  91 >\r
  92 >       - store the binary database blobs externally. We could make the\r
  93 >         same argument about infrequent changes here, except that even\r
  94 >         the tiniest change will in practice require a whole new copy be\r
  95 >         stored.\r
  96 >\r
  97 >         - compute a checksum file\r
  98 >         - test: add machinery to fetch and publish test databases\r
  99 >         - test: commit database-v1.tar.xz checksum, ignore actual database\r
 100 >\r
 101 >        - this is an independant bug fix that should probably be pushed anyway:\r
 102 >\r
 103 >          - test: don't use $(dir) in recipes.\r
 104 >\r
 105 >       - I also made some unrelated changes including switching to xz\r
 106 >         compression. I intentionally didn't squash my patches into yours\r
 107 >         so you see and decide\r
 108 >\r
 109 >         - convert to XZ compression\r
 110 >         - convert test to XZ\r
 111 >\r
 112 >         Originally I thought metadata in the database tarball is\r
 113 >         needed. It turns out not. I think it's harmless, but feel free\r
 114 >         to drop it.\r
 115 >\r
 116 >         - add some metadata to the database tarball\r
 117 \r
 118 I'm fine with all of the above.\r
 119 \r
 120 >       - I struggled to find a way that would allow people to manage the\r
 121 >         tarballs in a nice way while not requiring everyone to\r
 122 >         install/learn new tools. My conclusion is in\r
 123 >       \r
 124 >         - test: add support for git annex managing test databases\r
 125 >\r
 126 > We still need to think about what to do for people installing from\r
 127 > tarballs. As it stands they will need to either have network access or\r
 128 > skip the relevant tests. Probably at least one followup patch is needed\r
 129 > here. The alternative would be to include the database tarballs in the\r
 130 > source; this will pretty quickly double the size of the source tarball.\r
 131 \r
 132 I think we should use the prereq mechanism in the tests to check if a\r
 133 required test database is in place, skipping the test if not. This\r
 134 should be fairly straightforward to do, and does not care *how* the test\r
 135 database is to be fetched. (I'm undecided whether the test should verify\r
 136 the checksum, or the part that fetches the database. Minor detail.)\r
 137 \r
 138 I know git annex in principle, but don't have any practical experience\r
 139 with it. I don't really have any ideas for that.\r
 140 \r
 141 \r
 142 BR,\r
 143 Jani.\r