Re: [PATCH] cli: try to run external notmuch- prefixed commands as subcommands

2016-10-22 Thread Mark Walters
On Sat, 22 Oct 2016, Jani Nikula wrote: > If the given subcommand is not known to notmuch, try to execute > external notmuch- instead. This allows users to have their > own notmuch related tools be run via the notmuch command, not unlike > git does. Also notmuch-emacs-mua will

[PATCH] emacs: add compatability functions for emacs 23

2016-10-22 Thread Mark Walters
Some of the recent changes to the emacs code have used functions introduced in emacs 24. The functions used are read-char-choice and setq-local. This changeset adds compatability functions to notmuch-lib so that it should work on emacs 23. --- Hi I tried compiling under emacs 23 recently and

Re: [PATCH] Document emacs keybindings

2016-10-22 Thread David Bremner
Mark Walters writes: > This adds a file under devel listing all the keybindings sorted by key > in the main three modes (search, show and tree). > > This should make easier to see what keybindings are available when > adding new features. this seems fine to me,

Re: [PATCH] emacs: tag-jump: tag the region when active in search mode

2016-10-22 Thread David Bremner
Mark Walters writes: > + (extra-args (when (eq major-mode 'notmuch-search-mode) > +(notmuch-search-interactive-region))) I think I prefer the previous patch for simplicity. If people think it's better not to break the API, then I'd

Re: [PATCH] emacs: tree: make tag updates show in the message pane

2016-10-22 Thread David Bremner
Mark Walters writes: > This makes tag changes appear in the message pane as well as in the > tree window. > > Note that the message pane is reloaded each time a message is viewed > so the tags shown in the message pane can still be different from > those in the tree

Re: [PATCH v2] emacs: make faces clear on dark backgrounds.

2016-10-22 Thread David Bremner
Matt Armstrong writes: > The notmuch-tag-flagged, notmuch-search-flagged-face and > notmuch-crypto-part-header faces defaulted to "blue", which is nearly > unreadable when a dark background is in use. This is addressed by using > "gold" for dark backgrounds. While we're

Re: [PATCH] nmbug: allow excluded messages

2016-10-22 Thread David Bremner
David Bremner writes: > "W. Trevor King" writes: > >> On Sat, Oct 22, 2016 at 08:59:40AM +0100, Mark Walters wrote: >>> This makes nmbug work even if the notmuch mailing list messages are >>> excluded (i.e., have a tag in the excluded tags list). >>> … >>>

Re: [PATCH] nmbug: allow excluded messages

2016-10-22 Thread David Bremner
"W. Trevor King" writes: > On Sat, Oct 22, 2016 at 08:59:40AM +0100, Mark Walters wrote: >> This makes nmbug work even if the notmuch mailing list messages are >> excluded (i.e., have a tag in the excluded tags list). >> … >> (I think this is the only place it is needed, but I

Re: [PATCH] nmbug: allow excluded messages

2016-10-22 Thread W. Trevor King
On Sat, Oct 22, 2016 at 08:59:40AM +0100, Mark Walters wrote: > This makes nmbug work even if the notmuch mailing list messages are > excluded (i.e., have a tag in the excluded tags list). > … > (I think this is the only place it is needed, but I don't use nmbug > much so I haven't tested

Re: [PATCH] emacs: add mimetype completions to searches

2016-10-22 Thread Jani Nikula
On Sat, 22 Oct 2016, Keith Amidon wrote: > On Sat, 2016-10-22 at 12:55 +0300, Jani Nikula wrote: >> Any ideas how to get a list of mime types in shell, so I could do the >> same in bash completion without hard-coding some limited list? > > Not sure if this is really what

[PATCH] completion: complete mimetype: search prefix

2016-10-22 Thread Jani Nikula
Use /etc/mime.types if available, with a homebrew sed parser, and fall back to a handful of common types otherwise. --- completion/notmuch-completion.bash | 32 1 file changed, 32 insertions(+) diff --git a/completion/notmuch-completion.bash

Re: [PATCH] emacs: add mimetype completions to searches

2016-10-22 Thread Keith Amidon
On Sat, 2016-10-22 at 12:55 +0300, Jani Nikula wrote: > Any ideas how to get a list of mime types in shell, so I could do the > same in bash completion without hard-coding some limited list? Not sure if this is really what you're looking for, but on my archlinux system, the file /etc/mime.types

[PATCH] man: advertize notmuch-emacs-mua as notmuch emacs-mua subcommand

2016-10-22 Thread Jani Nikula
With subcommand handling for external commands we can now hide the implementation detail of emacs-mua being a separate notmuch-emacs-mua script. --- doc/man1/notmuch-emacs-mua.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/man1/notmuch-emacs-mua.rst

[PATCH] completion: complete notmuch emacs-mua

2016-10-22 Thread Jani Nikula
With subcommand handling for external commands we can easily complete 'notmuch emacs-mua' using the existing completion system. --- completion/notmuch-completion.bash | 34 +- 1 file changed, 33 insertions(+), 1 deletion(-) diff --git

Re: [PATCH v4] Add notmuch-show--build-queries.

2016-10-22 Thread David Bremner
Matt Armstrong writes: > notmuch-show--build-buffer now queries a list of queries built by the > former. This simplifies the logic. It also provides an easy place to > experiment with alternate sets of queries for given notmuch-show-* > variables (e.g. users can use

Re: [PATCH] Expand docstrings about fcc using notmuch-insert

2016-10-22 Thread David Bremner
Keith Amidon writes: > This commit expands docstrings for notmuch-fcc-dirs and > notmuch-maildir-fcc-with-notmuch-insert to describe how quoted strings > are processed and make the ability to configure sent folders containing > whitespace more discoverable. pushed to

Re: [PATCH 1/2] test-lib.sh: rename $DTACH_TERM to $SMART_TERM

2016-10-22 Thread David Bremner
Series pushed to release and master, d ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch

Re: [PATCH] NEWS for two bugfixes

2016-10-22 Thread David Bremner
Mark Walters writes: > This adds news items for the two bugs > > emacs: search face bugfix > and > emacs: fix notmuch-search-line-faces defcustom > --- > > Hi here is a NEWS update as requested. pushed to release and master d

Re: [PATCH] emacs: interesting-buffer bugfixes/updates

2016-10-22 Thread Tomi Ollila
On Sat, Oct 22 2016, Mark Walters wrote: > The command notmuch-interesting-buffer has got out of date -- it > doesn't mention notmuch-tree, and it still refers to message-mode not > notmuch-message-mode. Update both of these. > > This fixes the bug that

[PATCH] cli: try to run external notmuch- prefixed commands as subcommands

2016-10-22 Thread Jani Nikula
If the given subcommand is not known to notmuch, try to execute external notmuch- instead. This allows users to have their own notmuch related tools be run via the notmuch command, not unlike git does. Also notmuch-emacs-mua will be executable via 'notmuch emacs-mua'. By design, this does not

[PATCH] emacs: interesting-buffer bugfixes/updates

2016-10-22 Thread Mark Walters
The command notmuch-interesting-buffer has got out of date -- it doesn't mention notmuch-tree, and it still refers to message-mode not notmuch-message-mode. Update both of these. This fixes the bug that notmuch-cycle-notmuch-buffers does not include notmuch-tree or notmuch-message-mode buffers in

[PATCH v2] notmuch-emacs-mua: add support for mailto: URL handling

2016-10-22 Thread Jani Nikula
Add support for composing an email in the Notmuch Emacs UI using a mailto: URL. The mailto: URL mode is mutually exclusive with specifying other message modifying parameters and positional arguments. --- v2: wrap the progn in one assignment, with matching parens (Tomi) ---

Re: [PATCH] nmbug: allow excluded messages

2016-10-22 Thread Tomi Ollila
On Sat, Oct 22 2016, Mark Walters wrote: > This makes nmbug work even if the notmuch mailing list messages are > excluded (i.e., have a tag in the excluded tags list). > --- > > I keep all my mailing list emails under an excluded tag (initially > this was to test the

Re: bash completion install location

2016-10-22 Thread Tomi Ollila
On Tue, Oct 04 2016, David Bremner wrote: > Debian complains about /etc/bash_completion.d begin obsolete and also > claims that /usr/share/bash-completion/completions is the right location > [1]. Can someone who uses bash (ideally also some non-Debian platform) > please

Re: bash completion install location

2016-10-22 Thread Jani Nikula
On Tue, 04 Oct 2016, David Bremner wrote: > Debian complains about /etc/bash_completion.d begin obsolete and also > claims that /usr/share/bash-completion/completions is the right location > [1]. Can someone who uses bash (ideally also some non-Debian platform) > please

Re: [PATCH] emacs: add mimetype completions to searches

2016-10-22 Thread Jani Nikula
On Tue, 27 Sep 2016, Mark Walters wrote: > When doing the patch for commit > f94921520778ae4005500f5d1b943e2d4ddd3b2a emacs: show: let the user > override the mime-type of an attachment, I discovered how to do > mimetype completion (in particular where to get a list of

Re: [PATCH] nmbug: allow excluded messages

2016-10-22 Thread David Bremner
Mark Walters writes: > This makes nmbug work even if the notmuch mailing list messages are > excluded (i.e., have a tag in the excluded tags list). > --- > > I keep all my mailing list emails under an excluded tag (initially > this was to test the exclude code

[PATCH] nmbug: allow excluded messages

2016-10-22 Thread Mark Walters
This makes nmbug work even if the notmuch mailing list messages are excluded (i.e., have a tag in the excluded tags list). --- I keep all my mailing list emails under an excluded tag (initially this was to test the exclude code thoroughly, but I find it convenient to keep day to day email and

[PATCH] NEWS for two bugfixes

2016-10-22 Thread Mark Walters
This adds news items for the two bugs emacs: search face bugfix and emacs: fix notmuch-search-line-faces defcustom --- Hi here is a NEWS update as requested. Best wishes Mark NEWS | 9 + 1 file changed, 9 insertions(+) diff --git a/NEWS b/NEWS index ac3ceb1..547b961 100644