[PATCH 2/2] test: use python2 instead of python if available

Some distros (Arch Linux) ship Python as python2 and Python 3 as python. Checking for python2 is necessary for the Python tests to work on these platforms. --- test/test-lib.sh |9 - 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/test/test-lib.sh b/test/test-lib.sh

RFC: don't ask users to do 'sudo make install'

Hey everyone, I just noticed that running configure says: All required packages were found. You may now run the following commands to compile and install notmuch: make sudo make install I think that this is a very poor advice in general and we shouldn't give it to users. I

Re: [PATCH 1/4] notmuch-opts.[ch]: new argument parsing framework for notmuch.

On Tue, 06 Dec 2011 22:41:23 +0200, Jani Nikula j...@nikula.org wrote: We chatted about reserving notmuch-*.c to notmuch commands, but I guess it was only after you sent these. I think it would make sense though. renamed to command-line-arguments.[ch]. Hard luck for those of you on 8.3 file

[PATCH 4/4] notmuch-search: convert to command-line-arguments

From: David Bremner brem...@debian.org The switch on format_sel is slightly clunky, but it doesn't seem worth special casing argument processing for function pointers, when I think the function pointer approach will be modified/abandoned. --- notmuch-search.c | 109

[PATCH 1/4] command-line-arguments.[ch]: new argument parsing framework for notmuch.

From: David Bremner brem...@debian.org As we noticed when Jani kindly converted things to getopt_long, much of the work in argument parsing in notmuch is due to the the key-value style arguments like --format=(raw|json|text). The framework here provides positional arguments, simple switches, and

[PATCH 2/4] notmuch-dump: convert to command-line-arguments

From: David Bremner brem...@debian.org The output file is handled via positional arguments. There are currently no normal options. --- notmuch-dump.c | 31 +++ 1 files changed, 19 insertions(+), 12 deletions(-) diff --git a/notmuch-dump.c b/notmuch-dump.c index

[PATCH 3/4] notmuch-restore: convert to command-line-arguments

From: David Bremner brem...@debian.org The new argument handling is a bit more concise, and bit more flexible. It allows the input file name to go before the --accumulate option. --- notmuch-restore.c | 37 +++-- 1 files changed, 15 insertions(+), 22

Re: RFC: don't ask users to do 'sudo make install'

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/11 14:37, Tomi Ollila wrote: On Wed, 07 Dec 2011 13:25:45 +0100, Justus Winter 4win...@informatik.uni-hamburg.de wrote: Hey everyone, I just noticed that running configure says: All required packages were found. You may now run the

Re: [PATCH v4 0/3] emacs: do not call notmuch show for non-inlinable parts

Hi David. How about pushing this series? Just a humble ping :) Regards, Dmitry ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

Re: [PATCH 1/2] emacs: remove some code duplication in notmuch-show

David, this seems ready for push as well. Regards, Dmitry ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

Re: [PATCH v3 0/2] notmuch hooks

On Tue, 06 Dec 2011 22:16:37 -0500, Tom Prince tom.pri...@ualberta.net wrote: On Tue, 06 Dec 2011 18:47:01 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: Also, what if we make it so that the post-new hook script only runs if notmuch new processes new messages? All of my

Re: [PATCH v3 0/2] notmuch hooks

On Wed, 07 Dec 2011 20:05:22 +0200, Jani Nikula j...@nikula.org wrote: It would, but as I wrote in id:87mxb8kt5r@nikula.org, I think that should come as another patch afterwards. I know I can't decide yet what should be passed and how. Processed message counts (added, deleted, renamed)

Re: [PATCH 2/2] python: annotate all calls into libnotmuch with types

Quoting Justus Winter (2011-12-06 13:51:08) Quoting Sebastian Spaeth (2011-12-06 13:05:53) On Tue, 06 Dec 2011 10:46:31 +, Patrick Totzke patricktot...@googlemail.com wrote: This commit breaks raising XapianErrors for me. If I lock the index with some `notmuch tag +test '*'` and try to

Re: [PATCH 2/2] python: annotate all calls into libnotmuch with types

Quoting Justus Winter (2011-12-07 19:49:31) Quoting Justus Winter (2011-12-06 13:51:08) I began running alot in gdb since I get segfaults within libnotmuch from time to time and managed to get a stack trace pointing to notmuch_database_begin_atomic, but I couldn't figure out what caused it. And

[no subject]

