Re: [PATCH 3/3] test: implement and document NOTMUCH_TEST_QUIET variable usage

2013-11-13 Thread Tomi Ollila
On Wed, Nov 13 2013, Austin Clements amdra...@mit.edu wrote: Quoth Tomi Ollila on Nov 12 at 10:41 pm: return fi test_success=$(($test_success + 1)) +if test -n $NOTMUCH_TEST_QUIET; then +return 0 +fi say_color pass %-6s PASS echo

Re: [PATCH] Accumulated news for Austin's changes

2013-11-13 Thread David Bremner
Austin Clements amdra...@mit.edu writes: --- NEWS | 52 1 file changed, 52 insertions(+) pushed. Other NEWS patches welcome, hint, hint. d ___ notmuch mailing list notmuch@notmuchmail.org

[PATCH] remove notmuch-pick

2013-11-13 Thread Mark Walters
--- Remove all the notmuch-pick contrib directory. Everything relevant is in mainline. Best wishes Mark contrib/notmuch-pick/README | 43 -- contrib/notmuch-pick/TODO | 29 --- contrib/notmuch-pick/run-tests.sh | 46

[PATCH v2 2/5] compact: catch Xapian::Error consistently

2013-11-13 Thread Tomi Ollila
catch Xapian::Error in compact code in lib/database.cc to be consistent with other code in addition to not making software crash on uncaught other Xapian error. --- lib/database.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/database.cc b/lib/database.cc index

compactor adjustments v2

2013-11-13 Thread Tomi Ollila
This is v2 of id:1384192538-15291-1-git-send-email-tomi.oll...@iki.fi Changes include: tidy formatting (i.e. uncrustifying) on related code catching Xapian::Error (always) The tests done in id:m2fvr1tpkf@guru.guru-group.fi were redone with one extra (*) -- results same. Automatic

[PATCH v2 4/5] compact: unconditionally attempt to remove old wip database compact directory

2013-11-13 Thread Tomi Ollila
In case previous notmuch compact has been interrupted there is old work-in-progress database compact directory partially filled. Remove it just before starting to fill the directory with new files. --- lib/database.cc | 5 + 1 file changed, 5 insertions(+) diff --git a/lib/database.cc

[PATCH v2 5/5] compact: provide user more information on after-compaction failures

2013-11-13 Thread Tomi Ollila
After database has been compacted, there are steps to put the new database into place -- and these steps may fail. In case such failure happens, provide better information how to resolve it. Thanks to Ben Gamari for most of the information content. --- lib/database.cc | 39

Re: [PATCH] notmuch: Add maildir: search option

2013-11-13 Thread Peter Zijlstra
On Tue, Nov 12, 2013 at 02:39:52PM -0500, Austin Clements wrote: On Tue, 12 Nov 2013, Austin Clements acleme...@csail.mit.edu wrote: I think this is a great idea. Personally I think this is how folder: should work. I find the semantics of folder: to be useless except where they happen to

Re: [PATCH] notmuch: Add maildir: search option

