[PATCH v5] devel/emacs: add devel/try-emacs-mua

2016-01-04 Thread Tomi Ollila
-emacs-mua @@ -0,0 +1,156 @@ +#!/bin/sh +:; set -x; exec "${EMACS:-emacs}" --debug-init --load "$0" "$@"; exit +;; +;; Try the notmuch emacs client located in ../notmuch/emacs directory +;; +;; Run this without arguments; emacs window opens with some usage informatio

Re: Two Questions

2016-01-04 Thread Tomi Ollila
On Tue, Jan 05 2016, MaDhAt2r wrote: > Hello All! > > I have been using notmuch/emacs for about two years now. I of course > love the setup, but there have always been two main issues I have yet to > find a solution for. I am hoping someone on here has found some answers. >

Re: [PATCH] cli/insert: do not lose the SMTP envelope

2016-01-02 Thread Tomi Ollila
On Fri, Jan 01 2016, J Farkas wrote: > From: Janos Farkas > Subject: [PATCH] cli/insert: do not lose the SMTP envelope > > Make sure we store the envelope sender/recipient if provided by > qmail-command(8) in $RPLINE and

Re: notmuch-message-mode initialization follow-up

2016-01-02 Thread Tomi Ollila
On Sat, Jan 02 2016, Michal Sojka wrote: > Hi, > > the first patch fixes the problem reported in > id:87wprt2r4f.fsf@zancas.localnet. Patch 2/3 is just refactoring. > Patch 3/3 is reaction to id:87y4cc3qse.fsf@zancas.localnet; I'm not > sure whether it addresses exactly what

Re: [PATCH 3/5] nmbug-status: Add an nmbug-status(5) man page

2016-01-01 Thread Tomi Ollila
On Thu, Dec 31 2015, David Bremner wrote: > Jani Nikula writes: > >> On Wed, 30 Dec 2015, "W. Trevor King" wrote: >>> To describe the config file format, so folks don't have to dig through >>> NEWS or the nmbug-status source to get that

Re: [PATCH 0/2] nmbug-status: h1 title and unbalanced fixups to default templates

