[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 op

[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: > >> >> 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=f93e32b11e5

[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 >> datab

[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 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 op

Re: [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 __

Re: [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: > >> >> 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=f93e32b11e5

Re: [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 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 >> datab

[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 a

[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 tes

Re: [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 tes

Re: [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 a

[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 > https://gitorious.org/jani/notmuch/commits/0b3

Re: [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 > https://gitorious.org/jani/notmuch/commits/0b3

[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 this

Re: [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 this

[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 "/**

[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, th

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 wrote: > Hi all, th

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

2014-02-22 Thread Jani Nikula
Hi all, this is v2 of id:cover.1389304779.git.j...@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 "/**" s