[RFC PATCH 0/2] natural language date range search

2012-02-25 Thread Jani Nikula
On Sat, 25 Feb 2012 17:05:44 +0200, Tomi Ollila wrote: > On Mon, 20 Feb 2012 00:55:50 +0200, Jani Nikula wrote: > > Hi all, these patches add support for natural language date range search > > of the form date:since..until, where since and until can be fairly free > > form date/time expressions

[PATCH v6 00/10] Reply improvements

2012-02-25 Thread Austin Clements
Quoth Adam Wolfe Gordon on Feb 25 at 3:25 pm: > On Sat, Feb 25, 2012 at 14:29, David Bremner wrote: > > By coincidence I "unpromoted it", perhaps because I was missing > > context. It seemed strange to push one patch from the middle of the > > series. > > I think that part was promoted by

[PATCH v6 00/10] Reply improvements

2012-02-25 Thread David Bremner
On Tue, 21 Feb 2012 23:46:29 -0700, Adam Wolfe Gordon wrote: > > I noticed that the factoring out reply creation patch has been promoted to > "maybe ready" in nmbug. I think this version is unchanged, but since the > old one hasn't been pushed this one can probably

[RFC PATCH 0/2] natural language date range search

2012-02-25 Thread Tomi Ollila
On Mon, 20 Feb 2012 00:55:50 +0200, Jani Nikula wrote: > Hi all, these patches add support for natural language date range search > of the form date:since..until, where since and until can be fairly free > form date/time expressions in English. > > Examples: > > date:two-days..yesterday (all

Replacing my name/email with "me" (or similar) in author lists

2012-02-25 Thread Daniel
Have somebody tried to implement replacement of the user author with "me" in author lists (using defined email address(es))? >From what I understand, at least the Python bindings deliver primarily author names (not addresses), so it would seem more appropriate that this be done in the backend.

[RFC PATCH 1/3] emacs: selection-menu.el

2012-02-25 Thread Tomi Ollila
On Fri, 24 Feb 2012 23:36:23 +, Mark Walters wrote: > On Thu, 23 Feb 2012 17:10:15 +0200, Tomi Ollila wrote: > > RFC/Idea for "improving" some selections made (in notmuch or elsewhere) > > In the hope that this will be useful, and to get some improvement advice. > > > > I've found it

[PATCH v6 00/10] Reply improvements

2012-02-25 Thread Adam Wolfe Gordon
On Sat, Feb 25, 2012 at 14:29, David Bremner wrote: > By coincidence I "unpromoted it", perhaps because I was missing > context. It seemed strange to push one patch from the middle of the > series. I think that part was promoted by Austin because one of his otherwise-ready patches relies on it,

[RFC PATCH 1/3] emacs: selection-menu.el

2012-02-25 Thread Tom Prince
On Fri, 24 Feb 2012 23:36:23 +, Mark Walters wrote: > I have played with this and I like the feel of it: it is much more > informative than completing-read and much less cluttered than > ido-completing-read. I wonder if the completion method should be customizable, rather than forcing a

[PATCH] test: Add test for searching of uncommonly encoded messages

2012-02-25 Thread Serge Z
Hi! I've struck another problem: I've got an html/text email with body encoded with cp1251. Its encoding is mentioned in both Content-type: email header and html tag. So when the client tries to display it with external html2text converter, The message is decoded twice: first by client, second

[PATCH v5 00/12] emacs: more flexible and consistent tagging operations

2012-02-25 Thread David Bremner
On Fri, 24 Feb 2012 00:07:27 +0100, Pieter Praet wrote: > On Wed, 08 Feb 2012 11:58:32 -0400, David Bremner > wrote: > > On Sun, 5 Feb 2012 11:13:41 +0400, Dmitry Kurochkin > gmail.com> wrote: > > How about if '*' applies to all messages (as it currently does), > but 'C-u *' only to open

[PATCH v6 1/3] emacs: `notmuch-show-get-message-id': optionally return Message-Id sans prefix

2012-02-25 Thread David Bremner
On Sun, 19 Feb 2012 21:50:25 +0100, Pieter Praet wrote: > * emacs/notmuch-show.el > > (notmuch-show-get-message-id): > Add optional arg BARE. When non-nil, return a Message-Id without > quotes and prefix, thus obviating the need to strip them off again > in various places. Pushed

[PATCH v3 0/3] Fix refreshing with state and make it the default

2012-02-25 Thread David Bremner
On Tue, 21 Feb 2012 10:42:30 -0500, Austin Clements wrote: > Third time's the charm. This fixes show refresh to not mark anything > read when keeping state while refreshing and makes it the default. > Pushed. d

[PATCH 1/2] NEWS: add entry for python 3.2 compatibility

2012-02-25 Thread David Bremner
On Wed, 22 Feb 2012 20:33:30 +0100, Justus Winter <4winter at informatik.uni-hamburg.de> wrote: > Signed-off-by: Justus Winter <4winter at informatik.uni-hamburg.de> > --- > NEWS |7 +++ > 1 files changed, 7 insertions(+), 0 deletions(-) Pushed both. d

plans for 0.12

2012-02-25 Thread David Bremner
ave branched for release. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 315 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120225/66831e3d/attachment.pgp>

[PATCH v2 2/2] emacs: support text/calendar mime type

2012-02-25 Thread David Bremner
On Thu, 9 Feb 2012 14:46:03 +, Jani Nikula wrote: > Replace text/x-vcalendar with text/calendar, while maintaining support > and backwards compatibility for text/x-vcalendar. pushed

[PATCH] test: replace occurrences of $PWD with vars that are more stable

2012-02-25 Thread David Bremner
On Wed, 1 Feb 2012 21:37:21 +0100, Pieter Praet wrote: > Thanks to Dmitry Kurochkin > for pointing this out: id:"87d39ymyb4.fsf at gmail.com" Pushed. d

[PATCH] emacs: Fix out of date comment

2012-02-25 Thread Austin Clements
The behavior of the header line in show-mode changed from showing the subject of the first open message to showing the subject of the first message in 4d77f18b. Update a comment to reflect this. --- emacs/notmuch-show.el |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

[Patch v6 13/13] emacs: notmuch.el ignore excluded matches

2012-02-25 Thread Mark Walters
This is a small change to make notmuch.el ignore excluded matches. In the future it could do something better like add a button for rerunning the search with the excludes (particularly if nothing matches with the excludes) or having them invisible and allowing the visibility to be toggled. It is

[Patch v6 12/13] emacs: show: recognize the exclude flag.

2012-02-25 Thread Mark Walters
Show mode will recognize the exclude flag by not opening excluding messages by default, and will start at the first matching non-excluded message. If there are no matching non-excluded messages it will go to the first matching (necessarily excluded) message. --- emacs/notmuch-show.el | 19

[Patch v6 11/13] cli: omit excluded messages in results where appropriate.

2012-02-25 Thread Mark Walters
In all cases of notmuch count/search/show where the results returned cannot reflect the exclude flag return just the matched not-excluded results. If the caller wishes to have all the matched results (i.e., including the excluded ones) they should call with the --no-exclude option. The relevant

[Patch v6 10/13] man: update manpage for notmuch-show --no-exclude option

2012-02-25 Thread Mark Walters
--- man/man1/notmuch-show.1 |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/man/man1/notmuch-show.1 b/man/man1/notmuch-show.1 index b2301d8..8383a05 100644 --- a/man/man1/notmuch-show.1 +++ b/man/man1/notmuch-show.1 @@ -128,6 +128,13 @@ multipart/encrypted part will

[Patch v6 09/13] test: update tests to reflect the exclude flag

2012-02-25 Thread Mark Walters
notmuch show outputs the exclude flag so many tests using notmuch show failed. This commit adds "excluded:0" or "excluded: false" to the expected outputs. After this commit there should be no failing tests. --- test/crypto|9 - test/encoding |2 +- test/json

[Patch v6 08/13] cli: Make notmuch-show respect excludes.

2012-02-25 Thread Mark Walters
This adds the excludes to notmuch-show.c. We do not exclude when only a single message (or part) is requested. notmuch-show will output the exclude information when either text or json format is requested. As this changes the output from notmuch-show it breaks many tests (in a trivial and expected

[Patch v6 07/13] test: update search test to reflect exclude flag

2012-02-25 Thread Mark Walters
notmuch-search.c now returns all matching threads even if it the match is a search.tag_excluded message (but with a mark indicating this). Update the test to reflect this. --- test/search |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/test/search b/test/search index

[Patch v6 06/13] lib: Add the exclude flag to notmuch_query_search_threads

2012-02-25 Thread Mark Walters
Add the NOTMUCH_MESSAGE_FLAG_EXCLUDED flag to notmuch_query_search_threads. Implemented by inspecting the tags directly in _notmuch_thread_create/_thread_add_message rather than as a Xapian query for speed reasons. Note notmuch_thread_get_matched_messages now returns the number of non-excluded

[Patch v6 05/13] lib: Make notmuch_query_search_messages set the exclude flag

2012-02-25 Thread Mark Walters
Add a flag NOTMUCH_MESSAGE_FLAG_EXCLUDED which is set by notmuch_query_search_messages for excluded messages. Also add an option omit_excluded_messages to the search that we do not want the excludes at all. This exclude flag will be added to notmuch_query_search threads in the next patch. ---

[Patch v6 04/13] lib: Rearrange the exclude code in query.cc

2012-02-25 Thread Mark Walters
Slightly refactor the exclude code to give the callers access to the exclude query itself. There should be no functional change. --- lib/query.cc | 29 +++-- 1 files changed, 19 insertions(+), 10 deletions(-) diff --git a/lib/query.cc b/lib/query.cc index

[Patch v6 03/13] test: add tests for new cli --no-exclude option

2012-02-25 Thread Mark Walters
The tests test the new --no-exclude option to search and count. There were no existing tests for the exclude behaviour for count so added these too. --- test/count | 21 + test/search |5 + 2 files changed, 26 insertions(+), 0 deletions(-) diff --git a/test/count

[Patch v6 02/13] cli: Add --no-exclude to the man pages for search and count

2012-02-25 Thread Mark Walters
--- man/man1/notmuch-count.1 |7 +++ man/man1/notmuch-search.1 |7 +++ 2 files changed, 14 insertions(+), 0 deletions(-) diff --git a/man/man1/notmuch-count.1 b/man/man1/notmuch-count.1 index 25fe329..97972a7 100644 --- a/man/man1/notmuch-count.1 +++ b/man/man1/notmuch-count.1

[Patch v6 01/13] cli: add --no-exclude option to count and search.

2012-02-25 Thread Mark Walters
This option turns off the exclusion so all matching messages are returned. We do not need to add this to notmuch-show as that does not (yet) exclude. --- notmuch-count.c | 17 +++-- notmuch-search.c | 17 +++-- 2 files changed, 22 insertions(+), 12 deletions(-) diff

[Patch v6 00/13] Add NOTMUCH_MESSAGE_FLAG_EXCLUDED flag

2012-02-25 Thread Mark Walters
Here is the latest version of the series. It fixes all of Austin's review comments. I don't think there are any significant outstanding issues. Having asked whether excluded messages should still be marked excluded when the tag is explicitly mentioned in the query I realised that they must not

[PATCH v5 00/12] emacs: more flexible and consistent tagging operations

2012-02-25 Thread Pieter Praet
On Fri, 24 Feb 2012 23:38:20 +0100, Pieter Praet wrote: > [...] > Same here. In v2 [1], '*' tags all and 'C-u *' tags only open. > > One can always swap the keybindings if so inclined. > Did I really say that? There's only a single keybinding, so swap it with *what* ?!? Itself ? This should

[Patch v6 00/13] Add NOTMUCH_MESSAGE_FLAG_EXCLUDED flag

2012-02-25 Thread Mark Walters
Here is the latest version of the series. It fixes all of Austin's review comments. I don't think there are any significant outstanding issues. Having asked whether excluded messages should still be marked excluded when the tag is explicitly mentioned in the query I realised that they must not

Re: [PATCH] test: Add test for searching of uncommonly encoded messages

2012-02-25 Thread Serge Z
Hi! I've struck another problem: I've got an html/text email with body encoded with cp1251. Its encoding is mentioned in both Content-type: email header and html meta tag. So when the client tries to display it with external html2text converter, The message is decoded twice: first by client,

[Patch v6 11/13] cli: omit excluded messages in results where appropriate.

2012-02-25 Thread Mark Walters
In all cases of notmuch count/search/show where the results returned cannot reflect the exclude flag return just the matched not-excluded results. If the caller wishes to have all the matched results (i.e., including the excluded ones) they should call with the --no-exclude option. The relevant

[PATCH] man: add references to maildir flag synchronization

2012-02-25 Thread Jani Nikula
notmuch new, restore, and tag commands support maildir flag synchronization with notmuch tags. Reference the notmuch-config(1) man page about it in the relevant man pages. --- man/man1/notmuch-new.1 |5 + man/man1/notmuch-restore.1 |6 ++ man/man1/notmuch-tag.1 |6

[Patch v6 04/13] lib: Rearrange the exclude code in query.cc

2012-02-25 Thread Mark Walters
Slightly refactor the exclude code to give the callers access to the exclude query itself. There should be no functional change. --- lib/query.cc | 29 +++-- 1 files changed, 19 insertions(+), 10 deletions(-) diff --git a/lib/query.cc b/lib/query.cc index

[Patch v6 12/13] emacs: show: recognize the exclude flag.

2012-02-25 Thread Mark Walters
Show mode will recognize the exclude flag by not opening excluding messages by default, and will start at the first matching non-excluded message. If there are no matching non-excluded messages it will go to the first matching (necessarily excluded) message. --- emacs/notmuch-show.el | 19

[Patch v6 06/13] lib: Add the exclude flag to notmuch_query_search_threads

2012-02-25 Thread Mark Walters
Add the NOTMUCH_MESSAGE_FLAG_EXCLUDED flag to notmuch_query_search_threads. Implemented by inspecting the tags directly in _notmuch_thread_create/_thread_add_message rather than as a Xapian query for speed reasons. Note notmuch_thread_get_matched_messages now returns the number of non-excluded

[Patch v6 10/13] man: update manpage for notmuch-show --no-exclude option

2012-02-25 Thread Mark Walters
--- man/man1/notmuch-show.1 |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/man/man1/notmuch-show.1 b/man/man1/notmuch-show.1 index b2301d8..8383a05 100644 --- a/man/man1/notmuch-show.1 +++ b/man/man1/notmuch-show.1 @@ -128,6 +128,13 @@ multipart/encrypted part will

Re: [RFC PATCH 1/3] emacs: selection-menu.el

2012-02-25 Thread Tomi Ollila
On Fri, 24 Feb 2012 23:36:23 +, Mark Walters markwalters1...@gmail.com wrote: On Thu, 23 Feb 2012 17:10:15 +0200, Tomi Ollila tomi.oll...@iki.fi wrote: RFC/Idea for improving some selections made (in notmuch or elsewhere) In the hope that this will be useful, and to get some improvement

Re: [PATCH v6 00/10] Reply improvements

2012-02-25 Thread Adam Wolfe Gordon
On Sat, Feb 25, 2012 at 14:29, David Bremner da...@tethera.net wrote: By coincidence I unpromoted it, perhaps because I was missing context. It seemed strange to push one patch from the middle of the series. I think that part was promoted by Austin because one of his otherwise-ready patches

Re: [RFC PATCH 0/2] natural language date range search

2012-02-25 Thread Jani Nikula
On Sat, 25 Feb 2012 17:05:44 +0200, Tomi Ollila tomi.oll...@iki.fi wrote: On Mon, 20 Feb 2012 00:55:50 +0200, Jani Nikula j...@nikula.org wrote: Hi all, these patches add support for natural language date range search of the form date:since..until, where since and until can be fairly free

[Patch v6 13/13] emacs: notmuch.el ignore excluded matches

2012-02-25 Thread Mark Walters
This is a small change to make notmuch.el ignore excluded matches. In the future it could do something better like add a button for rerunning the search with the excludes (particularly if nothing matches with the excludes) or having them invisible and allowing the visibility to be toggled. It is

Re: [PATCH 1/2] Convert non-UTF-8 parts to UTF-8 before indexing them

2012-02-25 Thread Austin Clements
LGTM. I'm assuming this interacts with the uuencoding filter in the right order (I don't see how any other order could be correct), but don't actually know. Quoth Michal Sojka on Feb 24 at 8:36 am: This fixes a bug that didn't allow to search for non-ASCII words such parts. The code here was

[Patch v6 05/13] lib: Make notmuch_query_search_messages set the exclude flag

2012-02-25 Thread Mark Walters
Add a flag NOTMUCH_MESSAGE_FLAG_EXCLUDED which is set by notmuch_query_search_messages for excluded messages. Also add an option omit_excluded_messages to the search that we do not want the excludes at all. This exclude flag will be added to notmuch_query_search threads in the next patch. ---

Re: [PATCH v6 00/10] Reply improvements

2012-02-25 Thread Austin Clements
Quoth Adam Wolfe Gordon on Feb 25 at 3:25 pm: On Sat, Feb 25, 2012 at 14:29, David Bremner da...@tethera.net wrote: By coincidence I unpromoted it, perhaps because I was missing context. It seemed strange to push one patch from the middle of the series. I think that part was promoted by

[Patch v6 01/13] cli: add --no-exclude option to count and search.

2012-02-25 Thread Mark Walters
This option turns off the exclusion so all matching messages are returned. We do not need to add this to notmuch-show as that does not (yet) exclude. --- notmuch-count.c | 17 +++-- notmuch-search.c | 17 +++-- 2 files changed, 22 insertions(+), 12 deletions(-) diff

plans for 0.12

2012-02-25 Thread David Bremner
I would like to start a freeze (i.e. merge master to release) for 0.12 within the next week, say March 1 for an easy to remember date. If you have suggestions for things that should really go in, feel free to follow up. The main point here is bug fixes, or features which are there in a kindof

Re: [PATCH v2 2/2] emacs: support text/calendar mime type

2012-02-25 Thread David Bremner
On Thu, 9 Feb 2012 14:46:03 +, Jani Nikula j...@nikula.org wrote: Replace text/x-vcalendar with text/calendar, while maintaining support and backwards compatibility for text/x-vcalendar. pushed ___ notmuch mailing list notmuch@notmuchmail.org

Re: [PATCH] test: replace occurrences of $PWD with vars that are more stable

2012-02-25 Thread David Bremner
On Wed, 1 Feb 2012 21:37:21 +0100, Pieter Praet pie...@praet.org wrote: Thanks to Dmitry Kurochkin dmitry.kuroch...@gmail.com for pointing this out: id:87d39ymyb4@gmail.com Pushed. d ___ notmuch mailing list notmuch@notmuchmail.org

Re: [PATCH v6 00/10] Reply improvements

2012-02-25 Thread David Bremner
On Tue, 21 Feb 2012 23:46:29 -0700, Adam Wolfe Gordon awg+notm...@xvx.ca wrote: I noticed that the factoring out reply creation patch has been promoted to maybe ready in nmbug. I think this version is unchanged, but since the old one hasn't been pushed this one can probably replace it there.

Re: [PATCH v6 1/3] emacs: `notmuch-show-get-message-id': optionally return Message-Id sans prefix

2012-02-25 Thread David Bremner
On Sun, 19 Feb 2012 21:50:25 +0100, Pieter Praet pie...@praet.org wrote: * emacs/notmuch-show.el (notmuch-show-get-message-id): Add optional arg BARE. When non-nil, return a Message-Id without quotes and prefix, thus obviating the need to strip them off again in various

Re: [PATCH v3 0/3] Fix refreshing with state and make it the default

2012-02-25 Thread David Bremner
On Tue, 21 Feb 2012 10:42:30 -0500, Austin Clements amdra...@mit.edu wrote: Third time's the charm. This fixes show refresh to not mark anything read when keeping state while refreshing and makes it the default. Pushed. d ___ notmuch mailing list

Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations

2012-02-25 Thread David Bremner
On Fri, 24 Feb 2012 00:07:27 +0100, Pieter Praet pie...@praet.org wrote: On Wed, 08 Feb 2012 11:58:32 -0400, David Bremner da...@tethera.net wrote: On Sun, 5 Feb 2012 11:13:41 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: How about if '*' applies to all messages (as it

Re: [PATCH 1/2] NEWS: add entry for python 3.2 compatibility

2012-02-25 Thread David Bremner
On Wed, 22 Feb 2012 20:33:30 +0100, Justus Winter 4win...@informatik.uni-hamburg.de wrote: Signed-off-by: Justus Winter 4win...@informatik.uni-hamburg.de --- NEWS |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) Pushed both. d ___

[Patch v6 03/13] test: add tests for new cli --no-exclude option

2012-02-25 Thread Mark Walters
The tests test the new --no-exclude option to search and count. There were no existing tests for the exclude behaviour for count so added these too. --- test/count | 21 + test/search |5 + 2 files changed, 26 insertions(+), 0 deletions(-) diff --git a/test/count