Re: [PATCH] emacs: fix MML quoting in replies

2012-03-14 Thread Jani Nikula
On Mar 15, 2012 12:08 AM, "Austin Clements" wrote: > > On Sun, 4 Mar 2012 10:25:38 +0200, Jani Nikula wrote: > > The reply MML quoting added in commit ae438cc unintentionally MML > > quotes also the signature/encryption MML tags added via > > message-setup-hook, causing the reply not to be signe

[PATCH 6/6] NEWS: cli: manual page for notmuch configuration options

2012-03-14 Thread Jani Nikula
--- bb8ab4c man: document the notmuch configuration settings in notmuch-config(1) --- NEWS |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index b2c07ab..09c5d9c 100644 --- a/NEWS +++ b/NEWS @@ -20,6 +20,12 @@ Mail store folder/file ignore NOTE: *Every

[PATCH 5/6] NEWS: emacs: enable `notmuch-search-line-faces' by default

2012-03-14 Thread Jani Nikula
--- e06943f emacs: add default value to notmuch-search-line-faces --- NEWS |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index 83d764a..b2c07ab 100644 --- a/NEWS +++ b/NEWS @@ -95,6 +95,12 @@ Generate inline patch fake attachment file names from message

[PATCH 4/6] NEWS: emacs: inline patch fake attachment file names

2012-03-14 Thread Jani Nikula
--- f893d31 emacs: create patch filename from subject for inline patch fake parts --- NEWS |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index 3c2cd82..83d764a 100644 --- a/NEWS +++ b/NEWS @@ -88,6 +88,13 @@ Support text/calendar MIME type The text/c

[PATCH 3/6] NEWS: emacs: support text/calendar MIME type

2012-03-14 Thread Jani Nikula
--- cfdc9a4 emacs: support text/calendar mime type --- NEWS |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index 86c0690..3c2cd82 100644 --- a/NEWS +++ b/NEWS @@ -83,6 +83,11 @@ Show view archiving key binding changes results. The thread archiving funct

[PATCH 2/6] NEWS: emacs: show view archiving key binding changes

2012-03-14 Thread Jani Nikula
--- d2ef4ed emacs: make show view a/A/x/X key bindings more consistent --- NEWS |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index 6778442..86c0690 100644 --- a/NEWS +++ b/NEWS @@ -74,6 +74,15 @@ Fix MML tag quoting in replies inserted in `message

[PATCH 1/6] NEWS: emacs: Fix MML tag quoting in replies

2012-03-14 Thread Jani Nikula
--- 90f310b emacs: fix MML quoting in replies --- NEWS |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index 4152e7c..6778442 100644 --- a/NEWS +++ b/NEWS @@ -68,6 +68,12 @@ New functions optionally visiting a URI to the current message at one of a numb

[PATCH 0/6] news for 0.12

2012-03-14 Thread Jani Nikula
Hi David, a bunch of 0.12 news items for you, the way you wanted: separate and boring. ;) They apply both to master and release branches at the time of writing. Drop the un-news-worthy ones as you see fit. J. Jani Nikula (6): NEWS: emacs: Fix MML tag quoting in replies NEWS: emacs: show view

[PATCH 0/6] news for 0.12

2012-03-14 Thread David Bremner
On Wed, 14 Mar 2012 23:21:05 +0200, Jani Nikula wrote: > Hi David, a bunch of 0.12 news items for you, the way you wanted: separate and > boring. ;) They apply both to master and release branches at the time of > writing. Drop the un-news-worthy ones as you see fit. > I pushed all six. I was a b

anecdotical evidence: number mbox files in my maildirs (was: Re: Patches for 0.12)

2012-03-14 Thread Gregor Zattler
Hi David, notmuch developers, * David Bremner [10. Mar. 2012]: > We had a somewhat lively debate about desirablity of the following > patch. My decision is that for now, we avoid changing the behaviour of > notmuch, and do apply the patch. We can discuss later deprecating or > changing this permis

Re: [PATCH 0/6] news for 0.12

2012-03-14 Thread David Bremner
On Wed, 14 Mar 2012 23:21:05 +0200, Jani Nikula wrote: > Hi David, a bunch of 0.12 news items for you, the way you wanted: separate and > boring. ;) They apply both to master and release branches at the time of > writing. Drop the un-news-worthy ones as you see fit. > I pushed all six. I was a b

