[PATCH 1/2] emacs: hello: bugfix: make alphabetically sorted saved searches work

2014-04-12 Thread Mark Walters
My recent changes to the saved search format broke the alphabetically sorted saved sort option. This makes it work again. Also, since the saved-search list is no longer an alist change the names in the sort function to avoid confusion. --- Thanks to Jani for pointing out the bug. Best wishes

[PATCH 2/2] NEWS: document possible breakage from saved-search format change

2014-04-12 Thread Mark Walters
If the user has unsorted or alphabetically sorted saved-searches these should continue to work. If they have some custom lisp sort function then it will need updating to work with the new saved-sort format. Document this, and how the updating should be done. Also roll in the fix for the markdown

[PATCH] emacs: hello: update docs for saved-search sort defcustom.

2014-04-12 Thread Mark Walters
Update the documentation for notmuch-saved-search-sort-function to match the new format and recommend the best way to access the fields (which should minimize breakage if we need to change the format again in the future). --- This could be rolled into

Re: [PATCH 1/2] emacs: hello: bugfix: make alphabetically sorted saved searches work

2014-04-12 Thread Jani Nikula
On Sat, 12 Apr 2014, Mark Walters markwalters1...@gmail.com wrote: My recent changes to the saved search format broke the alphabetically sorted saved sort option. This makes it work again. Also, since the saved-search list is no longer an alist change the names in the sort function to avoid

Re: v7 gzip dump restore patches

2014-04-12 Thread David Bremner
David Bremner da...@tethera.net writes: Supercedes id:1396554083-3892-2-git-send-email-da...@tethera.net - adds new analogues of strerror - util_error_string - gz_error_string pushed, amended as suggested by Austin and Tomi. d ___

[PATCH] emacs: add $(srcdir) to notmuch-version.el.tmpl dependency

2014-04-12 Thread Tomi Ollila
This fixes out-of-tree build when generating emacs/notmuch-version.el. --- emacs/Makefile.local | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/emacs/Makefile.local b/emacs/Makefile.local index ca039e5..dcb4995 100644 --- a/emacs/Makefile.local +++ b/emacs/Makefile.local @@

[PATCH v2 1/2] emacs: hello: bugfix: make alphabetically sorted saved searches work

2014-04-12 Thread Mark Walters
My recent changes to the saved search format broke the alphabetically sorted saved sort option. This makes it work again. Also update docs for saved-search sort defcustom to match the new format. Finally, since the saved-search list is no longer an alist change the names in the sort function to

[PATCH v2 0/2] emacs: hello: saved search sort fixes

2014-04-12 Thread Mark Walters
This is v2 of the collection of patches in id:1397292033-26502-1-git-send-email-markwalters1...@gmail.com The series makes the alphabetically sorted saved searches options work again, documents the new saved search sorting format, and fixes some of the markdown problesm in the original NEWS

[PATCH v2 2/2] NEWS: document possible breakage from saved-search format change

2014-04-12 Thread Mark Walters
If the user has unsorted or alphabetically sorted saved-searches these should continue to work. If they have some custom lisp sort function then it will need updating to work with the new saved-sort format. Document this, and how the updating should be done. Also roll in fixes for the markdown in

Re: [PATCH v2 0/2] emacs: hello: saved search sort fixes

2014-04-12 Thread Tomi Ollila
On Sat, Apr 12 2014, Mark Walters markwalters1...@gmail.com wrote: This is v2 of the collection of patches in id:1397292033-26502-1-git-send-email-markwalters1...@gmail.com The series makes the alphabetically sorted saved searches options work again, documents the new saved search sorting

Re: notmuch-hello buffer slow doe to slow query

2014-04-12 Thread David Bremner
Nils Dagsson Moskopp n...@dieweltistgarnichtso.net writes: mjw1009 can reproduce if the from-query contains an @ and thinks the problem may be something deeper down in notmuch (actually probably in xapian). My own experiments contradict that (see attached patch for a test to run). The exact

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

