Re: Thanks for the nmbug-status program

2014-04-22 Thread Carl Worth
"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 worth, I don't disagree. > > Does that mean your pa

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

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

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

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 di

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

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

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

Thanks for the nmbug-status program

2014-04-22 Thread W. Trevor King
and I don't mind if others want something like this. Cheers, Trevor -- This email may be signed or encrypted with GnuPG (http://www.gnupg.org). For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy -- next part -- A non-text attachment was scrubbe

Re: Thanks for the nmbug-status program

2014-04-22 Thread Carl Worth
"W. Trevor King" 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 works.) And now,

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: > >>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): > >

Thanks for the nmbug-status program

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

Re: Thanks for the nmbug-status program

2014-04-22 Thread Carl Worth
"W. Trevor King" 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 think that

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

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

2014-04-22 Thread David Bremner
Gaute Hope 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 used to add a tag

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: > > http://g

Thanks for the nmbug-status program

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

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 (

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 notmuch

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 wrote: > LGTM +1 Looks good, and fixes the problem for me. BR, Jani. > > 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 strin

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

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

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

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

Re: 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 di