[PATCH v7.1 00/11] Reply enhancements, second attempt

2012-03-14 Thread Austin Clements
Quoth Adam Wolfe Gordon on Mar 13 at 10:30 pm: > Hi everyone, > > This is mostly a re-send of last night's series [1]. The only changes are: > > * Fixed the emacs patch, where I had a rebase issue. > * Fixed the commit that re-added test/test-lib by accident. > * Fixed devel/schemata per Austin's

Patches for 0.12

2012-03-14 Thread Austin Clements
Quoth David Bremner on Mar 10 at 10:09 am: > > As usual, nothing brings out the bug reports like a freeze. > > There are some patches that should probably go in, but could use one > more pair of eyes. > > id:"1330849538-24558-1-git-send-email-jani at nikula.org" I looked at this and just sent

[PATCH] emacs: fix MML quoting in replies

2012-03-14 Thread Austin Clements
On Sun, 4 Mar 2012 10:25:38 +0200, Jani Nikula wrote: > The reply MML quoting added in commit ae438cc unintentionally MML > quotes also the signature/encryption MML tags added via > message-setup-hook, causing the reply not to be signed/encrypted. > > MML quote just the original message in the t

[PATCH 2/2] Allow selecting which version of gmime is used to build notmuch.

2012-03-14 Thread Tomi Ollila
This allows for testing against both versions of gmime on a single machine, without having to mess with pkg-config paths. This is rework of Tom Prince's patch submitted in id:"1331402091-15663-1-git-send-email-tom.prince at ualberta.net" --- configure | 15 ++- 1 files changed, 14 i

[PATCH 1/2] configure: store $IFS to $DEFAULT_IFS readonly variable

2012-03-14 Thread Tomi Ollila
In the future, IFS value needs to be changed in a few places in configure -- and then restored. Store the original value to $DEFAULT_IFS for easy restoration. --- configure |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/configure b/configure index ee0ae73..3b2b6b7

anecdotical evidence: number mbox files in my maildirs

2012-03-14 Thread Daniel Kahn Gillmor
On 03/14/2012 05:45 PM, Gregor Zattler wrote: > Hi David, notmuch developers, > * David Bremner [10. Mar. 2012]: >> We had a somewhat lively debate about desirablity of the following >> patch. My decision is that for now, we avoid changing the behaviour of >> notmuch, and do apply the patch. We ca

Re: Patches for 0.12

2012-03-14 Thread Jameson Graef Rollins
On Wed, 14 Mar 2012 18:16:01 -0400, Austin Clements wrote: > > We had a somewhat lively debate about desirablity of the following > > patch. My decision is that for now, we avoid changing the behaviour of > > notmuch, and do apply the patch. We can discuss later deprecating or > > changing this pe

Patches for 0.12

2012-03-14 Thread Jameson Graef Rollins
sort of coherent approach for rejecting mbox, but I think it will require a lot more thought. jamie. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120314/ac3c0b67/attachment.pgp>

Re: [PATCH v7.1 00/11] Reply enhancements, second attempt

2012-03-14 Thread Austin Clements
Quoth Adam Wolfe Gordon on Mar 13 at 10:30 pm: > Hi everyone, > > This is mostly a re-send of last night's series [1]. The only changes are: > > * Fixed the emacs patch, where I had a rebase issue. > * Fixed the commit that re-added test/test-lib by accident. > * Fixed devel/schemata per Austin's

Re: Patches for 0.12

2012-03-14 Thread Austin Clements
Quoth David Bremner on Mar 10 at 10:09 am: > > As usual, nothing brings out the bug reports like a freeze. > > There are some patches that should probably go in, but could use one > more pair of eyes. > > id:"1330849538-24558-1-git-send-email-j...@nikula.org" I looked at this and just sent a

Re: [PATCH] emacs: fix MML quoting in replies

2012-03-14 Thread Austin Clements
On Sun, 4 Mar 2012 10:25:38 +0200, Jani Nikula wrote: > The reply MML quoting added in commit ae438cc unintentionally MML > quotes also the signature/encryption MML tags added via > message-setup-hook, causing the reply not to be signed/encrypted. > > MML quote just the original message in the t

[PATCH 6/6] NEWS: cli: manual page for notmuch configuration options

