[PATCH 2/2] lib: fix error handling

2014-01-13 Thread Tomi Ollila
On Tue, Nov 19 2013, Tomi Valkeinen wrote: > Currently if a Xapian exception happens in notmuch_message_get_header, > the exception is not caught leading to crash. In > notmuch_message_get_date the exception is caught, but an internal error > is raised, again leading to crash. > > This patch fixe

[PATCH 2/3] man: partial conversion to pod.

2014-01-13 Thread Tomi Ollila
David Bremner writes: > From: David Bremner > > This allows generation of man page and info document from the same source. > It is also a bit more friendly to edit for most people. IMHO it is good idea to have common format where to produce man, info & html files. If we're going to need pandoc

[PATCH 0/2] emacs: tree: use notmuch-tag-format-tags

2014-01-13 Thread Tomi Ollila
On Sat, Jan 11 2014, Mark Walters wrote: > This is a reworking of the second patch in the series > id:1387480344-14326-1-git-send-email-markwalters1009 at gmail.com (it > applies on top of the first patch in that series which has been marked > ready) > > The first patch gives tree default matchin

[PATCH 1/1] test: leave T\d\d\d- in variable $this_test and introduce $this_test_bare

2014-01-13 Thread Tomi Ollila
Script `notmuch-test` expects the results file have T\d\d\d- part intact so the results files (and some test output files) are now named as such. The T\d\d\d- part is dropped in new variable $this_test_bare which is used in progress informational messages and when loading .el files in emacs tests

[PATCH 1/1] test: check that expected results file exists when test script exits != 0

2014-01-13 Thread Tomi Ollila
This way tests continue to run after one test script does not report success. --- I failed to notice this when MANUALLY testing the test change in question. This is quick fix and makes notmuch-test & test-lib.sh have interdependency how to modify the test script name for output file. I'll think w

[PATCH 1/1] emacs: Makefile.local: HAVE_EMACS usage fixes

2014-01-13 Thread David Bremner
Tomi Ollila writes: > If we don't have emacs, disable targets that used EMACS while doing > the recipes of that target. > pushed. d

[PATCH 1/1] build: remove trailing '/.' when doing mkdir -p .deps/.

2014-01-13 Thread David Bremner
Tomi Ollila writes: > When make variable $@ does not contain directory part, $(@D) > resolves as '.'. In this case .deps/$(@D) is '.deps/.' > In some systems `mkdir [-p] directory/.` fails. > To make this compatible with more system substitute trailing > '/.' (slashdot) with '' (empty string) whe

[PATCH 1/2] emacs: tree remove comma separator tags

2014-01-13 Thread David Bremner
Mark Walters writes: > Previously the tags on each line in tree view were separarted by ", " > not just " ". This is different from show and search views. pushed. d

[PATCH 1/2] test: basic: drop 'ensure all available tests are run'

2014-01-13 Thread David Bremner
Tomi Ollila writes: > When naming test scripts in format 'T\d\d\d-name.sh' the list of > tests to run are created dynamically. This makes test series pushed. d

[PATCH] cli: initialize quiet variable in compact

2014-01-13 Thread David Bremner
Jani Nikula writes: > Surprisingly there's no compiler warning! pushed d

[PATCH] test: add test for syntax of emacs test library

2014-01-13 Thread David Bremner
David Bremner writes: > If there is a syntax error in the emacs test library, it causes other > tests to hang or crash without a useful error message. > > This test could be eliminated if the error reporting for emacs tests > was somehow improved. pushed, and git even figured out what file to pa

Re: [PATCH 2/2] lib: fix error handling

2014-01-13 Thread Tomi Ollila
On Tue, Nov 19 2013, Tomi Valkeinen wrote: > Currently if a Xapian exception happens in notmuch_message_get_header, > the exception is not caught leading to crash. In > notmuch_message_get_date the exception is caught, but an internal error > is raised, again leading to crash. > > This patch fixe

Re: [PATCH 2/3] man: partial conversion to pod.

2014-01-13 Thread Tomi Ollila
David Bremner writes: > From: David Bremner > > This allows generation of man page and info document from the same source. > It is also a bit more friendly to edit for most people. IMHO it is good idea to have common format where to produce man, info & html files. If we're going to need pandoc

Re: [PATCH 0/2] emacs: tree: use notmuch-tag-format-tags

2014-01-13 Thread Tomi Ollila
On Sat, Jan 11 2014, Mark Walters wrote: > This is a reworking of the second patch in the series > id:1387480344-14326-1-git-send-email-markwalters1...@gmail.com (it > applies on top of the first patch in that series which has been marked > ready) > > The first patch gives tree default matching/n

[PATCH 1/1] test: leave T\d\d\d- in variable $this_test and introduce $this_test_bare

2014-01-13 Thread Tomi Ollila
Script `notmuch-test` expects the results file have T\d\d\d- part intact so the results files (and some test output files) are now named as such. The T\d\d\d- part is dropped in new variable $this_test_bare which is used in progress informational messages and when loading .el files in emacs tests

[PATCH 1/1] test: check that expected results file exists when test script exits != 0

2014-01-13 Thread Tomi Ollila
This way tests continue to run after one test script does not report success. --- I failed to notice this when MANUALLY testing the test change in question. This is quick fix and makes notmuch-test & test-lib.sh have interdependency how to modify the test script name for output file. I'll think w

Re: [PATCH] cli: initialize quiet variable in compact

2014-01-13 Thread David Bremner
Jani Nikula writes: > Surprisingly there's no compiler warning! pushed d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

Re: [PATCH 1/1] emacs: Makefile.local: HAVE_EMACS usage fixes

2014-01-13 Thread David Bremner
Tomi Ollila writes: > If we don't have emacs, disable targets that used EMACS while doing > the recipes of that target. > pushed. d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

Re: [PATCH 1/2] test: basic: drop 'ensure all available tests are run'

2014-01-13 Thread David Bremner
Tomi Ollila writes: > When naming test scripts in format 'T\d\d\d-name.sh' the list of > tests to run are created dynamically. This makes test series pushed. d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/no

Re: [PATCH 1/2] emacs: tree remove comma separator tags

2014-01-13 Thread David Bremner
Mark Walters writes: > Previously the tags on each line in tree view were separarted by ", " > not just " ". This is different from show and search views. pushed. d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listin

Re: [PATCH 1/1] build: remove trailing '/.' when doing mkdir -p .deps/.

2014-01-13 Thread David Bremner
Tomi Ollila writes: > When make variable $@ does not contain directory part, $(@D) > resolves as '.'. In this case .deps/$(@D) is '.deps/.' > In some systems `mkdir [-p] directory/.` fails. > To make this compatible with more system substitute trailing > '/.' (slashdot) with '' (empty string) whe

Re: [PATCH] test: add test for syntax of emacs test library

2014-01-13 Thread David Bremner
David Bremner writes: > If there is a syntax error in the emacs test library, it causes other > tests to hang or crash without a useful error message. > > This test could be eliminated if the error reporting for emacs tests > was somehow improved. pushed, and git even figured out what file to pa

[PATCH] show: add In-reply-to, References fields to structured formats

2014-01-13 Thread Peter Wang
On Sun, 12 Jan 2014 17:31:32 +0200, Jani Nikula wrote: > On Sun, 12 Jan 2014, Peter Wang wrote: > > This is useful when 'show' is used to retrieve a draft message > > which is in reply to another message. > > I'd like to know more about *how* this is useful. Indeed the whole big > picture about