2016-01-01 Thread Tomi Ollila
On Thu, Dec 31 2015, "W. Trevor King" <wk...@tremily.us> wrote: > On Thu, Dec 31, 2015 at 03:46:50PM +0200, Tomi Ollila wrote: >> This series LGTM. (html) tidy complains about imo irrelevant things >> -- or I just did not know how to use it correctly -- as `|

Re: [PATCH 0/2] nmbug-status: h1 title and unbalanced fixups to default templates

2016-01-01 Thread Tomi Ollila
On Fri, Jan 01 2016, "W. Trevor King" <wk...@tremily.us> wrote: > On Fri, Jan 01, 2016 at 01:30:57PM +0200, Tomi Ollila wrote: >> yes, I have old tidy (20091203), and the complaints were about >> utf8-characters (being invalid) in output and > > UTF-

Re: [PATCH 0/2] nmbug-status: h1 title and unbalanced fixups to default templates

2015-12-31 Thread Tomi Ollila
On Wed, Dec 30 2015, "W. Trevor King" wrote: > Polishing the templates a bit. Details in the individual patches. This series LGTM. (html) tidy complains about imo irrelevant things -- or I just did not know how to use it correctly -- as `| tidy -eq`. > > Cheers, > Trevor

Re: [PATCH] test: Always use paths without symlinks

2015-12-29 Thread Tomi Ollila
On Tue, Dec 29 2015, Michal Sojka wrote: > When notmuch sources are at a symlinked path, some tests fail because > one part of the test uses physical path and another uses logical > path (with symlinks). For example the following test output is > produced when the test is

Re: [PATCH v2] test: Unset ALTERNATE_EDITOR before running emacsclient

2015-12-29 Thread Tomi Ollila
On Tue, Dec 29 2015, Michal Sojka wrote: > ALTERNATE_EDITOR causes emacsclient to run an alternate editor if the > emacs server is not ready. This can collide with intended > functionality in test-lib.sh. > > If the ALTERNATE_EDITOR is set but empty, emacsclient runs emacs >

Re: [PATCH] test: Unset ALTERNATE_EDITOR before running emacsclient

2015-12-28 Thread Tomi Ollila
On Mon, Dec 28 2015, Michal Sojka wrote: > ALTERNATE_EDITOR causes emacsclient to run an alternate editor if the > emacs server is not ready. This can collide with intended > functionality in test-lib.sh. > > If the ALTERNATE_EDITOR is set but empty, emacsclient runs emacs >

[PATCH v4] devel/emacs: add devel/try-emacs-mua

2015-12-20 Thread Tomi Ollila
s client located in ../notmuch/emacs directory +;; +;; Run this without arguments; emacs window opens with some usage information +;; +;; Authors: Tomi Ollila <tomi.oll...@iki.fi> +;; +;; http://www.emacswiki.org/emacs/EmacsScripts was a useful starting point... +;; +;; Licence: GPLv3+ +;; +

Re: Compilation failure with gcc 4.4.7

2015-12-19 Thread Tomi Ollila
On Sat, Dec 19 2015, Justus Winter <4win...@informatik.uni-hamburg.de> wrote: > Hello, > > I just tried to compile notmuch with an oldish compiler, and that > failed (if the compiler is indeed too old, then at least the error > message should be more helpful): see thread

Re: [Patch v3 4/8] test: initial tests for S/MIME and notmuch-emacs

2015-12-14 Thread Tomi Ollila
On Mon, Dec 14 2015, David Bremner wrote: > David Bremner writes: > >> Test the ability of notmuch-mua-mail to send S/MIME signed (and >> encrypted) messages; this really relies on existing functionality in >> message-mode. >> >> The generated keys and

Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t

2015-12-13 Thread Tomi Ollila
On Sun, Dec 13 2015, Tomi Ollila <tomi.oll...@iki.fi> wrote: > On Sun, Dec 13 2015, David Bremner <da...@tethera.net> wrote: > >> >> Without weighing in on the advisibility of searching for gpg in $PATH, >> there is a glib function g_find_program_in_path. We're

Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t

2015-12-13 Thread Tomi Ollila
On Sun, Dec 13 2015, David Bremner <da...@tethera.net> wrote: > Daniel Kahn Gillmor <d...@fifthhorseman.net> writes: > >> On Fri 2015-12-11 17:02:33 -0500, Tomi Ollila wrote: >>> The above code finds gpg/gpg2 (when called w/ these args) from >>> _CS_

Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t

2015-12-12 Thread Tomi Ollila
On Sat, Dec 12 2015, Daniel Kahn Gillmor <d...@fifthhorseman.net> wrote: > On Fri 2015-12-11 17:02:33 -0500, Tomi Ollila wrote: >> The above code finds gpg/gpg2 (when called w/ these args) from >> _CS_PATH (seems to be /bin:/usr/bin by default in linux (tried to >>

Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t

2015-12-11 Thread Tomi Ollila
On Thu, Dec 10 2015, Daniel Kahn Gillmor wrote: > Exposing this to the user of the library lets the user point to > arbitrary gpg executables when trying to decrypt. > --- > lib/database-private.h | 3 ++ > lib/database.cc| 93 >

Re: [PATCH 2/9] Add a lazily-initialized crypto context to notmuch_database_t

2015-12-11 Thread Tomi Ollila
On Thu, Dec 10 2015, Daniel Kahn Gillmor wrote: > This is in large part a duplicate of parts of crypto.c, but that code > is in the client (outside the library), and we don't want to entangle > the libgmime API with the libnotmuch API. > > I welcome better proposals for

Re: [PATCH 6/9] search for a reasonable gpg implementation

2015-12-11 Thread Tomi Ollila
On Thu, Dec 10 2015, Daniel Kahn Gillmor wrote: > When the notmuch database needs to find gpg, have it search reasonable > paths first. > --- > lib/database.cc | 13 - > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/lib/database.cc

Re: [PATCH 1/9] reorganize indexing of multipart/signed and multipart/encrypted

2015-12-11 Thread Tomi Ollila
On Thu, Dec 10 2015, Daniel Kahn Gillmor wrote: > This prepares the codebase for a cleaner changeset for dealing with > indexing some encrypted messages in the clear. > --- > lib/index.cc | 38 ++ > 1 file changed, 18 insertions(+), 20

Re: allow indexing cleartext of encrypted messages

2015-12-11 Thread Tomi Ollila
On Fri, Dec 11 2015, Daniel Kahn Gillmor wrote: > On Wed 2015-12-09 22:39:37 -0500, Daniel Kahn Gillmor wrote: >> * the libnotmuch API is extended with >>notmuch_database_add_message_try_decrypt(). This should probably >>ultimately be more general, because there

Re: [PATCH] lib: have two definitions of NOTMUCH_DEPRECATED macro

2015-12-08 Thread Tomi Ollila
On Tue, Dec 08 2015, David Bremner <da...@tethera.net> wrote: > Tomi Ollila <tomi.oll...@iki.fi> writes: > >> Some compilers (older than gcc 4.5 and clang 2.9) do support >> __attribute__ ((deprecated)) but not >> __attribute__ ((deprecated("message"

Re: [PATCH] configure: drop use of "pkg-config emacs"

2015-12-06 Thread Tomi Ollila
On Sun, Dec 06 2015, David Bremner wrote: > This does not play well with --prefix. As Tomi notes in > id:m2k2p2rwth@guru.guru-group.fi, people still have the option of e.g. > > % ./configure ---emacslispdir=`pkg-config emacs --variable sitepkglispdir` > --- > oops, I

Re: Query to get emails from notmuch mailing list?

2015-12-04 Thread Tomi Ollila
On Fri, Dec 04 2015, Xu Wang <xuwang...@gmail.com> wrote: > On Thu, Dec 3, 2015 at 3:41 PM, Tomi Ollila <tomi.oll...@iki.fi> wrote: >> On Thu, Dec 03 2015, David Bremner <da...@tethera.net> wrote: >> >>> Damien Cassou <dam...@cassou.me> writes: &

Re: notmuch-reply doesn't use Reply-To

2015-12-04 Thread Tomi Ollila
On Fri, Dec 04 2015, David Bremner wrote: > Damien Cassou writes: > >>"To" : "r...@inria.fr", >>"Reply-To" : "r...@inria.fr", >>"From" : "seas...@rmod.inria.fr", >>"Subject" :

Re: Query to get emails from notmuch mailing list?

2015-12-03 Thread Tomi Ollila
On Thu, Dec 03 2015, David Bremner wrote: > Damien Cassou writes: > >> Hi, >> >> how do you write a notmuch query to get all emails of the notmuch >> mailing list? >> > > at one point I use > > notmuch tag +inbox::notmuch tag:inbox and '(' to:notmuch or

Re: [PATCH] configure: Use $prefix for emacs, even when pkg-config is available.

2015-11-28 Thread Tomi Ollila
On Sat, Nov 28 2015, David Bremner wrote: > Andrew Burgess writes: > >> Assumming that you have pkg-config installed, emacs installed, and the >> command 'pkg-config emacs --variable sitepkglispdir' returns a >> directory that only root can write

Re: [PATCH] configure: Use $prefix for emacs, even when pkg-config is available.

2015-11-26 Thread Tomi Ollila
On Wed, Nov 25 2015, Andrew Burgess wrote: > Hi, > > I like to maintain multiple copies of notmuch installed in parallel, > and so make use of the --prefix=$PREFIX argument to configure. > > I recently tried to configure and install from master, and ran into an >

Re: Question re undo

2015-11-21 Thread Tomi Ollila
On Sat, Nov 21 2015, Bart Bunting wrote: > Hi there, > > I've accidentally removed a bunch of tags in error and was dreaming > there may be a way to fix it :). > > Is there any way to restore tags as they were at a certain point in > time? some examination of lastmod:

Re: github mirror

2015-11-21 Thread Tomi Ollila
Exceptionally top-posting as the rest is left just for reference (for anyone interested and loosing the thread context) Marking this as notmuch::fixed as https://github.com/notmuch/notmuch fixes this and is usually up-to-date (I use this link on one of my notmuch installations) Interestingly

[PATCH v3] devel/emacs: add devel/try-emacs-mua.sh

2015-11-13 Thread Tomi Ollila
devel/try-emacs-mua.sh provides an easy way to try and experiment with the notmuch emacs client provided in emacs subdirectory of notmuch source tree. User is required to choose whether to run emacs with -q, -Q or neither -- and experienced ones may add other command line options, like '-f

Re: [PATCH] devel/emacs: add devel/try-emacs-mua.sh

2015-11-12 Thread Tomi Ollila
On Thu, Nov 12 2015, Mark Walters wrote: >> devel/try-emacs-mua.sh provides an easy way to try and experiment with >> the notmuch emacs client provided in emacs subdirectory of notmuch >> source tree. >> >> User is required to choose whether to run emacs with -q, -Q or

Re: [PATCH] emacs: Fix regression in (notmuch-)message-mode initialization

2015-11-09 Thread Tomi Ollila
On Mon, Nov 09 2015, Michal Sojka wrote: > On Mon, Nov 09 2015, David Bremner wrote: >> Michal Sojka writes: >> >>> This commit uses advice mechanism to call notmuch-message-mode instead >>> of message-mode. This way, a call to message-mail initializes

Re: multiple directories for path?

2015-11-07 Thread Tomi Ollila
On Sat, Nov 07 2015, Julien Cubizolles wrote: > I've my mail in two different directories: a Maildir sync by offlineimap > and nnml mail spools. How can I have notmuch index both ? The path entry > in .notmuch-config doesn't seem to allow multiple values. You can make

Re: weird paths in notmuch database

2015-11-06 Thread Tomi Ollila
On Fri, Nov 06 2015, Davide Mancusi wrote: > 2015-11-06 14:09 GMT+01:00 David Bremner : >>> $ notmuch search --output=files 'path:/home/**' >>> /export/home/username/mail//home/username/local/mail/archive/cur/1446805732.2711_9.hostname:2,S >>>

Re: weird paths in notmuch database

2015-11-06 Thread Tomi Ollila
On Fri, Nov 06 2015, Davide Mancusi <are...@yahoo.it> wrote: > 2015-11-06 16:14 GMT+01:00 Tomi Ollila <tomi.oll...@iki.fi>: >> How is the symlink exactly shown in ls -l /path/to/symlink output ? > > $ ls -l /home/username/local > lrwxrwxrwx 1 username group 21 Jan

Re: [PATCH] emacs: Fontify From with message-header-other

2015-11-06 Thread Tomi Ollila
On Thu, Nov 05 2015, "H. J. Illikainen" wrote: > Commit e26d767897e8ab59f05808a12ac5c9c1e3c7030f changed the > fontification of the body associated with the From header to > message-header-from. However, that face is non-existent, and in > message.el

Re: weird paths in notmuch database

2015-11-06 Thread Tomi Ollila
On Fri, Nov 06 2015, Davide Mancusi <are...@yahoo.it> wrote: > 2015-11-06 16:24 GMT+01:00 Tomi Ollila <tomi.oll...@iki.fi>: >> And notmuch config get database.path prints /export/home/username/mail >> I presume ? > > Correct. > >> If that is the ca

[PATCH] devel/emacs: add devel/try-emacs-mua.sh

2015-11-01 Thread Tomi Ollila
devel/try-emacs-mua.sh provides an easy way to try and experiment with the notmuch emacs client provided in emacs subdirectory of notmuch source tree. User is required to choose whether to run emacs with -q, -Q or neither -- and experienced ones may add other command line options, like '-f

[PATCH v3] devel/release-checks: added checking of copyright year in documentation

2015-11-01 Thread Tomi Ollila
Check that copyright year will be current year in generated documentation. Checking is done my matching that copyright line contains current year as a substring which is good enough "approximation" in this context. --- v3 of id:1440616236-17866-1-git-send-email-tomi.oll...@iki.fi changed $year

Re: [PATCH] emacs: poll: return useful errors when poll fails.

2015-10-28 Thread Tomi Ollila
On Wed, Oct 28 2015, Mark Walters wrote: > Previously poll called from emacs would fail silently. This makes it > return a useful error message. > > In the non-deprecated case of notmuch new and appropriate hooks, it > uses notmuch-call-notmuch-process which gives an

Re: elisp + company completion patches, v7

2015-10-27 Thread Tomi Ollila
On Mon, Oct 26 2015, Carl Worth <cwo...@cworth.org> wrote: > On Sun, Oct 25 2015, Tomi Ollila wrote: >> I, if this does not sound nitpicking, would amend those 'This patch' and >> 'With this patch' texts in commit messages away by some rewording -- those >> pose a

Re: elisp + company completion patches, v7

2015-10-25 Thread Tomi Ollila
On Sun, Oct 25 2015, David Bremner wrote: > This round contains several bug fixes from Mark, and a slightly > re-organized initialization from me. In particular, TAB should now > behave sanely under company. Also there's a stub > notmuch-address-message-insinuate to prevent

[PATCH] NEWS: a few formatting updates for 0.21 news items (wiki compatibility)

2015-10-19 Thread Tomi Ollila
Trailing dots were removed from 3 NEWS items so that those appear in same level as surrounding "heading" lines in generated wiki page. One trailing dot was added to nmbug-status item so it appears as normal text in generated wiki page. `nmbug-status` was put in backticks so it looks the same as

Re: NEWS patches v2

2015-10-19 Thread Tomi Ollila
On Sun, Oct 18 2015, Jani Nikula <j...@nikula.org> wrote: > On Sun, 18 Oct 2015, Tomi Ollila <tomi.oll...@iki.fi> wrote: >> On Sat, Oct 17 2015, David Bremner <da...@tethera.net> wrote: >> >>> I had a series in progress more or less at the same time as J

Re: NEWS patches v2

2015-10-18 Thread Tomi Ollila
On Sat, Oct 17 2015, David Bremner wrote: > I had a series in progress more or less at the same time as Jani, so > of course there were all kinds of silly conflicts. > > I rebased the two series together, and tried to get most of Tomi's > formatting comments. From the

Re: [PATCH 3/4] NEWS: news for notmuch address --deduplicate option

2015-10-17 Thread Tomi Ollila
On Sat, Oct 17 2015, Jani Nikula wrote: > --- > NEWS | 13 + > 1 file changed, 13 insertions(+) > > diff --git a/NEWS b/NEWS > index 317e7e62deaa..9940f8185e82 100644 > --- a/NEWS > +++ b/NEWS > @@ -14,6 +14,19 @@ Date queries now support `date:..!` shorthand for >

Re: [PATCH 4/4] NEWS: news for notmuch_directory_delete

2015-10-17 Thread Tomi Ollila
On Sat, Oct 17 2015, Jani Nikula wrote: > --- > NEWS | 13 + > 1 file changed, 13 insertions(+) > > diff --git a/NEWS b/NEWS > index 9940f8185e82..c5417144b755 100644 > --- a/NEWS > +++ b/NEWS > @@ -47,6 +47,19 @@ Library > The use of absolute paths is now enforced

Re: [PATCH 2/4] NEWS: news for date:..!

2015-10-17 Thread Tomi Ollila
On Sat, Oct 17 2015, Jani Nikula wrote: > --- > NEWS | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/NEWS b/NEWS > index 95470d7c070b..317e7e62deaa 100644 > --- a/NEWS > +++ b/NEWS > @@ -7,6 +7,13 @@ General > Notmuch now requires gmime >= 2.6.7. The gmime 2.4

Re: [PATCH 1/4] NEWS: nmbug-status supports sort order

2015-10-17 Thread Tomi Ollila
On Sat, Oct 17 2015, Jani Nikula wrote: > --- > NEWS | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/NEWS b/NEWS > index 3593ded71bd7..95470d7c070b 100644 > --- a/NEWS > +++ b/NEWS > @@ -27,6 +27,11 @@ Library > The use of absolute paths is now enforced when

[PATCH] doc: update copyright year

2015-10-15 Thread Tomi Ollila
--- doc/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/conf.py b/doc/conf.py index 8fbc8544b0b1..4609bfe3709e 100644 --- a/doc/conf.py +++ b/doc/conf.py @@ -12,7 +12,7 @@ master_doc = 'index' # General information about the project. project = u'notmuch'

[PATCH v2] doc: update copyright year to contain range 2009-2015

2015-10-15 Thread Tomi Ollila
Now it is accurate and provides more information to the reader of the documents. --- doc/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/conf.py b/doc/conf.py index 8fbc8544b0b1..65adafeb1cef 100644 --- a/doc/conf.py +++ b/doc/conf.py @@ -12,7 +12,7 @@ master_doc =

Re: [PATCH] emacs: show: increase default max-text-part-size

2015-10-15 Thread Tomi Ollila
On Thu, Oct 15 2015, Mark Walters wrote: > Currently notmuch-show-max-text-part-size is 1 which means some > relatively normal messages have all parts hidden by default. Increase > this to 10 by default. > > The setting was introduced to alleviate problems with

Re: [PATCH] emacs: tree bugfix

2015-10-15 Thread Tomi Ollila
On Thu, Oct 15 2015, Mark Walters wrote: > Formerly replying to an encrypted message in tree-view did not work: > the message was not decrypted. This commit makes notmuch-tree respect > the setting of notmuch-crypto-process-mime. In particular, if >

Re: V2 of directory delete patches

2015-10-11 Thread Tomi Ollila
On Sat, Oct 10 2015, David Bremner wrote: > Changes since id:cover.1443213654.git.j...@nikula.org > > - tidy commit messages > - update docs in lib/notmuch.h > - make an executive decision not to push _remove_directory into lib > for now; it can always be a new api

[PATCH] notmuch-emacs-mua: set EMACS{, CLIENT} variables to defaults when empty

2015-10-04 Thread Tomi Ollila
... in addition to doing this when these variables are unset. It is more useful to use defaults (emacs or emacsclient) than empty string as a command name. --- Without this: $ EMACS= ./notmuch-emacs-mua ./notmuch-emacs-mua: line 150: exec: --: invalid option exec: usage: exec [-cl] [-a name]

[PÄTCH] notmuch-emacs-mua: set EMACS{,CLIENT} variables to defaults when empty

2015-10-04 Thread Tomi Ollila
... in addition to doing this when these variables are unset. It is more useful to use defaults (emacs or emacsclient) than empty string as a command name. --- Without this: $ EMACS= ./notmuch-emacs-mua ./notmuch-emacs-mua: line 150: exec: --: invalid option exec: usage: exec [-cl] [-a name]

Re: [PATCH] nmbug-status: print config errors to stderr

2015-10-03 Thread Tomi Ollila
On Sat, Oct 03 2015, Jani Nikula wrote: > Particularly scripted usage with stdout redirection can be confusing > if errors are printed to stdout instead of stderr. LGTM > --- > devel/nmbug/nmbug-status | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH v4 5/5] notmuch-emacs-mua: do not create a frame by default with --client

2015-09-30 Thread Tomi Ollila
On Tue, Sep 29 2015, Tomi Ollila <tomi.oll...@iki.fi> wrote: > On Mon, Sep 28 2015, David Bremner <da...@tethera.net> wrote: > >> Tomi Ollila <tomi.oll...@iki.fi> writes: >> >> >>> if [ -n "$AUTO_DAEMON" -a -z "$CREATE

[PATCH] notmuch-emacs-mua: make --auto-daemon imply --create-frame

2015-09-30 Thread Tomi Ollila
Before this change with --auto-daemon but without --create-frame emacs server was started but no clients stay connected to it (in both graphical and terminal displays). Note that this changes how --client --auto-daemon works on graphical display; New emacs frame is now created for the message

Re: [PATCH v4 5/5] notmuch-emacs-mua: do not create a frame by default with --client

2015-09-29 Thread Tomi Ollila
On Mon, Sep 28 2015, David Bremner <da...@tethera.net> wrote: > Tomi Ollila <tomi.oll...@iki.fi> writes: > > >> if [ -n "$AUTO_DAEMON" -a -z "$CREATE_FRAME" ]; then >> echo "$0: --auto-daemon is only applicable with --create-frame.&

Re: [PATCH v2] nmbug-status: add support for specifying sort order for each view

2015-09-29 Thread Tomi Ollila
On Tue, Sep 29 2015, David Bremner wrote: > Jani Nikula writes: > >> Let each view have a "sort" key, typically used with values >> "oldest-first" or "newest-first" (although all values in Query.SORT >> are accepted), and sort the results accordingly. Oldest

Re: [PATCH] lib: add support for date:..! to mean date:..

2015-09-26 Thread Tomi Ollila
On Sat, Sep 26 2015, David Bremner wrote: > Jani Nikula writes: > >> It doesn't seem likely we can support simple date: expanding to >> date:.. any time soon. (This can be done with a future >> version of Xapian, or with a custom query query parser.) In the

[PATCH] emacs/Makefile.local: notmuch-lib.elc depend on notmuch-version.elc

2015-09-25 Thread Tomi Ollila
emacs/make-depend.el will compute all other related dependencies except this one: notmuch-version is not top-level `require' expression in notmuc-lib.el[c] but conditional based on the existence of notmuch-version.el[c]. emacs/make-depend.el does not know now notmuch-version.el[c] becomes into