2012-03-14 Thread Jani Nikula
--- bb8ab4c man: document the notmuch configuration settings in notmuch-config(1) --- NEWS |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index b2c07ab..09c5d9c 100644 --- a/NEWS +++ b/NEWS @@ -20,6 +20,12 @@ Mail store folder/file ignore NOTE: *Every

[PATCH 5/6] NEWS: emacs: enable `notmuch-search-line-faces' by default

2012-03-14 Thread Jani Nikula
--- e06943f emacs: add default value to notmuch-search-line-faces --- NEWS |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index 83d764a..b2c07ab 100644 --- a/NEWS +++ b/NEWS @@ -95,6 +95,12 @@ Generate inline patch fake attachment file names from message

[PATCH 4/6] NEWS: emacs: inline patch fake attachment file names

2012-03-14 Thread Jani Nikula
--- f893d31 emacs: create patch filename from subject for inline patch fake parts --- NEWS |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index 3c2cd82..83d764a 100644 --- a/NEWS +++ b/NEWS @@ -88,6 +88,13 @@ Support text/calendar MIME type The text/c

[PATCH 3/6] NEWS: emacs: support text/calendar MIME type

2012-03-14 Thread Jani Nikula
--- cfdc9a4 emacs: support text/calendar mime type --- NEWS |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index 86c0690..3c2cd82 100644 --- a/NEWS +++ b/NEWS @@ -83,6 +83,11 @@ Show view archiving key binding changes results. The thread archiving funct

[PATCH 2/6] NEWS: emacs: show view archiving key binding changes

2012-03-14 Thread Jani Nikula
--- d2ef4ed emacs: make show view a/A/x/X key bindings more consistent --- NEWS |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index 6778442..86c0690 100644 --- a/NEWS +++ b/NEWS @@ -74,6 +74,15 @@ Fix MML tag quoting in replies inserted in `message

[PATCH 1/6] NEWS: emacs: Fix MML tag quoting in replies

2012-03-14 Thread Jani Nikula
--- 90f310b emacs: fix MML quoting in replies --- NEWS |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index 4152e7c..6778442 100644 --- a/NEWS +++ b/NEWS @@ -68,6 +68,12 @@ New functions optionally visiting a URI to the current message at one of a numb

[PATCH 0/6] news for 0.12

2012-03-14 Thread Jani Nikula
Hi David, a bunch of 0.12 news items for you, the way you wanted: separate and boring. ;) They apply both to master and release branches at the time of writing. Drop the un-news-worthy ones as you see fit. J. Jani Nikula (6): NEWS: emacs: Fix MML tag quoting in replies NEWS: emacs: show view

Re: anecdotical evidence: number mbox files in my maildirs

2012-03-14 Thread Daniel Kahn Gillmor
On 03/14/2012 05:45 PM, Gregor Zattler wrote: Hi David, notmuch developers, * David Bremner [10. Mar. 2012]: We had a somewhat lively debate about desirablity of the following patch. My decision is that for now, we avoid changing the behaviour of notmuch, and do apply the patch. We can discuss

anecdotical evidence: number mbox files in my maildirs (was: Re: Patches for 0.12)

2012-03-14 Thread Gregor Zattler
Hi David, notmuch developers, * David Bremner [10. Mar. 2012]: > We had a somewhat lively debate about desirablity of the following > patch. My decision is that for now, we avoid changing the behaviour of > notmuch, and do apply the patch. We can discuss later deprecating or > changing this permis

[Patch v2 3/3] lib: fix an exclude bug