So it turns out the completely untested NOTMUCH_OPT_STRING code was (surpise!) buggy. I also fixed a few remaining formatting issues, added a unit test, and some documentation (in command-line-arguements.h). I also reordered option description struct so that the two mandatory (in the sense of

[PATCH v4 3/5] notmuch-dump: convert to command-line-arguments

From: David Bremner brem...@debian.org The output file is handled via positional arguments. There are currently no normal options. --- notmuch-dump.c | 31 +++ 1 files changed, 19 insertions(+), 12 deletions(-) diff --git a/notmuch-dump.c b/notmuch-dump.c index

[PATCH v4 4/5] notmuch-restore: convert to command-line-arguments

From: David Bremner brem...@debian.org The new argument handling is a bit more concise, and bit more flexible. It allows the input file name to go before the --accumulate option. --- notmuch-restore.c | 37 +++-- 1 files changed, 15 insertions(+), 22

[PATCH v4 2/5] test: tests for command-line-arguments.c

From: David Bremner brem...@debian.org This was needed because no current notmuch code exercises the NOTMUCH_OPT_STRING style arguments. --- test/Makefile.local | 11 - test/arg-test.c | 52 + test/argument-parsing | 16

[PATCH v4 1/5] command-line-arguments.[ch]: new argument parsing framework for notmuch.

From: David Bremner brem...@debian.org As we noticed when Jani kindly converted things to getopt_long, much of the work in argument parsing in notmuch is due to the the key-value style arguments like --format=(raw|json|text). The framework here provides positional arguments, simple switches, and

Re: [PATCH v3 0/2] notmuch hooks

On Tue, 06 Dec 2011 18:27:27 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: On Tue, 6 Dec 2011 15:22:36 +0200, Jani Nikula j...@nikula.org wrote: Hi all, this is v3 of the notmuch hooks patches. I think this is nearing completion apart from final review and, most notably,

Re: RFC: don't ask users to do 'sudo make install'

On Wed, 07 Dec 2011 15:54:05 +0100, Rainer M Krug r.m.k...@gmail.com wrote: Interesting. I assume, I have to do ./configure make make debian-snapshot Just make debian-snapshot should work if you have the packages dpkg-dev and fakeroot installed. dpkg-buildpackage: warning: (Use -d flag to

Re: [PATCH v3 0/2] notmuch hooks

Quoth Jani Nikula on Dec 07 at 8:05 pm: On Tue, 06 Dec 2011 22:16:37 -0500, Tom Prince tom.pri...@ualberta.net wrote: On Tue, 06 Dec 2011 18:47:01 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: Also, what if we make it so that the post-new hook script only runs if

[PATCH] Release memory allocated by internet_address_list_parse_string()

g_object_unref() releases the memory of the InternetAddressList object returned by internet_address_list_parse_string() -- when last (only) reference is released, internet_address_list_finalize() will do cleanup. --- When reviewing, see also gmime-2.4.25/gmime/internet-address.c (or older) I

Re: [PATCH] Release memory allocated by internet_address_list_parse_string()

On Wed, 07 Dec 2011 22:13:19 +0200, Tomi Ollila tomi.oll...@iki.fi wrote: g_object_unref() releases the memory of the InternetAddressList object returned by internet_address_list_parse_string() -- when last (only) reference is released, internet_address_list_finalize() will do cleanup. ---

Re: [PATCH v3 0/2] notmuch hooks

On Wed, 07 Dec 2011 21:42:33 +0200, Jani Nikula j...@nikula.org wrote: I wonder, though, whether the samples should be created only when creating the database, or whenever the hooks directory does not exist? The latter, I guess. Otherwise existing notmuch users would hardly get the sample

Re: [PATCH 0/3] Configuration file option to exclude files/directories

On Wed, 14 Sep 2011 00:32:01 +0300, tomi.oll...@iki.fi wrote: This patch set adds a configuration option 'database.exclude'; a list of files/directories to be excluded when doing 'notmuch new' operation. Hi All; I could see idea of excluding spam and trash folders from indexing being useful,

Re: [PATCH 0/3] Configuration file option to exclude files/directories

On Wed, 07 Dec 2011 19:51:32 -0400, David Bremner da...@tethera.net wrote: I could see idea of excluding spam and trash folders from indexing being useful, but this has been sitting in the list for a few months without comment. Does that mean no-one but Tomi thinks the functionality is

Re: [PATCH v4 0/3] emacs: do not call notmuch show for non-inlinable parts

On Wed, 30 Nov 2011 01:19:51 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: Changes: v4 since v3: * Use $(()) instead of expr(1) for math. pushed d ___ notmuch mailing list notmuch@notmuchmail.org

Re: [PATCH 1/2] emacs: remove some code duplication in notmuch-show

On Sat, 26 Nov 2011 02:23:30 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: Add optional props argument to `notmuch-show-get-header'. Use it to get headers in `notmuch-show-insert-part-multipart/signed' and `notmuch-show-insert-part-multipart/encrypted'. Pushed this series. d

Re: [PATCH 1/2] test: date_relative in notmuch search json output

On Sat, 23 Jul 2011 10:54:03 +0100, pazz patricktot...@googlemail.com wrote: expect the date_relative field for thread entries in notmuch search's json output --- I pushed an amended version of this one (explaining why the date won't change in the future), and the next patch as well. d

Patch Amnesty Day: July 1, 2011

Hi All; As some of you know I have been making some effort to get a handle on the patches to notmuch floating around using nmbug [1]. We are doing pretty well keeping track of current patches, and I started working my way back through things that looked like patches. Sometime around the release

[PATCH] debian: add a recommends for w3m-el or w3m-el-snapshot to notmuch-emacs

From: David Bremner brem...@debian.org Installing w3m-el provides a nicer out of the box experience for viewing html in notmuch, and the overhead is not too bad (about 5M diskspace). --- debian/control |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/debian/control

ANNOUNCE: nottoomuch-addresses.pl

On Tue, 06 Dec 2011 08:45:49 +1100, Bart Bunting wrote: > Hi, > > I agree with Jamie on this one. > > The case sensitivity appears to get in the way of searching. I also > think that enabling the regular expressionsearching is a good idea. > > All in all though this is great. I did

[PATCH 2/2] test: use python2 instead of python if available

Some distros (Arch Linux) ship Python as python2 and Python 3 as python. Checking for python2 is necessary for the Python tests to work on these platforms. --- test/test-lib.sh |9 - 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/test/test-lib.sh b/test/test-lib.sh

[PATCH 1/2] test: add a function to run Python tests

The new test_python() function makes writing Python tests a little easier: - it sets the environment variables as needed - it redirects stdout to the OUTPUT file (like test_emacs()). This commit also declares python as an external prereq. The stdout redirection is required to avoid trouble when

RFC: don't ask users to do 'sudo make install'

: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20111207/21e4c9fe/attachment.obj>

[PATCH 1/4] notmuch-opts.[ch]: new argument parsing framework for notmuch.

On Tue, 06 Dec 2011 22:41:23 +0200, Jani Nikula wrote: > > We chatted about reserving notmuch-*.c to notmuch commands, but I guess > it was only after you sent these. I think it would make sense though. renamed to "command-line-arguments.[ch]". Hard luck for those of you on 8.3 file systems.

[PATCH 4/4] notmuch-search: convert to command-line-arguments

From: David Bremner The switch on format_sel is slightly clunky, but it doesn't seem worth special casing argument processing for function pointers, when I think the function pointer approach will be modified/abandoned. --- notmuch-search.c | 109

[PATCH 1/4] command-line-arguments.[ch]: new argument parsing framework for notmuch.

From: David Bremner As we noticed when Jani kindly converted things to getopt_long, much of the work in argument parsing in notmuch is due to the the key-value style arguments like --format=(raw|json|text). The framework here provides positional arguments, simple switches,

[PATCH 3/4] notmuch-restore: convert to command-line-arguments

From: David Bremner The new argument handling is a bit more concise, and bit more flexible. It allows the input file name to go before the --accumulate option. --- notmuch-restore.c | 37 +++-- 1 files changed, 15 insertions(+), 22

RFC: don't ask users to do 'sudo make install'

On Wed, 07 Dec 2011 13:25:45 +0100, Justus Winter <4winter at informatik.uni-hamburg.de> wrote: > Hey everyone, > > I just noticed that running configure says: > > > All required packages were found. You may now run the following > > commands to compile and install notmuch: > > > >make

RFC: don't ask users to do 'sudo make install'

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/11 14:37, Tomi Ollila wrote: > On Wed, 07 Dec 2011 13:25:45 +0100, Justus Winter > <4winter at informatik.uni-hamburg.de> > wrote: >> Hey everyone, >> >> I just noticed that running configure says: >> >>> All required packages were found.

RFC: don't ask users to do 'sudo make install'

On Wed, 07 Dec 2011 15:05:02 +0100, Rainer M Krug wrote: > Because of these problems, I use checkinstall to create a deb file ( I > am using Ubuntu) which I can then un-install again if I want to, or > update, or anything the package manager can do. Note that in recent versions of notmuch,

[PATCH v4 0/3] emacs: do not call notmuch show for non-inlinable parts

Hi David. How about pushing this series? Just a humble ping :) Regards, Dmitry