Re: [PATCH] cli: use designated initializer to initialize add_files_state

2015-09-25 Thread Tomi Ollila
On Fri, Sep 25 2015, Jani Nikula wrote: > The side effect is that all of add_files_state will be initialized to > zero, removing any lingering doubt that some of it might not be > initialized. It's not a small struct, and the initialization is > scattered around a bit, so this

Re: [PATCH] test suite: don't consider skipped individual tests as failing

2015-09-25 Thread Tomi Ollila
On Fri, Sep 25 2015, David Bremner <da...@tethera.net> wrote: > Tomi Ollila <tomi.oll...@iki.fi> writes: > >> >> Looks OK, but I think there should be (some good) comment here informing >> any potential viewer that we're not checking $skipped... >>

Re: [PATCH] test suite: don't consider skipped individual tests as failing

2015-09-24 Thread Tomi Ollila
On Thu, Sep 24 2015, David Bremner wrote: > It isn't completely clear what we want to do here, but > > 1) We currently don't fail if we skip a whole test file (mainly because > we neglect to count those skipped tests properly). This change at least > makes the two kinds of

Re: [PATCH v2 6/9] cli: change the data structure for notmuch address deduplication

2015-09-24 Thread Tomi Ollila
On Thu, Sep 24 2015, David Bremner wrote: > David Bremner writes: > >> Jani Nikula writes: >> >> >>> +else >>> + v = !!m1->name - !!m2->name; >> >> Is this really idiomatic? It seems a little difficult to follow to me. >> For me it

