[PATCH v2] test: Test thread linking in all possible delivery orders

2014-04-22 Thread Mark Walters
Hi Broadly this looks good but I am somewhat confused by the python part. Is it intended to produce the same tree multiple times? It does seem to produce all the possible trees (*) so it doesn't matter but might be worth a comment. Best wishes Mark (*) I think there should be 64 rooted trees

[PATCH 1/1] building from git: use --abbrev=7 for version string

2014-04-22 Thread Mark Walters
LGTM +1 MW On Tue, 22 Apr 2014, Tomi Ollila wrote: > Users may have set core.abbrev=n, where n != 7 in their git config > file(s) which would give them different than expected version strings > when building notmuch from git. This fixes the commit hash part of > version string to 7 hexadecimal

[WIP PATCH 4/4] emacs: search: add key binding for marking a thread

2014-04-22 Thread Mark Walters
Add a function for toggling the mark on a thread, by default bound to ",". Also make a marked thread clearly visible: this uses inverse video as being a bit like selecting. Perhaps actually using the same face (mainly backgorund colour) as a selection would make sense? --- emacs/notmuch.el |

[WIP PATCH 3/4] emacs: search: split foreach-result into list and region variants

2014-04-22 Thread Mark Walters
--- emacs/notmuch.el | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/emacs/notmuch.el b/emacs/notmuch.el index 7b06458..e1ece47 100644 --- a/emacs/notmuch.el +++ b/emacs/notmuch.el @@ -380,7 +380,20 @@ (defun notmuch-search-result-end ( pos)

[WIP PATCH 2/4] emacs: search: add a marked thread variable and add to relevant functions

2014-04-22 Thread Mark Walters
Add a marked thread variable containing a list of thread ids of marked threads and make it be passed through all relevant functions. This ends up being quite large as there are quite a few callers. Alternatively we could overload beg rather than adding a thread-list variable. This would make a

[WIP PATCH 1/4] emacs: search: tidy notmuch-search-foreach-result

2014-04-22 Thread Mark Walters
notmuch-search-foreach-result is cumbersome as it applies the callback function with the position of the start of the thread as the argument. It is more natural to move point to the start of each relevant thread and not pass any arguments to the callback function. --- emacs/notmuch.el | 52

[WIP PATCH 0/4] Add a mark thread option

2014-04-22 Thread Mark Walters
Allow users to mark threads in the search buffer and then apply tagging operations to all marked threads. This was requested on irc and is also something I have wanted several times. Patch 1 is a cleanup which simplifies the existing code (imo) and makes the subsequent patches easier. Patch 2

0.18: call for NEWS, feature freeze April 21

2014-04-22 Thread David Bremner
David Bremner writes: > Hi Gang; > > We're overdue for a release. I'd like to feature freeze on April 21 and > release not too long after that. I ran through the commits since 0.17 > and we are missing quite a few NEWS items. We are still missing some NEWS items, but I tagged 0.18_rc0 (and

[PATCH 1/1] building from git: use --abbrev=7 for version string

2014-04-22 Thread Tomi Ollila
Users may have set core.abbrev=n, where n != 7 in their git config file(s) which would give them different than expected version strings when building notmuch from git. This fixes the commit hash part of version string to 7 hexadecimal values. --- tested with: git config core.abbrev 12 git

[PATCH v2] test: Test thread linking in all possible delivery orders

2014-04-22 Thread Austin Clements
The intent was to produce distinct trees, but obviously combinatorics is not my strong suit. Any ideas how to fix/rewrite the algorithm, other than just uniq'ing the output? Quoth Mark Walters on Apr 22 at 10:31 pm: > > Hi > > Broadly this looks good but I am somewhat confused by the python >

Thanks for the nmbug-status program

2014-04-22 Thread W. Trevor King
PG (http://www.gnupg.org). For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: OpenPGP digital signature URL: <http://not

Thanks for the nmbug-status program

2014-04-22 Thread W. Trevor King
bytes Desc: OpenPGP digital signature URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20140422/9d0729cd/attachment.pgp>

Thanks for the nmbug-status program

2014-04-22 Thread W. Trevor King
ze: 836 bytes Desc: OpenPGP digital signature URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20140422/91f8c6f0/attachment.pgp>

[PATCH 00/11] Improve charset and cid: handling

2014-04-22 Thread Tomi Ollila
On Mon, Apr 21 2014, Austin Clements wrote: > I set out to quickly add support for cid: links in the shr renderer > and wound up making our charset handling more robust and rewriting our > content-ID handling. The test introduced in patch 2 passes in all but > one really obscure case, but only

Re: 0.18: call for NEWS, feature freeze April 21

2014-04-22 Thread David Bremner
David Bremner da...@tethera.net writes: Hi Gang; We're overdue for a release. I'd like to feature freeze on April 21 and release not too long after that. I ran through the commits since 0.17 and we are missing quite a few NEWS items. We are still missing some NEWS items, but I tagged

[PATCH 1/1] building from git: use --abbrev=7 for version string

2014-04-22 Thread Tomi Ollila
Users may have set core.abbrev=n, where n != 7 in their git config file(s) which would give them different than expected version strings when building notmuch from git. This fixes the commit hash part of version string to 7 hexadecimal values. --- tested with: git config core.abbrev 12 git

[WIP PATCH 0/4] Add a mark thread option

2014-04-22 Thread Mark Walters
Allow users to mark threads in the search buffer and then apply tagging operations to all marked threads. This was requested on irc and is also something I have wanted several times. Patch 1 is a cleanup which simplifies the existing code (imo) and makes the subsequent patches easier. Patch 2

[WIP PATCH 2/4] emacs: search: add a marked thread variable and add to relevant functions

2014-04-22 Thread Mark Walters
Add a marked thread variable containing a list of thread ids of marked threads and make it be passed through all relevant functions. This ends up being quite large as there are quite a few callers. Alternatively we could overload beg rather than adding a thread-list variable. This would make a

[WIP PATCH 3/4] emacs: search: split foreach-result into list and region variants

2014-04-22 Thread Mark Walters
--- emacs/notmuch.el | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/emacs/notmuch.el b/emacs/notmuch.el index 7b06458..e1ece47 100644 --- a/emacs/notmuch.el +++ b/emacs/notmuch.el @@ -380,7 +380,20 @@ (defun notmuch-search-result-end (optional pos)

[WIP PATCH 1/4] emacs: search: tidy notmuch-search-foreach-result

2014-04-22 Thread Mark Walters
notmuch-search-foreach-result is cumbersome as it applies the callback function with the position of the start of the thread as the argument. It is more natural to move point to the start of each relevant thread and not pass any arguments to the callback function. --- emacs/notmuch.el | 52

[WIP PATCH 4/4] emacs: search: add key binding for marking a thread

2014-04-22 Thread Mark Walters
Add a function for toggling the mark on a thread, by default bound to ,. Also make a marked thread clearly visible: this uses inverse video as being a bit like selecting. Perhaps actually using the same face (mainly backgorund colour) as a selection would make sense? --- emacs/notmuch.el | 15

Re: [PATCH 1/1] building from git: use --abbrev=7 for version string

2014-04-22 Thread Mark Walters
LGTM +1 MW On Tue, 22 Apr 2014, Tomi Ollila tomi.oll...@iki.fi wrote: Users may have set core.abbrev=n, where n != 7 in their git config file(s) which would give them different than expected version strings when building notmuch from git. This fixes the commit hash part of version string to

Re: [PATCH 1/1] building from git: use --abbrev=7 for version string

2014-04-22 Thread Jani Nikula
On Tue, 22 Apr 2014, Mark Walters markwalters1...@gmail.com wrote: LGTM +1 Looks good, and fixes the problem for me. BR, Jani. MW On Tue, 22 Apr 2014, Tomi Ollila tomi.oll...@iki.fi wrote: Users may have set core.abbrev=n, where n != 7 in their git config file(s) which would give them

Thanks for the nmbug-status program

2014-04-22 Thread Carl Worth
Hi David, I just wanted to give you a quick thank you for the nmbug-status program. It's really handy to be able to publish decent-looking HTML pages based on notmuch search results with this. The script is quite general, (only a tiny portion was tied directly to nmbug), so I renamed it

Re: [PATCH v2] test: Test thread linking in all possible delivery orders

2014-04-22 Thread Mark Walters
Hi Broadly this looks good but I am somewhat confused by the python part. Is it intended to produce the same tree multiple times? It does seem to produce all the possible trees (*) so it doesn't matter but might be worth a comment. Best wishes Mark (*) I think there should be 64 rooted trees

Re: Thanks for the nmbug-status program

2014-04-22 Thread W. Trevor King
On Tue, Apr 22, 2014 at 02:12:30PM -0700, Carl Worth wrote: The script is quite general, (only a tiny portion was tied directly to nmbug), so I renamed it notmuch-to-html, and hosted it here after a tiny bit of generalization and a couple of minor user-interface changes:

Re: [PATCH] Add configurable changed tag to messages that have been changed on disk

2014-04-22 Thread David Bremner
Gaute Hope e...@gaute.vetsj.com writes: I am talking about syncing tags to a maildir _folder_, not flags. It could be implemented as maildir.synchronize is now, but it would be a larger feature which could work in a lot of different ways. So to try and clarify the use case, this could be

Re: [PATCH v2] test: Test thread linking in all possible delivery orders

2014-04-22 Thread Austin Clements
The intent was to produce distinct trees, but obviously combinatorics is not my strong suit. Any ideas how to fix/rewrite the algorithm, other than just uniq'ing the output? Quoth Mark Walters on Apr 22 at 10:31 pm: Hi Broadly this looks good but I am somewhat confused by the python part.

Re: Thanks for the nmbug-status program

2014-04-22 Thread Carl Worth
W. Trevor King wk...@tremily.us writes: http://git.cworth.org/git/notmuch-to-html Is there a clonable URL for this? Yes. Sorry I didn't provide that, (and clearly I need to reconfigure my git web pages to mention the URL): git clone http://git.cworth.org/git/notmuch-to-html I

Re: Thanks for the nmbug-status program

2014-04-22 Thread W. Trevor King
On Tue, Apr 22, 2014 at 04:01:06PM -0700, Carl Worth wrote: W. Trevor King wk...@tremily.us writes: http://git.cworth.org/git/notmuch-to-html Is there a clonable URL for this? Yes. Sorry I didn't provide that, (and clearly I need to reconfigure my git web pages to mention the URL):

Re: Thanks for the nmbug-status program

2014-04-22 Thread Carl Worth
W. Trevor King wk...@tremily.us writes: $ git clone http://git.cworth.org/git/notmuch-to-html Sorry about that. What I had meant to say is: git clone git://git.cworth.org/git/notmuch-to-html (This time I actually copy-pasted from a terminal where I tested that the command actually

Re: Thanks for the nmbug-status program

2014-04-22 Thread W. Trevor King
On Tue, Apr 22, 2014 at 04:01:06PM -0700, Carl Worth wrote: W. Trevor King writes: I think that making nmbug and nmbug-status generally applicable for other projects is a good target. I don't see any need to pull them out into a stand-alone project to do that though. For what it's