[PATCH 1/2] emacs: remove some code duplication in notmuch-show

David, this seems ready for push as well. Regards, Dmitry

[PATCH v3 0/2] notmuch hooks

On Tue, 06 Dec 2011 22:16:37 -0500, Tom Prince wrote: > On Tue, 06 Dec 2011 18:47:01 -0800, Jameson Graef Rollins finestructure.net> wrote: > > Also, what if we make it so that the post-new hook script only runs if > > notmuch new processes new messages? All of my post-new functions don't > >

[PATCH v3 0/2] notmuch hooks

H_NEW_MESSAGES containing the number of new messages to the post-new script. jamie. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/

[PATCH 2/2] python: annotate all calls into libnotmuch with types

n2.7-2.7.2/Modules/_ctypes/callproc.c:827 with stderr saying: A Xapian exception occurred opening database: Unable to get write lock on /home/teythoon/Maildir/.notmuch/xapian: already locked Justus -- next part ------ A non-text attachment was scrubbed... Name: .signature Type: application/octet-stream Size: 17 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20111207/67d23630/attachment-0001.obj>

[PATCH 2/2] python: annotate all calls into libnotmuch with types

uncought c++ exception, right? If so I think it has to be cought somewhere in libnotmuch and turned into the appropriate error code (hm, there is only the generic XAPIAN_EXCEPTION, I thought there was a way to indicate that the db has been modified?). Justus -- next part -- A non-text attachment was scrubbed... Name: .signature Type: application/octet-stream Size: 17 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20111207/37160f77/attachment.obj>