Re: [PATCH] make test: NOTMUCH_TEST_QUIET=1 is now the default

2015-09-20 Thread Tomi Ollila
On Sun, Sep 20 2015, David Bremner <da...@tethera.net> wrote: > Tomi Ollila <tomi.oll...@iki.fi> writes: > >> make test V=1 (or any other value than 0) and make test V=0 >> works similar way as build in general >> --- >> >> This is 2nd try: In

[PATCH] make test: NOTMUCH_TEST_QUIET=1 is now the default

2015-09-11 Thread Tomi Ollila
make test V=1 (or any other value than 0) and make test V=0 works similar way as build in general --- This is 2nd try: In first one I changed test-lib.sh -- there problem was the precedence between command line argument and NOTMUCH_TEST_QUIET environment variable. Here I don't see a problem there

Re: using the fringe to indicate good signatures

2015-09-08 Thread Tomi Ollila
On Tue, Sep 08 2015, David Edmondson wrote: > On Tue, Sep 08 2015, Daniel Kahn Gillmor wrote: >>> More seriously, we could use the indentation space for a similar >>> indicator, which would allow us some room at an appropriate depth for >>> each message

Re: [PATCH v3] emacs: wrap current search in parens when filtering

2015-09-07 Thread Tomi Ollila
On Mon, Sep 07 2015, Uli Scholler wrote: > When filtering the current search further with notmuch-search-filter, > wrap the current search in parens (if necessary). > > This fixes unexpected behavior when the current search is > complex (like "(tag:this and

Re: [PATCH v5 3/3] emacs: Bind filter in search to 'l'

2015-09-07 Thread Tomi Ollila
On Sat, Jun 13 2015, Mark Walters wrote: > Change the key binding for filter (or "limit") in search-mode. This > gives consistency with the new filter in show-mode, and frees 'f' for > forward-thread in the future. > --- > emacs/notmuch.el | 4 ++-- > 1 file changed,

[PATCH] emacs: add emacs/try-emacs-mua.sh

2015-09-07 Thread Tomi Ollila
try-emacs-mua.sh provides an easy way to try and experiment with the notmuch emacs client provided in emacs subdirectory of notmuch source tree. User is required to choose whether to run emacs with -q, -Q or neither -- and experienced ones may add other command line options, like '-f notmuch'.

Re: [PATCH 2/2] cli/count: apply uncrustify changes

2015-09-07 Thread Tomi Ollila
On Mon, Sep 07 2015, David Bremner wrote: > --- > notmuch-count.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/notmuch-count.c b/notmuch-count.c > index 66f5e53..a844384 100644 > --- a/notmuch-count.c > +++ b/notmuch-count.c > @@ -117,7

Re: [PATCH 1/2] test: fix whitespace/indentation in symbol-test

2015-09-06 Thread Tomi Ollila
On Fri, Sep 04 2015, Jani Nikula wrote: > Not of much consequence, but makes it nicer to do further edits. > --- These 2 patches LGTM ! +1! :D > test/symbol-test.cc | 34 ++ > 1 file changed, 18 insertions(+), 16 deletions(-) > > diff --git

Re: [PATCH 2/2] cli: reset db directory mtime upon directory removal

2015-09-06 Thread Tomi Ollila
On Sat, Sep 05 2015, Jani Nikula wrote: > The library does not have a function to remove a directory document > for a path. Usually this doesn't matter except for a slight waste of > space. However, if the same directory gets added to the filesystem > again, the old directory

Re: [PATCH v2] emacs: wrap current search in parens when filtering

2015-09-06 Thread Tomi Ollila
On Sun, Sep 06 2015, David Bremner wrote: > Some pretty fussy comments follow. Probably I could have fixed these in > the time it took to write this message ;). > > Uli Scholler writes: >> + (let ((grouped-query (notmuch-maybe-group-query-string query)) >>

[PATCH] test: add "set -u" to test-lib.sh

2015-09-03 Thread Tomi Ollila
This prohibits unset variables to expand to empty strings. Without this e.g misspelled variables cause unintentional results. Now all the test variables are either initialized, or in case of optional arguments and user-provided environment variables the form ${variable-} is used. Two unusable

Re: [PATCH] emacs: wrap current search in parens when filtering

2015-09-03 Thread Tomi Ollila
On Thu, Sep 03 2015, Uli Scholler wrote: > When filtering the current search further with notmuch-search-filter, > wrap the current search in parens. > > This fixes unexpected behavior when the current search is > complex (like "(tag:this and date:one_week_ago..) or

[PATCH 2/4] emacs: add function notmuch-address--message-insinuated

2015-09-02 Thread Tomi Ollila
This function is currently used in notmuch-address-message-insinuate (to not enable address completion if it is already enabled). In near future this will be called in other functions to know whether address completion can be used there, too. --- Since

[PATCH 4/4] emacs: bind notmuch-show-resend-message to 'b' in notmuch-show mode

2015-09-02 Thread Tomi Ollila
This binding is similar to mutt's, which is bind {mode} b "bounce-message"# remail a message to another user where {mode} is 'index', 'pager' or 'attach'. --- emacs/notmuch-show.el | 1 + 1 file changed, 1 insertion(+) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index

[PATCH 3/4] emacs: add function to resend message to new recipients

2015-09-02 Thread Tomi Ollila
The new function notmuch-show-message-resend re-sends message to new recipients using #'message-resend. Recipients are read from minibuffer as a comma-separated string (with some keyboard support including tab completion). Final confirmation before sending is asked. --- Since

Re: [PATCH 2/2] emacs: add function to resend message to new recipients

2015-08-31 Thread Tomi Ollila
On Mon, Aug 31 2015, David Bremner <da...@tethera.net> wrote: > Tomi Ollila <tomi.oll...@iki.fi> writes: > >> >> emacs -q -L $PWD/emacs -l emacs/notmuch.el -f notmuch --eval '(progn (setq >> notmuch-address-command "nottoomuch-addresses.sh") >

Re: [RFC PATCH 5/5] cli: add support for deduplicating based on case insensitive address

2015-08-31 Thread Tomi Ollila
On Sun, Aug 30 2015, David Bremner wrote: > Jani Nikula writes: > >> Consider all variants of an email address as one, and print the most >> common variant. > > Other than the quibbles already mentioned, the series looks ok to > me. For production it should

Re: [PATCH 1/2] emacs: add defsubst notmuch-address--message-insinuated

2015-08-30 Thread Tomi Ollila
On Sun, Aug 30 2015, David Bremner da...@tethera.net wrote: Tomi Ollila tomi.oll...@iki.fi writes: +(defsubst notmuch-address--message-insinuated () + (memq notmuch-address-message-alist-member message-completion-alist)) + Is there some advantage to defsubst other than (maybe

Re: [PATCH 2/2] emacs: add function to resend message to new recipients

2015-08-30 Thread Tomi Ollila
On Sun, Aug 30 2015, David Bremner da...@tethera.net wrote: Tomi Ollila tomi.oll...@iki.fi writes: The new function notmuch-show-message-resend re-sends message to new recipients using #'message-resend. Recipients are read from minibuffer as a comma-separated string (with some keyboard

Re: [PATCH v4 5/5] notmuch-emacs-mua: do not create a frame by default with --client

2015-08-28 Thread Tomi Ollila
On Fri, Aug 28 2015, Jani Nikula j...@nikula.org wrote: On Wed, 26 Aug 2015, Tomi Ollila tomi.oll...@iki.fi wrote: On Sat, Aug 15 2015, Jani Nikula j...@nikula.org wrote: Make the default behaviour for --client the same as emacsclient default: do not create a new frame. Add a new option

Re: notmuch-show-toggle-message behavior

2015-08-27 Thread Tomi Ollila
On Fri, Aug 28 2015, Bart Bunting b...@bunting.net.au wrote: Hi, Hi Bart Following the discussion in: id:874mjk614c@qmul.ac.uk regarding the behavior of notmuch-show-toggle-message and following hyperlinks, I wanted to chime in on a similar but not exactly the same topic. I am blind

Re: how to make return follow hyperlink?

2015-08-27 Thread Tomi Ollila
On Wed, Aug 26 2015, Peter Salazar cycleofs...@gmail.com wrote: How do I make it so that hitting RET on a hyperlink follows it? Is there a setting analogous to org-return-follows-link? I often have links in incoming emails, and I'd like to be able to open them in my browser with one

Re: [PATCH v4 5/5] notmuch-emacs-mua: do not create a frame by default with --client

2015-08-26 Thread Tomi Ollila
On Sat, Aug 15 2015, Jani Nikula j...@nikula.org wrote: Make the default behaviour for --client the same as emacsclient default: do not create a new frame. Add a new option --create-frame, passing the same option to emacsclient to create a frame. --- v2: fix killing frame with

[PATCH v2] devel/release-checks: added checking of copyright year in documentation

2015-08-26 Thread Tomi Ollila
Check that copyright year will be current year in generated documentation. Checking is done my matching that copyright line contains current year as a substring which is good enough approximation in this context. --- v2 of id:1438511187-11321-1-git-send-email-tomi.oll...@iki.fi, perhaps

[PATCH 2/2] emacs: add function to resend message to new recipients

2015-08-26 Thread Tomi Ollila
The new function notmuch-show-message-resend re-sends message to new recipients using #'message-resend. Recipients are read from minibuffer as a comma-separated string (with some keyboard support including tab completion). Final confirmation before sending is asked. --- Note that

[PATCH 1/2] emacs: add defsubst notmuch-address--message-insinuated

2015-08-26 Thread Tomi Ollila
This inline function is currently used in notmuch-address-message-insinuate (to not enable address completion if it is already enabled). In some functions later this will be called to know whether address completion can be used there, too. --- Previous version of this 2-patch series is at

Re: Resending the same email

2015-08-23 Thread Tomi Ollila
On Sun, Aug 23 2015, Jani Nikula j...@nikula.org wrote: On Sun, 23 Aug 2015, David Mazieres dm-list-email-notm...@scs.stanford.edu wrote: Sebastian Fischmeister sfisc...@uwaterloo.ca writes: Hi, My previous mail editor had a useful feature to resend already sent emails. It's essentially

Re: [PATCH] cli/lib: remove support for GMime 2.4

2015-08-17 Thread Tomi Ollila
On Mon, Aug 17 2015, Daniel Kahn Gillmor d...@fifthhorseman.net wrote: On Sun 2015-08-16 21:53:36 +0200, Tomi Ollila wrote: On Sun, Aug 16 2015, David Bremner da...@tethera.net wrote: It's becoming a maintenance burden to do anything things with the crypto glue code twice, once for 2.4

Re: [DRAFT PATCH] emacs: show local date next to Date: in case value differs

2015-08-16 Thread Tomi Ollila
On Sun, Aug 16 2015, David Bremner da...@tethera.net wrote: Tomi Ollila tomi.oll...@iki.fi writes: Now I took a little better look why the 'relative' date is not enough for me is that it doesn't always show the local date (it shows like (28 mins. ago) or (March 30)). If these were (28 mins

Re: [PATCH] cli/lib: remove support for GMime 2.4

2015-08-16 Thread Tomi Ollila
On Sun, Aug 16 2015, David Bremner da...@tethera.net wrote: It's becoming a maintenance burden to do anything things with the crypto glue code twice, once for 2.4 and once for 2.6. I don't have any 2.4 version available to test on my development machine anymore, so the 2.4 specific code paths

revision tracking patches round 4

2015-08-15 Thread Tomi Ollila
On Fri, Aug 14 2015, David Bremner wrote: > This obsoletes > > id:1439112285-6681-1-git-send-email-david at tethera.net > > The main differences since that series are in the command line syntax > for notmuch-count. > > We also now consider passing --uuid to notmuch-compact an error, since >

<    5   6   7   8   9   10   11   12   13   14   >