[RFC] notmuch new : {blacklist,whitelist,exclud,ignor}ing

2011-03-12 Thread Pieter Praet
Hi all, First of all, *immense respect* to each and every one of you for making the legendary MUA-hopper stopper a reality! Emacs has been aching for this for a *very* long time. Much to my misfortune, I discovered Notmuch merely a year ago (after declaring sup a rather unflatteringly executed

[PATCH] test: search-by-folder single-word search terms should be updated correctly when directories are renamed.

2011-03-12 Thread Pieter Praet
On Mon, 17 Jan 2011 14:00:41 +0100, Sebastian Spaeth sebast...@sspaeth.de wrote: However, when moving files into another folder and rerunning notmuch new, the folder value is never updated, it seems. This appears to occur only when searching with a single-word folder path. ---

[PATCH] emacs: message-cited-text-face is obsolete since Emacs 22.1. Use message-cited-text instead.

2011-03-16 Thread Pieter Praet
--- emacs/notmuch-wash.el |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/emacs/notmuch-wash.el b/emacs/notmuch-wash.el index c4a7a41..0e64eb2 100644 --- a/emacs/notmuch-wash.el +++ b/emacs/notmuch-wash.el @@ -125,7 +125,7 @@ is what to put on the button. (let*

[PATCH 0/1] emacs: Use message-cited-text instead of message-cited-text-face

2011-03-20 Thread Pieter Praet
On Wed, 16 Mar 2011 08:20:24 -0300, David Bremner brem...@unb.ca wrote: [...] I notice there is no commit message in the body, so I suspect the chances of Carl applying it as-is are pretty remote. [...] David, Thanks for the heads up! Guess I should've read the patch submission guidelines

Re: (auto-)tagging sent messages

2011-04-16 Thread Pieter Praet
On Thu, 14 Apr 2011 10:03:12 +0200, Florian Friesdorf f...@chaoflow.net wrote: With my current setup (implying that it might be already possible with notmuch but I am not aware how), when sending a message it is put in /home/cfl/mail/sent via Fcc. This can be configured with the variable

Re: Optimization for notmuch tag by implicit filters

2011-04-16 Thread Pieter Praet
On Thu, 14 Apr 2011 10:23:46 +0200, Florian Friesdorf f...@chaoflow.net wrote: With 60k messages and 12k tagged as sent: $ time notmuch tag +sent -- from:f...@chaoflow.net real 0m8.561s user 0m8.069s sys 0m0.212s $ time notmuch tag +sent -- from:f...@chaoflow.net and not tag:sent

Re: signed/encrypted tagging in crypto branch

2011-04-16 Thread Pieter Praet
On Sun, 06 Mar 2011 11:15:00 -0800, Jameson Rollins jroll...@finestructure.net wrote: On Sat, 05 Mar 2011 00:26:46 -0800, Jameson Rollins jroll...@finestructure.net wrote: Hey, folks. I just pushed a couple of patches to my crypto branch [0] that add support for auto-tagging of

Re: Questions about importing mail (mbox)

2011-04-16 Thread Pieter Praet
On Mon, 21 Mar 2011 19:02:45 -0700, Mueen Nawaz mu...@nawaz.org wrote: I think you misunderstood me. A part of me suspects this has something to do with my not explaining myself, but who's to say?G Same here, apparently :D I'm experimenting with notmuch, and if I can translate everything I

Re: Problem with Unexpected output messages

2011-04-17 Thread Pieter Praet
On Fri, 15 Apr 2011 12:28:00 +0100, Andreas Amann a.am...@ucc.ie wrote: On Wed, 06 Apr 2011 20:23:17 +0100, Andreas Amann a.am...@ucc.ie wrote: since commit 44d3c57e (emacs: Display any unexpected output from notmuch search) I see a number of messages of the form Error: Unexpected

Re: (auto-)tagging sent messages

2011-04-18 Thread Pieter Praet
On Sun, 17 Apr 2011 19:17:40 -0400, Tom Prince tom.pri...@ualberta.net wrote: On 2011-04-17, Pieter Praet wrote: On Sat, 16 Apr 2011 15:23:44 -0400, Tom Prince tom.pri...@ualberta.net wrote: Further, for certain mails I sent (like this one ) I would like a WAITING tag (or similar

Re: Filtering out replies to a patch series

2011-04-18 Thread Pieter Praet
On Fri, 1 Apr 2011 11:47:25 +0300, Amit Kucheria amit.kuche...@verdurent.com wrote: Hi, I am wondering how developers on this list handle the case of grabbing a patch series posted to the list and using something like git-am to apply it to their local trees. See

Re: [PATCH 2/2] emacs: notmuch-hello: Add n and p to mod map

2011-04-18 Thread Pieter Praet
On Mon, 18 Apr 2011 09:36:45 +0200, Xavier Maillard xav...@maillard.im wrote: Sounds logical in theory, but in practice, you'll soon get stuck spamming npppnwtf in the search field. That, and TAB/S-TAB feels more natural. More natural than what exactly ? [...] More natural than

Re: [PATCH] use custom-face-edit value-type in notmuch-search-line-faces

2011-04-19 Thread Pieter Praet
On Sat, 26 Mar 2011 22:26:26 -0700, Jameson Rollins jroll...@finestructure.net wrote: On Sat, 26 Mar 2011 22:16:32 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: This enables the proper face customization UI for notmuch-search-line-faces. Hey, folks. amdragon was the

Re: notmuch painfully slow

2011-04-22 Thread Pieter Praet
On Fri, 22 Apr 2011 16:14:52 +0200, Xavier Maillard x...@gnu.org wrote: Hi, as time goes, notmuch is really becoming slower and slower. It may be normal as it has more and more messages to track but I am not sure it is that normal give these numbers: - notmuch version: notmuch

Re: notmuch painfully slow

2011-04-24 Thread Pieter Praet
On Sat, 23 Apr 2011 07:18:52 +0200, Florian Friesdorf f...@chaoflow.net wrote: [...] Is this the switch from xapian 1.0.x to 1.2.x or are further steps necessary? Are there instructions on how to do the switch? For the mailing list archive a link would be great - a quick search for 'xapian

Re: Optimization for notmuch tag by implicit filters

2011-04-27 Thread Pieter Praet
On Tue, 26 Apr 2011 23:31:09 +0200, Florian Friesdorf f...@chaoflow.net wrote: On Sat, 16 Apr 2011 14:59:34 +0200, Pieter Praet pie...@praet.org wrote: On Thu, 14 Apr 2011 10:23:46 +0200, Florian Friesdorf f...@chaoflow.net wrote: With 60k messages and 12k tagged as sent

Re: Problem with Unexpected output messages

2011-05-07 Thread Pieter Praet
---BeginMessage--- On Mon, 18 Apr 2011 09:10:45 +0100, Andreas Amann a.am...@ucc.ie wrote: On Sun, 17 Apr 2011 19:24:16 +0200, Pieter Praet pie...@praet.org wrote: On Fri, 15 Apr 2011 12:28:00 +0100, Andreas Amann a.am...@ucc.ie wrote: On Wed, 06 Apr 2011 20:23:17 +0100, Andreas Amann a.am

Re: release-candidate/0.6

2011-05-12 Thread Pieter Praet
On Tue, 10 May 2011 09:42:39 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: Arg. One last bit of churn. dkg found a bug in the new sanitize_string function that was causing segfaults on messages with empty headers. This is obviously an imprtant thing to fix. After

Re: [PATCH] test: search-by-folder single-word search terms should be updated correctly when directories are renamed.

2011-05-12 Thread Pieter Praet
On Sun, 13 Mar 2011 00:44:14 +0100, Pieter Praet pie...@praet.org wrote: On Mon, 17 Jan 2011 14:00:41 +0100, Sebastian Spaeth sebast...@sspaeth.de wrote: However, when moving files into another folder and rerunning notmuch new, the folder value is never updated, it seems. This appears

Re: release-candidate/0.6

2011-05-12 Thread Pieter Praet
On Thu, 12 May 2011 09:18:48 -0400, Austin Clements amdra...@mit.edu wrote: On Thu, May 12, 2011 at 8:22 AM, Pieter Praet pie...@praet.org wrote: The atomicity tests were failing here because I didn't have GDB installed, so I've added it as a prereq. Sorry, I've had a patch to address

[PATCH] test: do folder-based search with a single-term path after moving message

2011-05-13 Thread Pieter Praet
When moving mail to a different folder, there appear to be residual incorrect folder: terms. This test exposes an existing bug, so is currently failing. Signed-off-by: Pieter Praet pie...@praet.org --- test/search-by-folder |4 1 files changed, 4 insertions(+), 0 deletions(-) diff

[PATCH 0/4] set test prereqs (Emacs, GDB, GPG)

2011-05-13 Thread Pieter Praet
test_expect_equal* tests [2] - [PATCH] test: Report test failures from test_expect_* [3] Pieter Praet (4): test: add 'GDB' prereq to 'atomicity' tests test: add 'GPG' prereq to 'crypto' tests test: add 'Emacs' prereq to 'emacs' tests test: add 'Emacs' prereq to 'emacs-large-search-buffer

[PATCH 1/4] test: add 'GDB' prereq to 'atomicity' tests

2011-05-13 Thread Pieter Praet
Signed-off-by: Pieter Praet pie...@praet.org --- test/atomicity | 10 -- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/test/atomicity b/test/atomicity index cd58c4c..78280b2 100755 --- a/test/atomicity +++ b/test/atomicity @@ -7,6 +7,12 @@ test_description='atomicity

[PATCH 2/4] test: add 'GPG' prereq to 'crypto' tests

2011-05-13 Thread Pieter Praet
Signed-off-by: Pieter Praet pie...@praet.org --- test/crypto | 30 ++ 1 files changed, 18 insertions(+), 12 deletions(-) diff --git a/test/crypto b/test/crypto index 961d035..c2f381c 100755 --- a/test/crypto +++ b/test/crypto @@ -7,6 +7,12 @@ test_description='PGP

[PATCH 3/4] test: add 'Emacs' prereq to 'emacs' tests

2011-05-13 Thread Pieter Praet
Signed-off-by: Pieter Praet pie...@praet.org --- test/emacs | 40 +++- 1 files changed, 23 insertions(+), 17 deletions(-) diff --git a/test/emacs b/test/emacs index 3264bf2..011deac 100755 --- a/test/emacs +++ b/test/emacs @@ -2,6 +2,12 @@ test_description

[PATCH] emacs: run notmuch-message-mark-replied on message-sent-hook, not message-send-hook

2011-05-13 Thread Pieter Praet
-by: Pieter Praet pie...@praet.org --- emacs/notmuch-message.el |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-message.el b/emacs/notmuch-message.el index d5c96c2..c8b19c3 100644 --- a/emacs/notmuch-message.el +++ b/emacs/notmuch-message.el @@ -47,6 +47,6

[PATCH 1/3] test: emacs: add subtest Send reply from within Emacs

2011-05-13 Thread Pieter Praet
Renamed subtest Reply within emacs to Compose reply in emacs as it doesn't really do a reply, but rather populates a reply buffer and then princ's it. ... and couldn't possibly justify that without writing an actual reply test. Signed-off-by: Pieter Praet pie...@praet.org --- test/emacs | 33

[PATCH 2/3] test: emacs: add subtest Verify that 'replied' tag is added to reply's parent message.

2011-05-13 Thread Pieter Praet
Signed-off-by: Pieter Praet pie...@praet.org --- test/emacs |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/test/emacs b/test/emacs index 18d8322..c0fba52 100755 --- a/test/emacs +++ b/test/emacs @@ -142,6 +142,10 @@ On Fri, 29 Mar 1974 10:00:00 -, Notmuch Test

[PATCH 3/3] emacs: generalize notmuch-message-mark-replied

2011-05-13 Thread Pieter Praet
Renamed 'notmuch-message-mark-replied' to 'notmuch-message-mod-tags'. Previously (semi)hardcoded vars are now to be provided as arguments, allowing much more flexibility re autotagging in message-mode. Paving the way towards proper draft handling? Signed-off-by: Pieter Praet pie...@praet.org

[PATCH 0/4] set test prereqs (Emacs, GDB, GPG) v3

2011-06-02 Thread Pieter Praet
Rebased the set test prereqs (Emacs, GDB, GPG) [1] patch series to Jameson's release-candidate/0.6 branch (0c0b4172). The GDB dependency for the atomicity tests was dropped since Austin has already included this in his amdragon/atomic-new-v4 branch [2], which has been included in

[PATCH 1/4] test: add 'GnuPG' prereq to dependent 'crypto' tests

2011-06-02 Thread Pieter Praet
Adds a new test that checks for the presence of 'gpg', and adds that test as a prereq to all subsequent tests that rely on GnuPG. This causes tests with unmet dependencies to be skipped. Signed-off-by: Pieter Praet pie...@praet.org --- test/crypto | 33 +++-- 1

[PATCH 2/4] test: add 'Emacs' prereq to dependent 'crypto' tests

2011-06-02 Thread Pieter Praet
Adds a new test that checks for the presence of 'emacs', and adds that test as a prereq to all subsequent tests that rely on Emacs. This causes tests with unmet dependencies to be skipped. Signed-off-by: Pieter Praet pie...@praet.org --- test/crypto | 17 ++--- 1 files changed, 14

[PATCH 3/4] test: add 'Emacs' prereq to dependent 'emacs' tests

2011-06-02 Thread Pieter Praet
Adds a new test that checks for the presence of 'emacs', and adds that test as a prereq to all subsequent tests that rely on Emacs. This causes tests with unmet dependencies to be skipped. Signed-off-by: Pieter Praet pie...@praet.org --- test/emacs | 43

Re: bug in emacs-ui ?

2011-06-20 Thread Pieter Praet
On Mon, 20 Jun 2011 12:53:37 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: Non-text part: multipart/mixed Non-text part: multipart/signed On Mon, 20 Jun 2011 18:43:01 +0200, Sebastien Binet seb.bi...@gmail.com wrote: You didn't try forwarding the email to the list ;) ah!

[PATCH] fix sum moar typos

2011-06-20 Thread Pieter Praet
Various typo fixes in docs, docstrings, comments, etc... Signed-off-by: Pieter Praet pie...@praet.org --- Rebased to current master (12d6f90e) NEWS | 10 +- TODO |4 ++-- compat/README|2 +- completion/Makefile

Re: [PATCH] fix sum moar typos

2011-06-24 Thread Pieter Praet
On Thu, 23 Jun 2011 16:22:27 -0700, Carl Worth cwo...@cworth.org wrote: Non-text part: multipart/signed On Mon, 20 Jun 2011 22:14:21 +0200, Pieter Praet pie...@praet.org wrote: Various typo fixes in docs, docstrings, comments, etc... Signed-off-by: Pieter Praet pie...@praet.org Thanks

Re: [PATCH] test: remove useless test_emacs call from an emacs FCC test

2011-06-26 Thread Pieter Praet
On Thu, 23 Jun 2011 22:15:25 -0300, David Bremner da...@tethera.net wrote: Non-text part: multipart/mixed Non-text part: multipart/signed [...] I'm going to skip the typo fixes for 0.6, unless convinced otherwise. Well, the whole point of my de-typo-ifying sprint was to make 0.6 a ~typo-free

Re: [PATCH] test:Improve test behaviors when --root is used

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 16:50:47 -0400, Austin Clements amdra...@mit.edu wrote: This looks great (modulo one bug, below). I've wanted to run the tests on tmpfs before, but was too lazy to actually fix the tests. Given how easy it is to accidentally use .., I wonder if there's a way to force

Re: Debian package not building

2011-06-28 Thread Pieter Praet
On Fri, 24 Jun 2011 16:34:02 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: Non-text part: multipart/mixed Non-text part: multipart/signed Hey, folks. As of today I am for some reason no longer able to build the Notmuch Debian package. I'm using the same build technique I have

Re: [PATCH] test:Folder tags shouldn't match after removal of file in given folder

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 11:12:24 -0600, Mark Anderson ma.sk...@gmail.com wrote: Test for bug. Current stemming support for notmuch adds extra terms to the DB which aren't removed when the file renames are detected. When folder tags are added to a message, Xapian terms for both XFOLDER and

Re: Race condition for '*' command

2011-06-28 Thread Pieter Praet
On Sun, 26 Jun 2011 10:00:41 +0100, Robin Green gree...@greenrd.org wrote: On Sat, 25 Jun 2011 16:57:50 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: On Sat, 25 Jun 2011 23:18:52 +0100, Robin Green gree...@greenrd.org wrote: A race condition in the '*' command was noted

Re: [PATCH] fix sum moar typos

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 14:17:27 -0700, Carl Worth cwo...@cworth.org wrote: Non-text part: multipart/signed On Fri, 24 Jun 2011 22:04:13 +0200, Pieter Praet pie...@praet.org wrote: You however, seem to be incredibly disciplined in this respect, so statistically speaking, you're destined to become

Re: Drafts being tagged as inbox

2011-06-28 Thread Pieter Praet
On Tue, 28 Jun 2011 08:00:53 +0100, Robin Green gree...@greenrd.org wrote: On Tue, 28 Jun 2011 08:34:44 +0200, Pieter Praet pie...@praet.org wrote: notmuch tag -inbox +draft -- folder:${where_your_drafts_reside} My notmuch help does not mention folder:, and indeed folder:drafts as a search

Re: Notmuch scripts

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 21:31:26 -0400, Ben Gamari bgamari.f...@gmail.com wrote: [SNIP] It would be nice, however, if the emacs interface supported hiding tags matching certain patterns (say /\..+/). This should be possible as of last month (eb4e0ea2), by courtesy of Daniel Schoepe [1]. - Ben

Re: [PATCH v3] test:Improve test behaviors when --root is used

2011-06-29 Thread Pieter Praet
On Tue, 28 Jun 2011 16:03:00 -0700, Carl Worth cwo...@cworth.org wrote: Non-text part: multipart/mixed Non-text part: multipart/signed On Tue, 28 Jun 2011 16:11:32 -0600, Mark Anderson ma.sk...@gmail.com wrote: Change add_email_corpus, emacs_deliver_message and tests to use $TEST_DIRECTORY

[PATCH] test: revert non-intentional changes introduced in eb4cf465

2011-06-29 Thread Pieter Praet
: PASS Message with .. in Message-Id: instead of: PASS Message with .. in Message-Id: PASS Sending a message via (fake) SMTP The second one is simply the insertion of a line break, so no harm there. This commit reverts both changes, as they were clearly accidental. Signed-off-by: Pieter

Re: notmuch Digest, Vol 20, Issue 57

2011-06-30 Thread Pieter Praet
On Wed, 29 Jun 2011 16:19:44 -0600, Mark Anderson ma.sk...@gmail.com wrote: On Wed, 29 Jun 2011 13:54:40 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: On Wed, 29 Jun 2011 14:21:11 -0600, Mark Anderson ma.sk...@gmail.com wrote: I personally prefer --output=files remain

Re: Preventing the user shooting themself in the foot

2011-06-30 Thread Pieter Praet
On Wed, 29 Jun 2011 22:40:07 -0700, Carl Worth cwo...@cworth.org wrote: Non-text part: multipart/mixed Non-text part: multipart/signed On Thu, 30 Jun 2011 13:04:23 +1000, Brian May br...@microcomaustralia.com.au wrote: On 30 June 2011 08:40, Carl Worth cwo...@cworth.org wrote: The 'a'

Re: Preventing the user shooting themself in the foot

2011-06-30 Thread Pieter Praet
On Wed, 29 Jun 2011 19:53:03 -0400, Austin Clements amdra...@mit.edu wrote: Non-text part: multipart/mixed Non-text part: multipart/alternative I've spent embarrassingly little time in the emacs UI, so my opinions on this should be taken lightly, but I feel like all of the bindings are of the

Re: [PATCH 4/4] emacs: Add pseudo tag to match all messages that have no tags

2011-06-30 Thread Pieter Praet
On Tue, 28 Jun 2011 07:31:32 +, Jani Nikula j...@nikula.org wrote: Add a pseudo tag that matches all the messages that have no tags. Add new customization option notmuch-tags-nomatch to enable and name the pseudo tag. --- emacs/notmuch-hello.el | 18 -- 1 files

[PATCH] remove prefixes from `--output={threads,messages}' results

2011-06-30 Thread Pieter Praet
it faster than Windoze can BSOD), but the patch will be so huge and disruptive that even Ubuntu devs might consider frowning disapprovingly. Signed-off-by: Pieter Praet pie...@praet.org --- notmuch-search.c |4 ++-- 1 files

[PATCH] add `tag:' prefix to `--output=tags' results

2011-06-30 Thread Pieter Praet
Alter `do_search_tags()' to prepend each result with `tag:', and update affected test. This makes its output consistent with `do_search_threads()' and `do_search_messages()'. See this discussion: id:871v6b79s0@yoom.home.cworth.org Signed-off-by: Pieter Praet pie...@praet.org --- notmuch

[PATCH] emacs: add keybind and function to stash Message-ID without prefix

2011-06-30 Thread Pieter Praet
of 1. Signed-off-by: Pieter Praet pie...@praet.org --- emacs/notmuch-show.el |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index 6685717..a703732 100644 --- a/emacs/notmuch-show.el +++ b/emacs/notmuch-show.el @@ -858,6

[PATCH] test: stashing in notmuch-{show,search}

2011-06-30 Thread Pieter Praet
Should provide full test coverage of the stashing feature. Signed-off-by: Pieter Praet pie...@praet.org --- test/emacs| 41 + test/emacs.expected-output/emacs-stashing |9 ++ 2 files changed, 50 insertions(+), 0 deletions

[PATCH] fix breakage in `notmuch-hello-generate-tag-alist' due to `tag:' prefix

2011-06-30 Thread Pieter Praet
...@praet.org Signed-off-by: Pieter Praet pie...@praet.org --- emacs/notmuch-hello.el |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el index 65fde75..4551be1 100644 --- a/emacs/notmuch-hello.el +++ b/emacs/notmuch-hello.el @@ -343,6

[PATCH] fix breakage in `notmuch-select-tag-with-completion' due to `tag:' prefix

2011-06-30 Thread Pieter Praet
...@praet.org Signed-off-by: Pieter Praet pie...@praet.org --- emacs/notmuch.el |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch.el b/emacs/notmuch.el index f11ec24..af66510 100644 --- a/emacs/notmuch.el +++ b/emacs/notmuch.el @@ -77,7 +77,11 @@ For example

Re: Preventing the user shooting themself in the foot

2011-06-30 Thread Pieter Praet
On Thu, 30 Jun 2011 09:50:27 +0200, Pieter Praet pie...@praet.org wrote: [...] And AFAIK, Archive does *not* mark a message as read in GMail. (see previous messages suggesting the inverse [...] ... be implemented in Notmuch. [...] ) Nobody claimed the inverse was the case in GMail, so I

[PATCH 1/2] test: add/remove tags from all matching messages with `notmuch-search-operate-all'

2011-07-01 Thread Pieter Praet
This test is meant solely to make sure my next patch doesn't break anything (in an obvious way, at least). In other words: it does *not* demonstrate a race condition issue. Signed-off-by: Pieter Praet pie...@praet.org --- Robin, Could you provide a test to expose the actual race condition

Re: [PATCH] remove prefixes from `--output={threads, messages}' results

2011-07-01 Thread Pieter Praet
On Thu, 30 Jun 2011 09:24:12 -0700, Carl Worth cwo...@cworth.org wrote: Non-text part: multipart/signed On Thu, 30 Jun 2011 10:19:49 +0200, Pieter Praet pie...@praet.org wrote: Alter `do_search_threads()' and `do_search_messages()' to not prepend each result with `thread:' respectively `id

Re: [PATCH 2/2] [RFC] possible solution for Race condition for '*' command

2011-07-01 Thread Pieter Praet
On Thu, 30 Jun 2011 18:08:28 +0200, Pieter Praet pie...@praet.org wrote: `notmuch-search-operate-all' may cause a race condition because repeatedly running `notmuch-tag' with the *original* query string makes the result list a moving target. One approach to resolving this, is to feed

[PATCH 2/2] [RFC] possible solution for Race condition for '*' command

2011-07-01 Thread Pieter Praet
of the latter itself. See discussion @ id:86d3i1d06r@dragonfly.greenrd.org Signed-off-by: Pieter Praet pie...@praet.org --- Carl, I've gone along a different route which assures only matched messages are touched, but it does come with quite a performance hit. Since there isn't a test

Re: Preventing the user shooting themself in the foot

2011-07-01 Thread Pieter Praet
On Fri, 01 Jul 2011 09:26:48 +1200, Michael Hudson-Doyle michael.hud...@canonical.com wrote: On Wed, 29 Jun 2011 22:40:07 -0700, Carl Worth cwo...@cworth.org wrote: Non-text part: multipart/mixed Non-text part: multipart/signed not sure why notmuch reply is putting that there :) The

Re: Preventing the user shooting themself in the foot

2011-07-01 Thread Pieter Praet
On Fri, 1 Jul 2011 11:28:10 +1000, Brian May br...@microcomaustralia.com.au wrote: On 30 June 2011 17:50, Pieter Praet pie...@praet.org wrote: And AFAIK, Archive does *not* mark a message as read in GMail. (see previous messages suggesting the inverse) gmail will mark all messages

Re: [PATCH 2/2] [RFC] possible solution for Race condition for '*' command

2011-07-02 Thread Pieter Praet
On Fri, 1 Jul 2011 12:37:11 -0400, Austin Clements amdra...@mit.edu wrote: Non-text part: multipart/alternative On Jul 1, 2011 10:55 AM, Austin Clements amdra...@mit.edu wrote: On Thu, Jun 30, 2011 at 3:38 PM, Pieter Praet pie...@praet.org wrote: Ok, even though my very first reply [1] may

Re: problem with message/rfc822 parts

2011-07-02 Thread Pieter Praet
On Sat, 02 Jul 2011 02:23:34 +0100, Andreas Amann a.am...@ucc.ie wrote: It's a known problem. As a short term fix, try the patches of id:1307320169-29905-1-git-send-email-jroll...@finestructure.net Or use `notmuch-show-view-raw-message' (bound to 'V'). thanks a lot David. I can

Re: Possible bug: wrong from address when forwarding

2011-07-02 Thread Pieter Praet
--- Either enable `notmuch-always-prompt-for-sender', or apply this quick-fix patch, which uses `name' and `primary_email' as configured in your .notmuch-config if aforementioned isn't enabled. emacs/notmuch-mua.el |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git

[PROTO] possible solution for Race condition for '*' command

2011-07-04 Thread Pieter Praet
Thanks Austin! Unfortunately, your patch causes *all* Message-Id's in the thread to be appended, as opposed to only the ones matching the query: #+BEGIN_EXAMPLE $ notmuch search tag:inbox AND from:amdra...@mit.edu thread:2777 Yest. 19:17 [1/3] Austin Clements| Pieter Praet; [PATCH 2

[PATCH 1/5] emacs: add property matched-msgids to each search result

2011-07-04 Thread Pieter Praet
Signed-off-by: Pieter Praet pie...@praet.org --- emacs/notmuch.el |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch.el b/emacs/notmuch.el index f11ec24..674deb7 100644 --- a/emacs/notmuch.el +++ b/emacs/notmuch.el @@ -801,13 +801,14 @@ non-authors is found

[PATCH 3/5] emacs: stashing (a region of) matched-msgids

2011-07-04 Thread Pieter Praet
Signed-off-by: Pieter Praet pie...@praet.org --- emacs/notmuch.el | 14 ++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/emacs/notmuch.el b/emacs/notmuch.el index 2338044..46e276a 100644 --- a/emacs/notmuch.el +++ b/emacs/notmuch.el @@ -226,6 +226,7 @@ For a mouse

[PATCH 4/5] test: emacs: add/remove tags from all matching messages with `notmuch-search-operate-all'

2011-07-04 Thread Pieter Praet
Signed-off-by: Pieter Praet pie...@praet.org --- test/emacs-search-operate-all | 29 + 1 files changed, 29 insertions(+), 0 deletions(-) create mode 100755 test/emacs-search-operate-all diff --git a/test/emacs-search-operate-all b/test/emacs-search-operate-all new

[PATCH 5/5] emacs: make `notmuch-search-operate-all' use matched-msgids instead of the original query string

2011-07-04 Thread Pieter Praet
Signed-off-by: Pieter Praet pie...@praet.org --- emacs/notmuch.el |7 +-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/emacs/notmuch.el b/emacs/notmuch.el index 46e276a..0d040a2 100644 --- a/emacs/notmuch.el +++ b/emacs/notmuch.el @@ -869,7 +869,10 @@ Each character

Re: [PATCH] emacs: add insisible space after the search widget field in notmuch-hello

2011-07-04 Thread Pieter Praet
On Mon, 4 Jul 2011 10:47:05 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: It is very convenient when C-e (bound to `widget-end-of-line') ignores trailing spaces inside the search widget. But it only does so if a widget is not followed by a newline (that is why it works in the

Re: [PATCH] emacs: add insisible space after the search widget field in notmuch-hello

2011-07-04 Thread Pieter Praet
On Mon, 04 Jul 2011 11:37:06 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: On Mon, 04 Jul 2011 09:24:17 +0200, Pieter Praet pie...@praet.org wrote: On Mon, 4 Jul 2011 10:47:05 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: It is very convenient when C-e (bound

Re: [PATCH] emacs: add insisible space after the search widget field in notmuch-hello

2011-07-04 Thread Pieter Praet
Besides, nobody commented on it except a noob like myself. Just by keeping half an eye on all that gets merged, imagining what a maintainer's job must be like sends shivers down my spine. ^ The following was sent in true top-posting style, for your personal bewilderment. Please stand on

Re: [PATCH 2/2] emacs: skip forward to visible text in notmuch-show-message-extent

2011-07-04 Thread Pieter Praet
-message-extent) (defun notmuch-show-message-top () (car (notmuch-show-message-extent))) -- 1.7.5.4 ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch Signed-off-by: Pieter Praet pie

Re: [PATCH v2] emacs: add insisible space after the search widget field in notmuch-hello

2011-07-04 Thread Pieter Praet
On Mon, 04 Jul 2011 13:08:43 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: On Mon, 04 Jul 2011 10:55:21 +0200, Pieter Praet pie...@praet.org wrote: On Mon, 4 Jul 2011 12:27:04 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: It is very convenient when C-e (bound

Re: notmuch 0.6: notmuch new dies with std::terminate

2011-07-04 Thread Pieter Praet
On Mon, 04 Jul 2011 09:20:25 -0300, David Bremner da...@tethera.net wrote: On Mon, 04 Jul 2011 11:58:24 +0100, Robin Green gree...@greenrd.org wrote: Some time after upgrading to notmuch 0.6, notmuch new suddenly started failing, with this backtrace: (this is on DragonFly BSD) Program

Re: [PROTO] possible solution for Race condition for '*' command

2011-07-04 Thread Pieter Praet
). Can't think of a use case for non-matched messages right now, but if required, we'll just use `set-exclusive-or'. -- Sent from my Android. Please excuse my brevity. Tip: K-9 Mail [1], apart from being much better than the stock email client, supports bottom-posting :) Pieter Praet pie

[PATCH] emacs: logically group def{custom,face}s

2011-07-05 Thread Pieter Praet
To allow for expansion whilst keeping everything tidy and organized, move all defcustom/defface variables to the following subgroups, defined in notmuch-lib.el: - Hello - Search - Show - Send - Crypto - Hooks - Appearance - External Commands Signed-off-by: Pieter Praet pie...@praet.org

Re: [PATCH v2] emacs: Turn id:message-id elements into buttons for notmuch searches

2011-07-07 Thread Pieter Praet
-by: Pieter Praet pie...@praet.org Fantastic usability improvement Daniel! Peace -- Pieter ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

Extend test coverage wrt tagging

2011-07-10 Thread Pieter Praet
Some tagging-related tests and a (soon to be deprecated) bugfix, which should be merged in *before* the (upcoming) race condition patch series [1], to prevent surprises of the non-enjoyable kind. Peace -- Pieter [1] id:CAH-f9WticM4EN8F1_ik_-mcBcBtrXwSpO+Drbtp7=un7mce...@mail.gmail.com

[PATCH 4/6] test: emacs: add test for `notmuch-search-operate-all'

2011-07-10 Thread Pieter Praet
`notmuch-search-operate-all' (bound to *) adds and removes tags to/from all messages which match the query used to populate the current search buffer. Signed-off-by: Pieter Praet pie...@praet.org --- test/emacs-search-operate-all | 29 + test/notmuch-test

[PATCH 5/6] test: emacs: `notmuch-search-operate-all' should operate on threads, not messages

2011-07-10 Thread Pieter Praet
In order to be consistent with `notmuch-search-{add,remove}-tag' (+/-), `notmuch-search-operate-all' (*) should operate on matching threads instead of matching messages. This test exposes an existing bug, so is currently failing. It should be fixed in the next commit. Signed-off-by: Pieter Praet

Re: [PATCH v2] emacs: bad regexp @ `notmuch-search-process-filter'

2011-07-13 Thread Pieter Praet
On Mon, 11 Jul 2011 17:05:32 -0400, Austin Clements amdra...@mit.edu wrote: Quoth Pieter Praet on Jul 11 at 10:43 pm: TL;DR: I can haz regex pl0x? Oof, what a pain. I'm happy to change the output format of search; I hadn't realized how difficult it would be to parse. In fact, I'm

Re: bug? notmuch earch doesn't search threads

2011-07-16 Thread Pieter Praet
On Sat, 16 Jul 2011 08:37:52 -0400, Jason Woofenden ja...@jasonwoof.com wrote: I added tag:me to all messages from me. Then I added this to my index page: tag:notmuch and tag:unread and tag:me So my attention could be drawn to threads I'd participated in, which had new messages. (This

Re: bug? notmuch earch doesn't search threads

2011-07-16 Thread Pieter Praet
On Sat, 16 Jul 2011 16:26:04 +0200, Pieter Praet pie...@praet.org wrote: [SNIP] The Notmuch project is in the unique position of being the product of Carl's (legimitate) dislike of email (see his talk @ LCA2011 [1]), so it's safe to say patches may take some time to pass the gatekeeper. Don't

Re: [PATCH v2] emacs: bad regexp @ `notmuch-search-process-filter'

2011-07-16 Thread Pieter Praet
On Wed, 13 Jul 2011 14:57:21 -0400, Austin Clements amdra...@mit.edu wrote: Quoth Pieter Praet on Jul 13 at 4:16 pm: On Mon, 11 Jul 2011 17:05:32 -0400, Austin Clements amdra...@mit.edu wrote: Quoth Pieter Praet on Jul 11 at 10:43 pm: TL;DR: I can haz regex pl0x? Oof, what

set test prereqs (Emacs, GDB, GPG) v4

2011-11-01 Thread Pieter Praet
Rebased to current master. Previous version: id:1307016220-17509-1-git-send-email-pie...@praet.org Discussion: id:1317660447-27520-1-git-send-email-schno...@schnouki.net ___ notmuch mailing list notmuch@notmuchmail.org

[PATCH 1/4] test: add 'GnuPG' prereq to dependent 'crypto' tests

2011-11-01 Thread Pieter Praet
Adds a new test that checks for the presence of 'gpg', and adds that test as a prereq to all subsequent tests that rely on GnuPG. This causes tests with unmet dependencies to be skipped. --- test/crypto | 33 +++-- 1 files changed, 19 insertions(+), 14 deletions(-)

[PATCH 2/4] test: add 'Emacs' prereq to dependent 'crypto' tests

2011-11-01 Thread Pieter Praet
Adds a new test that checks for the presence of 'emacs', and adds that test as a prereq to all subsequent tests that rely on Emacs. This causes tests with unmet dependencies to be skipped. --- test/crypto | 17 ++--- 1 files changed, 14 insertions(+), 3 deletions(-) diff --git

[PATCH 3/4] test: add 'Emacs' prereq to dependent 'emacs' tests

2011-11-01 Thread Pieter Praet
Adds a new test that checks for the presence of 'emacs', and adds that test as a prereq to all subsequent tests that rely on Emacs. This causes tests with unmet dependencies to be skipped. --- test/emacs | 65 --- 1 files changed, 35

[PATCH 4/4] test: add 'Emacs' prereq to dependent 'emacs-large-search-buffer' tests

2011-11-01 Thread Pieter Praet
Adds a new test that checks for the presence of 'emacs', and adds that test as a prereq to all subsequent tests that rely on Emacs. This causes tests with unmet dependencies to be skipped. --- test/emacs-large-search-buffer |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-)

Re: [PATCH 00/13] Test prereqs and screen-based Emacs tests

2011-11-01 Thread Pieter Praet
: 242/247 tests passed. 2 broken tests failed as expected. 3 tests failed. (The 3 failed tests come from some trouble with Emacs 24, I'll try to fix this later.) *Many* thanks to Dmitry Kurochkin and Pieter Praet for their work! Thanks to you as well! All this duplication of effort

Re: [PATCH 03/13] test: add 'Emacs' prereq to dependent 'crypto' tests

2011-11-01 Thread Pieter Praet
On Mon, 3 Oct 2011 18:47:17 +0200, Thomas Jost schno...@schnouki.net wrote: [...] The SCREEN prereq would preferrably be added in a separate commit. Also, you appear to have given *every* test the EMACS+GPG prereq, while only the ones using `emacs_deliver_message' require EMACS. Peace --

Re: [PATCH 04/13] test: add 'Emacs' prereq to dependent 'emacs' tests

2011-11-01 Thread Pieter Praet
On Mon, 3 Oct 2011 18:47:18 +0200, Thomas Jost schno...@schnouki.net wrote: [...] Nice catches @ - Sending a message via (fake) SMTP - Hiding message following HTML part (though I think the latter test wasn't there yet when I submitted my patch series) New issues: - The SCREEN prereq

Re: [PATCH] emacs: logically group def{custom,face}s

2011-11-01 Thread Pieter Praet
On Tue, 18 Oct 2011 08:18:18 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: On Mon, 10 Oct 2011 15:49:03 +0200, Daniel Schoepe dan...@schoepe.org wrote: On Tue, 5 Jul 2011 20:33:00 +0200, Pieter Praet pie...@praet.org wrote: To allow for expansion whilst keeping everything

Re: [PATCH 04/13] test: add 'Emacs' prereq to dependent 'emacs' tests

2011-11-01 Thread Pieter Praet
On Tue, 01 Nov 2011 20:57:59 +0100, Pieter Praet pie...@praet.org wrote: On Mon, 3 Oct 2011 18:47:18 +0200, Thomas Jost schno...@schnouki.net wrote: [...] - @ Verify that sent messages are saved/searchable (via FCC): This doesn't need an EMACS prereq. [...] My mistake, scrap

Re: [PATCH 03/13] test: add 'Emacs' prereq to dependent 'crypto' tests

2011-11-01 Thread Pieter Praet
On Tue, 01 Nov 2011 20:56:17 +0100, Pieter Praet pie...@praet.org wrote: On Mon, 3 Oct 2011 18:47:17 +0200, Thomas Jost schno...@schnouki.net wrote: [...] Also, you appear to have given *every* test the EMACS+GPG prereq, while only the ones using `emacs_deliver_message' require EMACS

Re: [PATCH 1/4] test: add 'GnuPG' prereq to dependent 'crypto' tests

2011-11-02 Thread Pieter Praet
On Tue, 01 Nov 2011 14:17:51 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: On Tue, 1 Nov 2011 20:49:11 +0100, Pieter Praet pie...@praet.org wrote: -test_expect_success 'emacs delivery of signed message' \ +test_expect_success GPG 'emacs delivery of signed message' \ Hi

Re: set test prereqs (Emacs, GDB, GPG) v4

2011-11-02 Thread Pieter Praet
On Tue, 1 Nov 2011 22:20:25 +0200, Ali Polatel pola...@gmail.com wrote: On Tue, Nov 01, 2011 at 08:49:10PM +0100, Pieter Praet wrote: Rebased to current master. Previous version: id:1307016220-17509-1-git-send-email-pie...@praet.org Discussion: id:1317660447-27520-1-git-send-email

  1   2   3   4   5   6   7   >