[PATCH 1/2] cli/insert: delay database open until after writing mail file

2016-11-28 Thread David Bremner
The idea is to get the mail written to disk, even if we can't open the database (e.g. because some other process has a write lock, and notmuch is compiled for non-blocking opens). --- notmuch-insert.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git

[PATCH 2/2] cli/insert: return EX_TEMPFAIL for some errors

2016-11-28 Thread David Bremner
Attempt to distinguish between errors indicating misconfiguration or programmer error, which we consider "permanant", in the sense that automatic retries are unlikely to be useful, and those indicating transient error conditions. We consider XAPIAN_EXCEPTION transient because it covers the

Re: [PATCH 1/2] cli/insert: delay database open until after writing mail file

2016-11-28 Thread David Bremner
David Bremner writes: > The idea is to get the mail written to disk, even if we can't open the > database (e.g. because some other process has a write lock, and notmuch > is compiled for non-blocking opens). > --- > notmuch-insert.c | 14 +++--- > 1 file changed, 7

Re: Trouble decrypting multipart/encrypted

2016-11-28 Thread David Bremner
Eric Skoglund writes: > Hello, I've recently started using notmuch (together with emacs) and > have now encountered some problems when trying to decrypt a > multipart/encrypted message. Is it just this one message, or all multipart/encrypted messages? > > Signing and

Re: [PATCH 2/2] cli/insert: return EX_TEMPFAIL for some errors

2016-11-28 Thread David Bremner
David Bremner writes: > Attempt to distinguish between errors indicating misconfiguration or > programmer error, which we consider "permanant", in the sense that > automatic retries are unlikely to be useful, and those indicating > transient error conditions. We consider

Re: Trouble decrypting multipart/encrypted

2016-11-28 Thread Eric Skoglund
David Bremner writes: > Eric Skoglund writes: > >> Hello, I've recently started using notmuch (together with emacs) and >> have now encountered some problems when trying to decrypt a >> multipart/encrypted message. > > Is it just this one message, or all

[PATCH] test: gdb insert: redirect input inside gdb script

2016-11-28 Thread Tomi Ollila
Running `gdb command < input` is not as reliable way to give input to the command (some installations of gdb consume it). Use "set args" gdb command to have input redirected at gdb 'run' time. --- With this change, these tests work on FreeBSD-11.0-RELEASE-amd64[.qcow2] when adding gdb 7.11.1

Re: Trouble decrypting multipart/encrypted

2016-11-28 Thread David Bremner
Eric Skoglund writes: >>> >>> Signing and encrypting email via C-c C-m s p and C-c C-m c p works >>> fine. And given that it doesn't work outside of emacs I don't think it >>> is an emacs problem. I'm also able to decrypt the message via the >>> command line using just gpg. >>

[PATCH 2/3] cli/insert: delay database open until after writing mail file

2016-11-28 Thread David Bremner
The idea is to get the mail written to disk, even if we can't open the database (e.g. because some other process has a write lock, and notmuch is compiled for non-blocking opens). --- notmuch-insert.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git

v2 of insert tempfail series

2016-11-28 Thread David Bremner
This incorporates Tomi's patch of id:1480367228-22183-1-git-send-email-tomi.oll...@iki.fi verbatim, to sort out conflicts. It fixes the issues I alread sent mail about, and puts back the --keep tests for various error codes. The interdiff follows; most of it is Tomi's fault :). diff

[PATCH 1/3] test: gdb insert: redirect input inside gdb script

2016-11-28 Thread David Bremner
From: Tomi Ollila Running `gdb command < input` is not as reliable way to give input to the command (some installations of gdb consume it). Use "set args" gdb command to have input redirected at gdb 'run' time. --- test/T070-insert.sh | 10 ++ 1 file changed, 6

[PATCH 3/3] cli/insert: return EX_TEMPFAIL for some errors

2016-11-28 Thread David Bremner
Attempt to distinguish between errors indicating misconfiguration or programmer error, which we consider "permanent", in the sense that automatic retries are unlikely to be useful, and those indicating transient error conditions. We consider XAPIAN_EXCEPTION transient because it covers the

Re: [PATCH v3] build: install notmuch-emacs-mua.desktop file with emacs

2016-11-28 Thread David Bremner
Jani Nikula writes: > There is really no need to have a separate install target for the > desktop file. Just install the desktop file with emacs, with a > configure option to opt out. > > --- > > v3: check for desktop-file-install in configure. > pushed to master. d

Re: [PATCH] doc: whitespace cleanup

2016-11-28 Thread David Bremner
Tomi Ollila writes: > In most part, our .rst documents are indented with 8 spaces instead > of tabs. Bring the rest of the lines to the same format. > Also, on one (supposedly empty) line, trailing spaces were removed. pushed to master d

Re: [PATCH v3] build: update the desktop database after installing the desktop file

2016-11-28 Thread David Bremner
Jani Nikula writes: > This makes the option to choose Notmuch as mailto: handler show up in > the desktop environment settings. Ignore errors. pushed to master. d ___ notmuch mailing list notmuch@notmuchmail.org

Re: [PATCH v2] test: check the handling of files vanishing between scandir and indexing

2016-11-28 Thread David Bremner
Jani Nikula writes: > +gdb --batch-silent --return-child-result -x notmuch-new-vanish.gdb \ > +--args notmuch new 2>OUTPUT 1>/dev/null I wonder if Tomi's suggestion of id:20161128221231.25528-2-da...@tethera.net applies here as well. In this case it is redirecting

Re: [PATCH v2] test: check the handling of files vanishing between scandir and indexing

2016-11-28 Thread Tomi Ollila
On Tue, Nov 29 2016, David Bremner wrote: > Jani Nikula writes: > >> +gdb --batch-silent --return-child-result -x notmuch-new-vanish.gdb \ >> +--args notmuch new 2>OUTPUT 1>/dev/null > > I wonder if Tomi's suggestion of > >

Re: Trouble decrypting multipart/encrypted

2016-11-28 Thread Eric Skoglund
David Bremner writes: > > Yeah, --verify only works with one of the structured output formats > (json or sexp). I suppose that's probably a documentation bug. > > try > > % notmuch show --format=json --verify $message Yes that did work, getting "sigstatus : [ { 'status':