2012-03-14 Thread Mark Walters
When the exclude tags contain a tag that does not occur anywhere in the Xapian database the exclusion fails. We modify the way the query is constructed to `work around' this. (In fact the new code is cleaner anyway.) It also seems to fix another exclusion failure bug reported by jrollins but we ha

[Patch v2 2/3] test: add tests for message only search

2012-03-14 Thread Mark Walters
This adds three tests for --output=messages searches. One test is for the case when one exclude tag does not occur in the Xapian database. This triggers a Xapian bug in some cases and causes the whole exclusion to fail. The next commit avoids this bug. --- test/search | 17 + 1 f

[Patch v2 1/3] test: the test for the exclude code mistakenly excludes the tag "="

2012-03-14 Thread Mark Walters
The tests for the exclude code in search and count use the line notmuch config set search.exclude_tags = deleted which actually sets the exclude tags to be "=" and "deleted". Remove the "=" from this line. --- test/count |2 +- test/search |2 +- 2 files changed, 2 insertions(+), 2 de

[Patch v2 0/3] Add test and fix exclude bug

2012-03-14 Thread Mark Walters
The test in the previous patches id:"1331551914-28323-1-git-send-email-markwalters1009 at gmail.com" triggered the bug accidentally. It accidentally set the exclude tags to be "=" and "deleted" rather than just "deleted". The non-existent "=" tag (i.e., the tag that does not occur anywhere in the X

[PATCH 0/2] A bug in the exclude code

2012-03-14 Thread Mark Walters
On Tue, 13 Mar 2012 22:08:24 -0400, Austin Clements wrote: > On Mon, 12 Mar 2012 11:31:52 +, Mark Walters gmail.com> wrote: > > There is a bug in the exclude code (found by jrollins in the > > --with-excluded series) but also present in master. None of the > > current tests were finding it s

plans for 0.12

2012-03-14 Thread David Bremner
On Thu, 01 Mar 2012 22:21:29 +0100, Pieter Praet wrote: > On Sat, 25 Feb 2012 10:27:02 -0400, David Bremner > wrote: > > > > 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. > > I have tagged and pushed

[PATCH 1/2] configure: store $IFS to $DEFAULT_IFS readonly variable

2012-03-14 Thread Tomi Ollila
In the future, IFS value needs to be changed in a few places in configure -- and then restored. Store the original value to $DEFAULT_IFS for easy restoration. --- configure |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/configure b/configure index ee0ae73..3b2b6b7

[PATCH 2/2] Allow selecting which version of gmime is used to build notmuch.

2012-03-14 Thread Tomi Ollila
This allows for testing against both versions of gmime on a single machine, without having to mess with pkg-config paths. This is rework of Tom Prince's patch submitted in id:"1331402091-15663-1-git-send-email-tom.pri...@ualberta.net" --- configure | 15 ++- 1 files changed, 14 inse

[Patch v2 3/3] lib: fix an exclude bug

2012-03-14 Thread Mark Walters
When the exclude tags contain a tag that does not occur anywhere in the Xapian database the exclusion fails. We modify the way the query is constructed to `work around' this. (In fact the new code is cleaner anyway.) It also seems to fix another exclusion failure bug reported by jrollins but we ha

[Patch v2 2/3] test: add tests for message only search

2012-03-14 Thread Mark Walters
This adds three tests for --output=messages searches. One test is for the case when one exclude tag does not occur in the Xapian database. This triggers a Xapian bug in some cases and causes the whole exclusion to fail. The next commit avoids this bug. --- test/search | 17 + 1 f

[Patch v2 1/3] test: the test for the exclude code mistakenly excludes the tag "="

2012-03-14 Thread Mark Walters
The tests for the exclude code in search and count use the line notmuch config set search.exclude_tags = deleted which actually sets the exclude tags to be "=" and "deleted". Remove the "=" from this line. --- test/count |2 +- test/search |2 +- 2 files changed, 2 insertions(+), 2 de

[Patch v2 0/3] Add test and fix exclude bug

2012-03-14 Thread Mark Walters
The test in the previous patches id:"1331551914-28323-1-git-send-email-markwalters1...@gmail.com" triggered the bug accidentally. It accidentally set the exclude tags to be "=" and "deleted" rather than just "deleted". The non-existent "=" tag (i.e., the tag that does not occur anywhere in the Xapi

Re: plans for 0.12

2012-03-14 Thread David Bremner
On Thu, 01 Mar 2012 22:21:29 +0100, Pieter Praet wrote: > On Sat, 25 Feb 2012 10:27:02 -0400, David Bremner wrote: > > > > 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. > > I have tagged and pushed 0.1

Re: [PATCH 0/2] A bug in the exclude code

2012-03-14 Thread Mark Walters
On Tue, 13 Mar 2012 22:08:24 -0400, Austin Clements wrote: > On Mon, 12 Mar 2012 11:31:52 +, Mark Walters > wrote: > > There is a bug in the exclude code (found by jrollins in the > > --with-excluded series) but also present in master. None of the > > current tests were finding it so the fi