2014-04-12 Thread David Bremner
dm-list-email-notm...@scs.stanford.edu writes: As far as updating the test suite, etc., it's almost certain that the core notmuch developers would be unsatisfied with whatever I've done, since the code base is very clean and has a very uniform style. So when I say I'd want some indication

[PATCH 1/1] configure: add $(ZLIB_CFLAGS) to CONFIGURE_CFLAGS

2014-04-12 Thread Tomi Ollila
As it is defined in CONFIGURE_CXXFLAGS. --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index b8a3ce7..f447581 100755 --- a/configure +++ b/configure @@ -908,6 +908,7 @@ WITH_ZSH = ${WITH_ZSH} # Combined flags for compiling and linking against all of the

[PATCH] nmbug: mark repository as bare on clone

2014-04-12 Thread David Bremner
As far as I can figure out, if a git repository is non-bare, then it should either have core.worktree set, or the parent directory should be the worktree. Since the parent directory is usually the user's home directory, we don't want that. Although unlikely to be typed by mistake % cd $HOME ln

Re: [PATCH] nmbug: mark repository as bare on clone

2014-04-12 Thread Tomi Ollila
On Sat, Apr 12 2014, David Bremner da...@tethera.net wrote: As far as I can figure out, if a git repository is non-bare, then it should either have core.worktree set, or the parent directory should be the worktree. Since the parent directory is usually the user's home directory, we don't want

Re: [PATCH] nmbug: mark repository as bare on clone

2014-04-12 Thread David Bremner
Tomi Ollila tomi.oll...@iki.fi writes: Based on that info change looks reasonable. I edited my ~/.nmbug/config and changed bare = false - bare = true and nothing borke so far. so +1 from me. Tomi I forgot to mention that I ran that way for a week or so without problems. d

Re: [PATCH] emacs: process crypto for reply only when specified

2014-04-12 Thread David Bremner
Jameson Graef Rollins jroll...@finestructure.net writes: This is a tweak to patch emacs: sign/encrypt replies to signed/encrypted messages to only add mml crypto flags for replys when crypto processing has been activated. The (merged) patch seems straightforward and seems to work. I'll

[PATCH 1/2] emacs: hello: bugfix: make alphabetically sorted saved searches work

2014-04-12 Thread Mark Walters
My recent changes to the saved search format broke the alphabetically sorted saved sort option. This makes it work again. Also, since the saved-search list is no longer an alist change the names in the sort function to avoid confusion. --- Thanks to Jani for pointing out the bug. Best wishes

[PATCH 2/2] NEWS: document possible breakage from saved-search format change

2014-04-12 Thread Mark Walters
If the user has unsorted or alphabetically sorted saved-searches these should continue to work. If they have some custom lisp sort function then it will need updating to work with the new saved-sort format. Document this, and how the updating should be done. Also roll in the fix for the markdown

[PATCH 1/2] emacs: hello: bugfix: make alphabetically sorted saved searches work

2014-04-12 Thread Jani Nikula
On Sat, 12 Apr 2014, Mark Walters wrote: > My recent changes to the saved search format broke the alphabetically > sorted saved sort option. This makes it work again. > > Also, since the saved-search list is no longer an alist change the > names in the sort function to avoid confusion. > --- >

v7 gzip dump restore patches

2014-04-12 Thread David Bremner
David Bremner writes: > Supercedes > > id:1396554083-3892-2-git-send-email-david at tethera.net > > - adds new analogues of strerror > - util_error_string > - gz_error_string > pushed, amended as suggested by Austin and Tomi. d

[PATCH] emacs: add $(srcdir) to notmuch-version.el.tmpl dependency

2014-04-12 Thread Tomi Ollila
This fixes out-of-tree build when generating emacs/notmuch-version.el. --- emacs/Makefile.local | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/emacs/Makefile.local b/emacs/Makefile.local index ca039e5..dcb4995 100644 --- a/emacs/Makefile.local +++ b/emacs/Makefile.local @@