2013-11-13 Thread Peter Zijlstra
On Wed, Nov 13, 2013 at 09:08:52PM +0100, Peter Zijlstra wrote: +if (i = 3) { + /* Consume trailing maildir directory entries */ + if (STRNCMP_LITERAL (maildir, cur) == 0 || + STRNCMP_LITERAL (maildir, new) == 0) maildir + i - 3 obviously..

Re: [PATCH] News for tree-view

2013-11-13 Thread Tomi Ollila
On Wed, Nov 13 2013, Mark Walters markwalters1...@gmail.com wrote: --- This is news for notmuch-tree. I am not sure if it is appropriate to have this big a section but I didn't see anywhere else obvious to put the information. I am also not certain about what markup to be using for NEWS so

Re: [PATCH] emacs: show: stop stderr appearing in buffer

2013-11-13 Thread Tomi Ollila
On Wed, Nov 13 2013, Mark Walters markwalters1...@gmail.com wrote: In emacs 24.3+ the stdout/stderr from externally displaying an attachment gets inserted into the show buffer. This is caused by changes in mm-display-external in mm-decode.el. Ideally, we would put this output in the notmuch

Re: [PATCH] emacs: Correct documentation of `notmuch-poll-script'

2013-11-13 Thread David Bremner
Austin Clements amdra...@mit.edu writes: The functions referred to in the documentation for this variable were replaced by the unified `notmuch-poll-and-refresh-this-buffer' in 21474f0e. Update the documentation to reflect the new function. pushed, d

Re: [PATCH] remove notmuch-pick

2013-11-13 Thread David Bremner
Mark Walters markwalters1...@gmail.com writes: --- Remove all the notmuch-pick contrib directory. Everything relevant is in mainline. pushed d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

alot: can't read sent emails, after encryption

2013-11-13 Thread Ruben Pollan
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=- Nos vamos a Croatan. -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: signature URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20131113/b2f2a9f7/attachment.pgp>

alot: can't read sent emails, after encryption

2013-11-13 Thread apman...@idaaas.com
Quoting Ruben Pollan (2013-11-13 00:01:21) > > Same problem here, I think what is stored is the email that was send. It will > be > great if alot stores it as you say with your own public gpg key. > > I think we should open an issue about in the alot github tracker: >

[PATCH 3/3] test: implement and document NOTMUCH_TEST_QUIET variable usage

2013-11-13 Thread Tomi Ollila
On Wed, Nov 13 2013, Austin Clements wrote: > Quoth Tomi Ollila on Nov 12 at 10:41 pm: >> return >> fi >> test_success=$(($test_success + 1)) >> +if test -n "$NOTMUCH_TEST_QUIET"; then >> +return 0 >> +fi >> say_color pass "%-6s" "PASS" >>

[PATCH] emacs: show: stop stderr appearing in buffer

2013-11-13 Thread Mark Walters
In emacs 24.3+ the stdout/stderr from externally displaying an attachment gets inserted into the show buffer. This is caused by changes in mm-display-external in mm-decode.el. Ideally, we would put this output in the notmuch errors buffer but the handler is called asynchronously so we don't know

[PATCH] Accumulated news for Austin's changes

2013-11-13 Thread David Bremner
Austin Clements writes: > --- > NEWS | 52 > 1 file changed, 52 insertions(+) pushed. Other NEWS patches welcome, hint, hint. d

[PATCH] News for tree-view

2013-11-13 Thread Mark Walters
--- This is news for notmuch-tree. I am not sure if it is appropriate to have this big a section but I didn't see anywhere else obvious to put the information. I am also not certain about what markup to be using for NEWS so that is just a best guess. Best wishes Mark NEWS | 30

[PATCH] remove notmuch-pick

2013-11-13 Thread Mark Walters
--- Remove all the notmuch-pick contrib directory. Everything relevant is in mainline. Best wishes Mark contrib/notmuch-pick/README | 43 -- contrib/notmuch-pick/TODO | 29 --- contrib/notmuch-pick/run-tests.sh | 46

[PATCH v2 3/5] compact: preserve backup database until compacted database is in place

2013-11-13 Thread Tomi Ollila
It is less error prone and window of failure opportunity is smaller if the old (backup) database is always renamed (instead of sometimes rmtree'd) before new (compacted) database is put into its place. Finally rmtree() old database in case old database backup is not kept. --- lib/database.cc | 42

[PATCH v2 2/5] compact: catch Xapian::Error consistently

2013-11-13 Thread Tomi Ollila
catch Xapian::Error in compact code in lib/database.cc to be consistent with other code in addition to not making software crash on uncaught other Xapian error. --- lib/database.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/database.cc b/lib/database.cc index

[PATCH v2 4/5] compact: unconditionally attempt to remove old wip database compact directory

2013-11-13 Thread Tomi Ollila
In case previous notmuch compact has been interrupted there is old work-in-progress database compact directory partially filled. Remove it just before starting to fill the directory with new files. --- lib/database.cc | 5 + 1 file changed, 5 insertions(+) diff --git a/lib/database.cc

compactor adjustments v2

2013-11-13 Thread Tomi Ollila
This is v2 of id:1384192538-15291-1-git-send-email-tomi.ollila at iki.fi Changes include: tidy formatting (i.e. "uncrustifying") on related code catching Xapian::Error (always) The tests done in id:m2fvr1tpkf.fsf at guru.guru-group.fi were redone with one extra (*) -- results same.

[PATCH v2 1/5] compact: tidy formatting

2013-11-13 Thread Tomi Ollila
Notmuch compact code whitespace changes to match devel/STYLE. --- lib/database.cc | 62 --- notmuch-compact.c | 4 ++-- 2 files changed, 34 insertions(+), 32 deletions(-) diff --git a/lib/database.cc b/lib/database.cc index a021bf1..3c008d6

[PATCH v2 5/5] compact: provide user more information on after-compaction failures

2013-11-13 Thread Tomi Ollila
After database has been compacted, there are steps to put the new database into place -- and these steps may fail. In case such failure happens, provide better information how to resolve it. Thanks to Ben Gamari for most of the information content. --- lib/database.cc | 39

[PATCH] notmuch: Add "maildir:" search option

2013-11-13 Thread Peter Zijlstra
On Tue, Nov 12, 2013 at 02:39:52PM -0500, Austin Clements wrote: > On Tue, 12 Nov 2013, Austin Clements wrote: > > I think this is a great idea. Personally I think this is how folder: > > should work. I find the semantics of folder: to be useless except where > > they happen to coincide with

[PATCH] notmuch: Add "maildir:" search option

2013-11-13 Thread Peter Zijlstra
On Wed, Nov 13, 2013 at 09:08:52PM +0100, Peter Zijlstra wrote: > +if (i >= 3) { > + /* Consume trailing maildir directory entries */ > + if (STRNCMP_LITERAL (maildir, "cur") == 0 || > + STRNCMP_LITERAL (maildir, "new") == 0) maildir + i - 3 obviously..

[PATCH] News for tree-view

2013-11-13 Thread Tomi Ollila
On Wed, Nov 13 2013, Mark Walters wrote: > --- > This is news for notmuch-tree. I am not sure if it is appropriate to > have this big a section but I didn't see anywhere else obvious to put > the information. > > I am also not certain about what markup to be using for NEWS so that > is just a

[PATCH] emacs: show: stop stderr appearing in buffer

2013-11-13 Thread Tomi Ollila
On Wed, Nov 13 2013, Mark Walters wrote: > In emacs 24.3+ the stdout/stderr from externally displaying an > attachment gets inserted into the show buffer. This is caused by > changes in mm-display-external in mm-decode.el. > > Ideally, we would put this output in the notmuch errors buffer but

[PATCH v4 0/7] emacs: help: remap keybindings

2013-11-13 Thread David Bremner
Mark Walters writes: > This is v4 of this patch set. v3 is at > id:1383932419-12533-1-git-send-email-markwalters1009 at gmail.com. > > David found a bug in v3 which I tracked down to some match-data being > over-written. This included the bugfix >

[PATCH] emacs: Correct documentation of `notmuch-poll-script'

2013-11-13 Thread David Bremner
Austin Clements writes: > The functions referred to in the documentation for this variable were > replaced by the unified `notmuch-poll-and-refresh-this-buffer' in > 21474f0e. Update the documentation to reflect the new function. pushed, d

[PATCH 1/1] NEWS: emacs: fix notmuch-mua-reply point placement...

2013-11-13 Thread David Bremner
Tomi Ollila writes: > News for commit 4ceeaf8038ff241f438ad2a5d0bbda1c78aee385 pushed, with a minor edit for grammar/style d

[PATCH] remove notmuch-pick

2013-11-13 Thread David Bremner
Mark Walters writes: > --- > Remove all the notmuch-pick contrib directory. Everything relevant is > in mainline. > pushed d