Re: release-candidate/0.6

2011-05-13 Thread Jameson Graef Rollins
On Thu, 12 May 2011 15:36:43 -0700, Carl Worth cwo...@cworth.org wrote: Does anyone want to attempt to fix up this first patch? (It doesn't necessarily have to be David). Hi, Carl. I went through dme's multipart patch series and cleaned things up. I split up that first commit into a couple of

[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
The atomicity tests were failing here because I didn't have GDB installed, so I've added it as a prereq. While I was at it, I fixed the crypto and emacs tests as well. Thank Austin for providing me with a hammer (I was using a brick) [1]. (!) Depends: - [PATCH] test: Fix message when skipping

[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 @@

[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 @@

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

2011-05-13 Thread Pieter Praet
Parent message is tagged replied prematurely. Only do so when reply is *really* sent. (describe-variable 'message-send-hook) Hook run before sending messages. This hook is run quite early when sending. (describe-variable 'message-sent-hook) Hook run after sending messages. Signed-off-by:

[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

Re: Warning when GMime is parsing broken email addresses

2011-05-13 Thread Matthieu Lemerre
On Wed, 27 Apr 2011 21:59:09 +0200, Pieter Praet pie...@praet.org wrote: On Wed, 27 Apr 2011 18:30:09 +0200, Xavier Maillard xav...@maillard.im wrote: On Mon, 25 Apr 2011 15:23:41 -0700, Carl Worth cwo...@cworth.org wrote: On Wed, 17 Nov 2010 23:20:26 +0100, Matthieu Lemerre ra...@free.fr

release-candidate/0.6

2011-05-13 Thread Jameson Graef Rollins
e: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110513/cd3cbe1b/attachment.pgp>

[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 --- test/search-by-folder |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git

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

2011-05-13 Thread Pieter Praet
The atomicity tests were failing here because I didn't have GDB installed, so I've added it as a prereq. While I was at it, I "fixed" the crypto and emacs tests as well. Thank Austin for providing me with a hammer (I was using a brick) [1]. (!) Depends: - "[PATCH] test: Fix message when

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

2011-05-13 Thread Pieter Praet
Signed-off-by: Pieter Praet --- 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' # final database

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

2011-05-13 Thread Pieter Praet
Signed-off-by: Pieter Praet --- 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/MIME signature

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

2011-05-13 Thread Pieter Praet
Signed-off-by: Pieter Praet --- test/emacs-large-search-buffer |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/test/emacs-large-search-buffer b/test/emacs-large-search-buffer index 2751a32..54b1827 100755 --- a/test/emacs-large-search-buffer +++

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

2011-05-13 Thread Pieter Praet
Signed-off-by: Pieter Praet --- 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="emacs

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

2011-05-13 Thread Pieter Praet
Parent message is tagged "replied" prematurely. Only do so when reply is *really* sent. (describe-variable 'message-send-hook) > Hook run before sending messages. > This hook is run quite early when sending. (describe-variable 'message-sent-hook) > Hook run after sending messages.

[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 --- 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 --- 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 Suite /dev/null #

[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 ---