[PATCH v2 0/2] emacs: hello: saved search sort fixes

2014-04-12 Thread Mark Walters
This is v2 of the collection of patches in id:1397292033-26502-1-git-send-email-markwalters1009 at gmail.com The series makes the alphabetically sorted saved searches options work again, documents the new saved search sorting format, and fixes some of the markdown problesm in the original NEWS

[PATCH v2 1/2] emacs: hello: bugfix: make alphabetically sorted saved searches work

2014-04-12 Thread Mark Walters
My recent changes to the saved search format broke the alphabetically sorted saved sort option. This makes it work again. Also update docs for saved-search sort defcustom to match the new format. Finally, since the saved-search list is no longer an alist change the names in the sort function to

[PATCH v2 2/2] NEWS: document possible breakage from saved-search format change

2014-04-12 Thread Mark Walters
If the user has unsorted or alphabetically sorted saved-searches these should continue to work. If they have some custom lisp sort function then it will need updating to work with the new saved-sort format. Document this, and how the updating should be done. Also roll in fixes for the markdown in

[PATCH v2 0/2] emacs: hello: saved search sort fixes

2014-04-12 Thread Tomi Ollila
On Sat, Apr 12 2014, Mark Walters wrote: > This is v2 of the collection of patches in > id:1397292033-26502-1-git-send-email-markwalters1009 at gmail.com > > The series makes the alphabetically sorted saved searches options work > again, documents the new saved search sorting format, and fixes

notmuch-hello buffer slow doe to slow query

2014-04-12 Thread David Bremner
as scrubbed... Name: 0001-WIP-new-performance-test-for-notmuch-count.patch Type: text/x-diff Size: 1119 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20140412/c9883ac1/attachment.patch>

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

2014-04-12 Thread David Bremner
dm-list-email-notmuch at scs.stanford.edu writes: > > As far as updating the test suite, etc., it's almost certain that the > core notmuch developers would be unsatisfied with whatever I've done, > since the code base is very clean and has a very uniform style. So when > I say I'd want some

[PATCH 1/1] configure: add $(ZLIB_CFLAGS) to CONFIGURE_CFLAGS

2014-04-12 Thread Tomi Ollila
As it is defined in CONFIGURE_CXXFLAGS. --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index b8a3ce7..f447581 100755 --- a/configure +++ b/configure @@ -908,6 +908,7 @@ WITH_ZSH = ${WITH_ZSH} # Combined flags for compiling and linking against all of the

[PATCH] nmbug: mark repository as bare on clone

2014-04-12 Thread David Bremner
As far as I can figure out, if a git repository is non-bare, then it should either have core.worktree set, or the parent directory should be the worktree. Since the parent directory is usually the user's home directory, we don't want that. Although unlikely to be typed by mistake % cd $HOME && ln

[PATCH] nmbug: mark repository as bare on clone

2014-04-12 Thread Tomi Ollila
On Sat, Apr 12 2014, David Bremner wrote: > As far as I can figure out, if a git repository is non-bare, then it > should either have core.worktree set, or the parent directory should > be the worktree. Since the parent directory is usually the user's home > directory, we don't want that.

[PATCH] nmbug: mark repository as bare on clone

2014-04-12 Thread David Bremner
Tomi Ollila writes: > > Based on that info change looks reasonable. I edited my ~/.nmbug/config > and changed bare = false -> bare = true and nothing borke so far. > > so +1 from me. > > Tomi I forgot to mention that I ran that way for a week or so without problems. d

[PATCH] emacs: process crypto for reply only when specified

2014-04-12 Thread David Bremner
Jameson Graef Rollins writes: > This is a tweak to patch "emacs: sign/encrypt replies to > signed/encrypted messages" to only add mml crypto flags for replys > when crypto processing has been activated. > The (merged) patch seems straightforward and seems to work. I'll probably push it tomorrow