[PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-05 Thread Tomi Ollila
On Wed, Mar 05 2014, David Bremner wrote: > Tomi Ollila writes: >> >> (IIRC openssl sha256 is available at least in Mac OS X, probably in other >> BSD:s & Solaris too) >> > > OTOH sha256sum is in coreutils, so everywhere with a gnu userland has > it. Some might not have openssl (it's priority

[PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-05 Thread David Bremner
Tomi Ollila writes: > > (IIRC openssl sha256 is available at least in Mac OS X, probably in other > BSD:s & Solaris too) > OTOH sha256sum is in coreutils, so everywhere with a gnu userland has it. Some might not have openssl (it's priority optional on Debian, e.g.). d

[PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-05 Thread Tomi Ollila
On Tue, Mar 04 2014, David Bremner wrote: > Jani Nikula writes: > >> >> I think we should use the prereq mechanism in the tests to check if a >> required test database is in place, skipping the test if not. This >> should be fairly straightforward to do, and does not care *how* the test >>

[PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-05 Thread David Bremner
Tomi Ollila writes: > > I can contribute script with interface > > ./fetch-and-check.sh [--verify-only] destdir url sha256sum > This script is basically embedded in http://pivot.cs.unb.ca/git?p=notmuch.git;a=commitdiff;h=f93e32b11e517e160c755355de55ab465206dc62 except that I rebased away

Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-05 Thread Tomi Ollila
On Tue, Mar 04 2014, David Bremner da...@tethera.net wrote: Jani Nikula j...@nikula.org writes: I think we should use the prereq mechanism in the tests to check if a required test database is in place, skipping the test if not. This should be fairly straightforward to do, and does not care

Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-05 Thread David Bremner
Tomi Ollila tomi.oll...@iki.fi writes: I can contribute script with interface ./fetch-and-check.sh [--verify-only] destdir url sha256sum This script is basically embedded in http://pivot.cs.unb.ca/git?p=notmuch.git;a=commitdiff;h=f93e32b11e517e160c755355de55ab465206dc62 except that I

Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-05 Thread Tomi Ollila
On Wed, Mar 05 2014, David Bremner da...@tethera.net wrote: Tomi Ollila tomi.oll...@iki.fi writes: I can contribute script with interface ./fetch-and-check.sh [--verify-only] destdir url sha256sum This script is basically embedded in

Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-05 Thread David Bremner
Tomi Ollila tomi.oll...@iki.fi writes: (IIRC openssl sha256 is available at least in Mac OS X, probably in other BSD:s Solaris too) OTOH sha256sum is in coreutils, so everywhere with a gnu userland has it. Some might not have openssl (it's priority optional on Debian, e.g.). d

Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-05 Thread Tomi Ollila
On Wed, Mar 05 2014, David Bremner da...@tethera.net wrote: Tomi Ollila tomi.oll...@iki.fi writes: (IIRC openssl sha256 is available at least in Mac OS X, probably in other BSD:s Solaris too) OTOH sha256sum is in coreutils, so everywhere with a gnu userland has it. Some might not have

[PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-04 Thread Jani Nikula
On Sun, 02 Mar 2014, David Bremner wrote: > Jani Nikula writes: >> >> I've dropped most of the content in patches 7 and 10 due to their >> size. The patches (and the whole series) are available in the >> boolean-folder-and-path-v2 branch at >> git://gitorious.org/jani/notmuch.git. Web interface

[PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-04 Thread David Bremner
Jani Nikula writes: > > I think we should use the prereq mechanism in the tests to check if a > required test database is in place, skipping the test if not. This > should be fairly straightforward to do, and does not care *how* the test > database is to be fetched. (I'm undecided whether the

Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-04 Thread Jani Nikula
On Sun, 02 Mar 2014, David Bremner da...@tethera.net wrote: Jani Nikula j...@nikula.org writes: I've dropped most of the content in patches 7 and 10 due to their size. The patches (and the whole series) are available in the boolean-folder-and-path-v2 branch at

Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-04 Thread David Bremner
Jani Nikula j...@nikula.org writes: I think we should use the prereq mechanism in the tests to check if a required test database is in place, skipping the test if not. This should be fairly straightforward to do, and does not care *how* the test database is to be fetched. (I'm undecided

[PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-02 Thread David Bremner
Jani Nikula writes: > > I've dropped most of the content in patches 7 and 10 due to their > size. The patches (and the whole series) are available in the > boolean-folder-and-path-v2 branch at > git://gitorious.org/jani/notmuch.git. Web interface at >

Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-03-02 Thread David Bremner
Jani Nikula j...@nikula.org writes: I've dropped most of the content in patches 7 and 10 due to their size. The patches (and the whole series) are available in the boolean-folder-and-path-v2 branch at git://gitorious.org/jani/notmuch.git. Web interface at

[PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-02-23 Thread Tomi Ollila
On Sun, Feb 23 2014, Mark Walters wrote: > I have read most of this series, tested it and run the tests and LGTM +1. > > I read the C code fairly carefully, the tests rather less so but they > looked sane, and I didn't really look at patch 9 for building old > databases. Well, I can add (at

[PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-02-23 Thread Jani Nikula
Hi all, this is v2 of id:cover.1389304779.git.jani at nikula.org. The new path: prefix is a literal boolean prefix matching the paths, relative from the maildir root, of the message files. There's no interpretation of the maildir special cur/new folders, but a recursive match is provided with

Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-02-23 Thread Tomi Ollila
On Sun, Feb 23 2014, Mark Walters markwalters1...@gmail.com wrote: I have read most of this series, tested it and run the tests and LGTM +1. I read the C code fairly carefully, the tests rather less so but they looked sane, and I didn't really look at patch 9 for building old databases.

[PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-02-22 Thread Mark Walters
I have read most of this series, tested it and run the tests and LGTM +1. I read the C code fairly carefully, the tests rather less so but they looked sane, and I didn't really look at patch 9 for building old databases. Best wishes Mark On Sat, 22 Feb 2014, Jani Nikula wrote: > Hi all,

Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix

2014-02-22 Thread Mark Walters
I have read most of this series, tested it and run the tests and LGTM +1. I read the C code fairly carefully, the tests rather less so but they looked sane, and I didn't really look at patch 9 for building old databases. Best wishes Mark On Sat, 22 Feb 2014, Jani Nikula j...@nikula.org