No subject

So it turns out the completely untested NOTMUCH_OPT_STRING code was (surpise!) buggy. I also fixed a few remaining formatting issues, added a unit test, and some documentation (in command-line-arguements.h). I also reordered option description struct so that the two mandatory (in the sense of

[PATCH v4 3/5] notmuch-dump: convert to command-line-arguments

From: David Bremner The output file is handled via positional arguments. There are currently no "normal" options. --- notmuch-dump.c | 31 +++ 1 files changed, 19 insertions(+), 12 deletions(-) diff --git a/notmuch-dump.c b/notmuch-dump.c index

[PATCH v4 4/5] notmuch-restore: convert to command-line-arguments

From: David Bremner The new argument handling is a bit more concise, and bit more flexible. It allows the input file name to go before the --accumulate option. --- notmuch-restore.c | 37 +++-- 1 files changed, 15 insertions(+), 22

[PATCH v4 2/5] test: tests for command-line-arguments.c

From: David Bremner This was needed because no current notmuch code exercises the NOTMUCH_OPT_STRING style arguments. --- test/Makefile.local | 11 - test/arg-test.c | 52 + test/argument-parsing | 16

[PATCH v4 5/5] notmuch-search: convert to command-line-arguments

From: David Bremner The switch on format_sel is slightly clunky, but it doesn't seem worth special casing argument processing for function pointers, when I think the function pointer approach will be modified/abandoned. --- notmuch-search.c | 106

[PATCH v4 1/5] command-line-arguments.[ch]: new argument parsing framework for notmuch.

From: David Bremner As we noticed when Jani kindly converted things to getopt_long, much of the work in argument parsing in notmuch is due to the the key-value style arguments like --format=(raw|json|text). The framework here provides positional arguments, simple switches,

[PATCH v3 0/2] notmuch hooks

On Tue, 06 Dec 2011 18:27:27 -0800, Jameson Graef Rollins wrote: > On Tue, 6 Dec 2011 15:22:36 +0200, Jani Nikula wrote: > > Hi all, this is v3 of the notmuch hooks patches. I think this is nearing > > completion apart from final review and, most notably, tests. > > Hey, Jani. Thanks so much

RFC: don't ask users to do 'sudo make install'

On Wed, 07 Dec 2011 15:54:05 +0100, Rainer M Krug wrote: > Interesting. I assume, I have to do > > ./configure > make > make debian-snapshot Just "make debian-snapshot" should work if you have the packages dpkg-dev and fakeroot installed. > dpkg-buildpackage: warning: (Use -d flag to

[PATCH v3 0/2] notmuch hooks

Quoth Jani Nikula on Dec 07 at 8:05 pm: > On Tue, 06 Dec 2011 22:16:37 -0500, Tom Prince > wrote: > > On Tue, 06 Dec 2011 18:47:01 -0800, Jameson Graef Rollins > finestructure.net> wrote: > > > Also, what if we make it so that the post-new hook script only runs if > > > notmuch new processes

[PATCH] Release memory allocated by internet_address_list_parse_string()

g_object_unref() releases the memory of the InternetAddressList object returned by internet_address_list_parse_string() -- when last (only) reference is released, internet_address_list_finalize() will do cleanup. --- When reviewing, see also gmime-2.4.25/gmime/internet-address.c (or older) I

[PATCH] Release memory allocated by internet_address_list_parse_string()

On Wed, 07 Dec 2011 22:13:19 +0200, Tomi Ollila wrote: > g_object_unref() releases the memory of the InternetAddressList object > returned by internet_address_list_parse_string() -- when last (only) > reference is released, internet_address_list_finalize() will do cleanup. > --- > When reviewing,

[PATCH 0/3] Configuration file option to exclude files/directories

On Wed, 14 Sep 2011 00:32:01 +0300, tomi.ollila at iki.fi wrote: > This patch set adds a configuration option 'database.exclude'; a list of > files/directories to be excluded when doing 'notmuch new' operation. Hi All; I could see idea of excluding spam and trash folders from indexing being

[PATCH 0/3] Configuration file option to exclude files/directories

On Wed, 07 Dec 2011 19:51:32 -0400, David Bremner wrote: > I could see idea of excluding spam and trash folders from indexing being > useful, but this has been sitting in the list for a few months without > comment. Does that mean no-one but Tomi thinks the functionality is > interesting? Or it

New test for exported symbols

On Mon, 28 Nov 2011 22:51:14 -0800, David Bremner wrote: > As long promised, here is a test to compare the list of symbols > retrieved with objdump to those from notmuch.sym > > I didn't verify whether this runs properly during out of tree builds. aidecoe did verify this for me on Gentoo, so I

[PATCH v4 0/3] emacs: do not call notmuch show for non-inlinable parts

On Wed, 30 Nov 2011 01:19:51 +0400, Dmitry Kurochkin wrote: > Changes: > > v4 since v3: > > * Use $(()) instead of expr(1) for math. > pushed d

[PATCH 1/2] emacs: remove some code duplication in notmuch-show

On Sat, 26 Nov 2011 02:23:30 +0400, Dmitry Kurochkin wrote: > Add optional props argument to `notmuch-show-get-header'. Use it to > get headers in `notmuch-show-insert-part-multipart/signed' and > `notmuch-show-insert-part-multipart/encrypted'. Pushed this series. d

[PATCH 1/2] test: date_relative in notmuch search json output

On Sat, 23 Jul 2011 10:54:03 +0100, pazz wrote: > expect the date_relative field for thread entries > in notmuch search's json output > --- I pushed an amended version of this one (explaining why the date won't change in the future), and the next patch as well. d

Patch Amnesty Day: July 1, 2011

org/pipermail/notmuch/attachments/20111207/594660de/attachment.pgp>

[PATCH] debian: add a recommends for w3m-el or w3m-el-snapshot to notmuch-emacs

From: David Bremner Installing w3m-el provides a nicer out of the box experience for viewing html in notmuch, and the overhead is not too bad (about 5M diskspace). --- debian/control |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/debian/control

RFC: don't ask users to do 'sudo make install'

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/11 15:34, David Bremner wrote: > On Wed, 07 Dec 2011 15:05:02 +0100, Rainer M Krug > wrote: > >> Because of these problems, I use checkinstall to create a deb >> file ( I am using Ubuntu) which I can then un-install again if I >> want to,

S/MIME support in notmuch

I'd like to report some success on getting S/MIME signature verification working using notmuch and the recently-released GMime 2.6. I specifically tested with notmuch-0.10.2 and gmime-2.6.1. The following changes were required: 1) notmuch: Apply patch from Redhat packaging to handle API changes