[PATCH] Rearchitect the way the Received: header is concatenated

2010-04-23 Thread Dirk Hohndel
The previous implementation misunderstood the way the message file was handled. Under certain circumstances this could cause SEGVs as we were trying to keep reading from the file after it was closed. Now we treat the Received: header as special and always concatenate it when parsing the headers.

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread David Edmondson
I like the current behaviour, but changing the default would be fine. On Friday, April 23, 2010, Carl Worth wrote: > On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson wrote: >> Add `notmuch-show-headers-visible' which, when set `t', causes headers >> to be shown by default. > > A nice

[PATCH 4/4] Integrate notmuch-fcc mechansim

2010-04-23 Thread Sebastian Spaeth
I have gone wild and added a defcustom "notmuch-fcc-dirs". Depending on the value of that variable we will not do any maildir fcc at all (nil, the default), or it is of the format (("defaultsentbox") ("full name " . "Work/sentbox") ("full name2 " . "Work2/sentbox")) The outbox name will be

[PATCH 3/4] notmuch-maildir-fcc: elisp syntax fixes

2010-04-23 Thread Sebastian Spaeth
1)use insert-buffer-substring Rather than the insert-buffer. Emacs complains that it is for interactive use and not for use within elisp. So use insert-buffer-substring which does the same thing when not handed any 'begin' 'end' parameters. 2)replace caddr with (car (cdr (cdr))) The former

[PATCH 2/4] Integrate notmuch-maildir-fcc into notmuch

2010-04-23 Thread Sebastian Spaeth
Require notmuch-maildir-fcc and also install it. Rename all jkr/* functions to notmuch-maildir-fcc-* Signed-off-by: Sebastian Spaeth --- emacs/Makefile.local |3 ++- emacs/notmuch-maildir-fcc.el | 42 ++ emacs/notmuch.el |1 +

[PATCH] emacs: Fix i-search to open up invisible citations as necessary

2010-04-23 Thread David Edmondson
Add an `isearch-open-invisible' property to the overlays used to hide citations and signatures, together with an appropriate function to leave the invisible text visible should that be required. --- emacs/notmuch-wash.el |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git

[PATCH] removed unused variables

2010-04-23 Thread Carl Worth
s ever since rather than just fixing them. ;-) -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423

[PATCH] Fix linker error from insufficient LDFLAGS

2010-04-23 Thread Carl Worth
9 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/78ab1b90/attachment.pgp>

[PATCH] emacs: Add notmuch-address.el for address completion using notmuch

2010-04-23 Thread Carl Worth
eventually, but it would be quite nice if "sudo make install" took care of that. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail

[PATCH] emacs/notmuch-show.el: Add `notmuch-show-toggle-all' bound to M-RET

2010-04-23 Thread Carl Worth
a rename of the function as I mentioned before). -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100

[PATCH] Reordering of thread authors to list matching authors first

2010-04-23 Thread Carl Worth
me to it. -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/dda8f466/attachment.pgp>

test suite for fancy From guessing

2010-04-23 Thread Carl Worth
ture Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/63b8027a/attachment.pgp>

[PATCH] RFC: Add From guessing when forwarding email

2010-04-23 Thread Dirk Hohndel
This adds a new "guess-from" option to notmuch and modifies the emacs UI to use this to use the best guess from address when forwarding email. Given how little elisp I know I'm quite interested in feedback and better implementations Signed-off-by: Dirk Hohndel --- emacs/notmuch-show.el |8

[PATCH] emacs: Add notmuch-hello.el, a friendly frontend to notmuch

2010-04-23 Thread Carl Worth
hello" from the name there. It doesn't really add anything. -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/4e19b2bc/attachment.pgp>

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Jameson Rollins
t;http://notmuchmail.org/pipermail/notmuch/attachments/20100423/4c64a3fe/attachment.pgp>

notmuch segfault

2010-04-23 Thread Michal Sojka
On Fri, 23 Apr 2010, Sebastian Spaeth wrote: > Can it be that in thread.cc in _thread_add_matched_message () > ... > subject = notmuch_message_get_header (message, "subject"); > > if ((strncasecmp (subject, "Re: ", 4) == 0) || > ... > > If the underlying message disappeared, get_header

notmuch segfault

2010-04-23 Thread Sebastian Spaeth
It happened again. Both times I had pressed "G" which calls offlineimap and which removed messages that the notmuch database still thought are there. Program received signal SIGSEGV, Segmentation fault. 0x7602bb14 in

[notmuch] Bulk message tagging

2010-04-23 Thread Mark Anderson
On Wed, 21 Apr 2010 18:02:59 -0500, Carl Worth wrote: > On Sat, 17 Apr 2010 20:32:27 +0200, Arian Kuschki googlemail.com> wrote: > > So one could query with sysconf and break things up into multiple > > commands as needed. > > > > Doesn't xargs do exactly this? > > Almost. > > The arguments

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Servilio Afre Puentes
On 23 April 2010 13:13, Dirk Hohndel wrote: > On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson wrote: >> Add `notmuch-show-headers-visible' which, when set `t', causes headers >> to be shown by default. > > Excellent - Carl, can you pull this into 0.3, please? +1 Servilio

[PATCH] test: Set fixed time zone

2010-04-23 Thread Carl Worth
otherwise seems a bit out-of-place). Thanks, -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/34249f1b/attachment.pgp>

[PATCH] emacs: Push the cursor to point-max on `n' or `N' at the end of a thread

2010-04-23 Thread Carl Worth
t. Thanks! This is pushed. -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/1397cdf7/attachment.pgp>

[PATCH] emacs: Re-arrange message sending code

2010-04-23 Thread Carl Worth
non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/92f5c904/attachment.pgp>

[PATCH] emacs: Remove `notmuch-search-authors-width' and fix the use of `notmuch-search-result-format' accordingly

2010-04-23 Thread Jameson Rollins
ail.org/pipermail/notmuch/attachments/20100423/a8aa01e8/attachment.pgp>

Unhandled Xapian exception

2010-04-23 Thread Sebastian Spaeth
Hi Carl, dme complained that my python bindings abort with Xapian::DatabaseModifiedException when doing a Database.find_message('id'). But libnotmuch.so terminates before python has even a chance to catch an execption, and I think it boils down to this:

[PATCH] emacs: Add notmuch-hello.el, a friendly frontend to notmuch

2010-04-23 Thread Carl Worth
! This is really slick. -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/a341e1d3/attachment.pgp>

pull request

2010-04-23 Thread David Edmondson
On Wed, 21 Apr 2010 15:47:12 -0400, Jameson Rollins wrote: > On Wed, 21 Apr 2010 12:15:48 -0700, Carl Worth wrote: > > [*] Even more simplification is possible if we stop trying to hide > > header components. Several people have requested that To and Cc be > > visible all the time. > > Hey,

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread David Edmondson
Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by default. --- emacs/notmuch-show.el |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index cd859f0..3ea07c8 100644 ---

pull request

2010-04-23 Thread Carl Worth
art support, and HTML rendering is all very encouraging. I look forward to the future! -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/2919d1d6/attachment.pgp>

[PATCH] emacs/notmuch-show.el: Add `notmuch-show-toggle-all' bound to M-RET

2010-04-23 Thread Carl Worth
oth would be something like notmuch-show-open-or-close-all ? -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Carl Worth
nature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/342c3ede/attachment.pgp>

[PATCH] emacs: Remove `notmuch-search-authors-width' and fix the use of `notmuch-search-result-format' accordingly

2010-04-23 Thread Carl Worth
ature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/20393f8e/attachment.pgp>

Updated elisp FCC patches (was: Fcc, Maildir, and Emacs message-mode)

2010-04-23 Thread Sebastian Spaeth
My last mail on this issue: I squashed the recent 7 patch series into 4 nicer ones. Rather than resending the patch series, here are the 4 commits from my repo at git at github.com:spaetz/notmuch-all-feature.git (let me know if I should mail them too). (These 4 are in the feature/elisp feature

[PATCH 1/2] Add 'cat' subcommand

2010-04-23 Thread Carl Worth
d job anticipating that failure mode before we ran into it. -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/397abbc6/attachment.pgp>

Notmuch success: Xapian database corrupt

2010-04-23 Thread Carl Worth
--- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/f7e738e3/attachment-0001.pgp>

[PATCH] Reintroduce patch to quote args in notmuch-show to facilitate remote use

2010-04-23 Thread Carl Worth
. Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/3d06fc9b/attachment.pgp>

add delete keybinding

2010-04-23 Thread Carl Worth
some sort of external expunge operation that actually deleted the files, of course. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/b840e74b/attachment.pgp>

improve from-header guessing

2010-04-23 Thread Carl Worth
next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/6853514b/attachment.pgp>

[PATCH 7/7] Integrate notmuch-fcc mechansim

2010-04-23 Thread Sebastian Spaeth
I have gone wild and added a defcustom "notmuch-fcc-dirs". Depending on the value of that variable we will not do any maildir fcc at all (nil, the default), or it is of the format (("defaultsentbox") ("full name " . "Work/sentbox") ("full name2 " . "Work2/sentbox")) The outbox name will be

[PATCH] emacs: Remove `notmuch-search-authors-width' and fix the use of `notmuch-search-result-format' accordingly

2010-04-23 Thread David Edmondson
The width of the authors field in search output was previously specified in two places: - `notmuch-search-authors-width': the limit beyond which the authors names are truncated, - `notmuch-search-result-format': the layout of the search results. Changing the configuration of one of these may

[Announce] notmuch release 0.2 now available

2010-04-23 Thread Carl Worth
re Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/0d0eaac1/attachment.pgp>

[PATCH] notmuch new --new-tags=tags...

2010-04-23 Thread Carl Worth
time, we can muddle along with the functionality we want, (but a little more manual effort to achieve it). -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/02786018/attachment.pgp>

[PATCH] Add NEWS updates for my last batch of patches

2010-04-23 Thread Dirk Hohndel
in the future I'll include those with my patches. Hope it's ok to do this as one single patch for this series. Signed-off-by: Dirk Hohndel --- NEWS | 35 +++ 1 files changed, 35 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index eba0fd5..5586386

[notmuch] [PATCH] notmuch-config: make new message tags configurable

2010-04-23 Thread Carl Worth
new_tags=inbox;unread; to instead be: [new] tags=inbox;unread; * Fixed "notmuch setup" to prompt for this new setting. Thanks again, -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application

sort order regression

2010-04-23 Thread Michal Sojka
On Fri, 23 Apr 2010, Sebastian Spaeth wrote: > Ohh, that is a good point. Maybe I should write some :-). Is the test > suite going to be changed any day now or does it still make sense to > write tests for the "monolitic" test suite? I do not have a plan to modularize the test suite in a near

[PATCH] Add 'G' keybinding to folder and search view that triggers external poll

2010-04-23 Thread Dirk Hohndel
On Fri, 23 Apr 2010 08:20:40 -0700, Carl Worth wrote: > On Fri, 23 Apr 2010 10:09:03 +0200, "Sebastian Spaeth" SSpaeth.de> wrote: > > Hehe. Very useful indeed. There is one more thing: Would it be possible to > > provide user > > feedback while this is running (synchronously, I guess)? Like

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Dirk Hohndel
On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson wrote: > Add `notmuch-show-headers-visible' which, when set `t', causes headers > to be shown by default. Excellent - Carl, can you pull this into 0.3, please? /D -- Dirk Hohndel Intel Open Source Technology Center

[PATCH] Add 'G' keybinding to folder and search view that triggers external poll

2010-04-23 Thread Sebastian Spaeth
On 2010-04-22, Dirk Hohndel wrote: > I appreciate how nicely you can say "I liked the idea and then > completely rewrote the crap elisp that you submitted" :-) Hehe. Very useful indeed. There is one more thing: Would it be possible to provide user feedback while this is running (synchronously, I

notmuch segfault

2010-04-23 Thread Sebastian Spaeth
Sorry, I won't be able to offer much debug info, but the current stock cworth/master segfaulted for me with only the "tag:inbox" search. Output: /home/spaetz/mail/INBOX/new/...2c57e:2,: No such file or directory Segmentation fault (core dumped) A "notmuch new" fixed the segfault, something is

[PATCH] test: Set fixed time zone

2010-04-23 Thread Michal Sojka
When the test suite is run in a different time zone that where Carl lives, some tests may fail depending on the time when the test suite is run. For example, just now I get: Search for all messages ("*"):... FAIL --- test-031.expected 2010-04-23 09:33:47.898634822

sort order regression

2010-04-23 Thread Sebastian Spaeth
On 2010-04-22, Carl Worth wrote: > On 22 Apr 2010, Sebastian Spaeth wrote: > > > > jkr and I noticed that patch series are shown in reverse order now, in > > fact threads seem to display messages at the same depth in reverse > > chronological order now. > > My fault! Sorry about that. No harm

[notmuch] Fcc, Maildir, and Emacs message-mode -- a bit of code

2010-04-23 Thread Sebastian Spaeth
On 2010-04-22, Dirk Hohndel wrote: > ;; This is the list of alternatives that should be configurable as > ;; defcustom (or simply set in .emacs for now) > (setq notmuch-fcc-dirs '( > ("Dirk Hohndel " . "Maildir/Sent Items") > ("Dirk Hohndel " . "MaildirInfradead/Sent"))) > > ;This

pull request

2010-04-23 Thread Jameson Rollins
pplication/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20100423/edccab98/attachment.pgp>

[PATCH] Add 'G' keybinding to folder and search view that triggers external poll

2010-04-23 Thread Carl Worth
Then I get a nice "busy" mouse cursor during this operation instead of the standard text-edit bar. -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <htt

sort order regression

2010-04-23 Thread Carl Worth
On Fri, 23 Apr 2010 10:15:22 +0200, Michal Sojka wrote: > On Fri, 23 Apr 2010, Sebastian Spaeth wrote: > > Ohh, that is a good point. Maybe I should write some :-). Is the test > > suite going to be changed any day now or does it still make sense to > > write tests for the "monolitic" test suite?

[PATCH] Add 'G' keybinding to folder and search view that triggers external poll

2010-04-23 Thread Dirk Hohndel
On Fri, 23 Apr 2010 10:09:03 +0200, "Sebastian Spaeth" wrote: > On 2010-04-22, Dirk Hohndel wrote: > > I appreciate how nicely you can say "I liked the idea and then > > completely rewrote the crap elisp that you submitted" :-) > > Hehe. Very useful indeed. There is one more thing: Would it be

Updated elisp FCC patches (was: Fcc, Maildir, and Emacs message-mode)

2010-04-23 Thread Dirk Hohndel
On Fri, 23 Apr 2010 12:08:31 +0200, "Sebastian Spaeth" wrote: > My last mail on this issue: I squashed the recent 7 patch series into 4 nicer > ones. > > Rather than resending the patch series, here are the 4 commits from my > repo at git at github.com:spaetz/notmuch-all-feature.git (let me know

[PATCH] Makefile.local: Fix LDFLAGS for notmuch-shared

2010-04-23 Thread Adrien Bustany
This commit adds GMIME_LDFLAGS and TALLOC_LDFLAGS to the linker flags when linking notmuch-shared. Without these flags, linking fails because of undefined symbols. --- Makefile.local |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile.local b/Makefile.local index

Re: [notmuch] Fcc, Maildir, and Emacs message-mode -- a bit of code

2010-04-23 Thread Sebastian Spaeth
On 2010-04-22, Dirk Hohndel wrote: ;; This is the list of alternatives that should be configurable as ;; defcustom (or simply set in .emacs for now) (setq notmuch-fcc-dirs '( (Dirk Hohndel dirk.hohn...@intel.com . Maildir/Sent Items) (Dirk Hohndel hohn...@infradead.org .

[PATCH] test: Set fixed time zone

2010-04-23 Thread Michal Sojka
When the test suite is run in a different time zone that where Carl lives, some tests may fail depending on the time when the test suite is run. For example, just now I get: Search for all messages (*):... FAIL --- test-031.expected 2010-04-23 09:33:47.898634822

notmuch segfault

2010-04-23 Thread Sebastian Spaeth
Sorry, I won't be able to offer much debug info, but the current stock cworth/master segfaulted for me with only the tag:inbox search. Output: /home/spaetz/mail/INBOX/new/...2c57e:2,: No such file or directory Segmentation fault (core dumped) A notmuch new fixed the segfault, something is not as

Re: [PATCH] Add 'G' keybinding to folder and search view that triggers external poll

2010-04-23 Thread Sebastian Spaeth
On 2010-04-22, Dirk Hohndel wrote: I appreciate how nicely you can say I liked the idea and then completely rewrote the crap elisp that you submitted :-) Hehe. Very useful indeed. There is one more thing: Would it be possible to provide user feedback while this is running (synchronously, I

Re: sort order regression

2010-04-23 Thread Michal Sojka
On Fri, 23 Apr 2010, Sebastian Spaeth wrote: Ohh, that is a good point. Maybe I should write some :-). Is the test suite going to be changed any day now or does it still make sense to write tests for the monolitic test suite? I do not have a plan to modularize the test suite in a near future.

[PATCH 7/7] Integrate notmuch-fcc mechansim

2010-04-23 Thread Sebastian Spaeth
I have gone wild and added a defcustom notmuch-fcc-dirs. Depending on the value of that variable we will not do any maildir fcc at all (nil, the default), or it is of the format ((defaultsentbox) (full name em...@address . Work/sentbox) (full name2 ema...@address2 . Work2/sentbox)) The outbox

[PATCH] emacs: Remove `notmuch-search-authors-width' and fix the use of `notmuch-search-result-format' accordingly

2010-04-23 Thread David Edmondson
The width of the authors field in search output was previously specified in two places: - `notmuch-search-authors-width': the limit beyond which the authors names are truncated, - `notmuch-search-result-format': the layout of the search results. Changing the configuration of one of these may

Unhandled Xapian exception

2010-04-23 Thread Sebastian Spaeth
Hi Carl, dme complained that my python bindings abort with Xapian::DatabaseModifiedException when doing a Database.find_message('id'). But libnotmuch.so terminates before python has even a chance to catch an execption, and I think it boils down to this:

Re: pull request

2010-04-23 Thread David Edmondson
On Wed, 21 Apr 2010 15:47:12 -0400, Jameson Rollins jroll...@finestructure.net wrote: On Wed, 21 Apr 2010 12:15:48 -0700, Carl Worth cwo...@cworth.org wrote: [*] Even more simplification is possible if we stop trying to hide header components. Several people have requested that To and Cc be

Re: pull request

2010-04-23 Thread Jameson Rollins
On Fri, 23 Apr 2010 12:57:42 +0100, David Edmondson d...@dme.org wrote: Hey, Carl. I had actually been meaning to bring this up. I actually don't like having the headers collapsed at all, so I am definitely in favor of the idea of doing away with this feature. At the very least, I would

Re: notmuch segfault

2010-04-23 Thread Sebastian Spaeth
It happened again. Both times I had pressed G which calls offlineimap and which removed messages that the notmuch database still thought are there. Program received signal SIGSEGV, Segmentation fault. 0x7602bb14 in

Re: notmuch segfault

2010-04-23 Thread Michal Sojka
On Fri, 23 Apr 2010, Sebastian Spaeth wrote: Can it be that in thread.cc in _thread_add_matched_message () ... subject = notmuch_message_get_header (message, subject); if ((strncasecmp (subject, Re: , 4) == 0) || ... If the underlying message disappeared, get_header will return

Re: sort order regression

2010-04-23 Thread Carl Worth
On Fri, 23 Apr 2010 10:15:22 +0200, Michal Sojka sojk...@fel.cvut.cz wrote: On Fri, 23 Apr 2010, Sebastian Spaeth wrote: Ohh, that is a good point. Maybe I should write some :-). Is the test suite going to be changed any day now or does it still make sense to write tests for the monolitic

Re: [PATCH] Add 'G' keybinding to folder and search view that triggers external poll

2010-04-23 Thread Carl Worth
On Fri, 23 Apr 2010 10:09:03 +0200, Sebastian Spaeth sebast...@sspaeth.de wrote: Hehe. Very useful indeed. There is one more thing: Would it be possible to provide user feedback while this is running (synchronously, I guess)? Like having some message in the minibuffer saying Calling all

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Dirk Hohndel
On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson d...@dme.org wrote: Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by default. Excellent - Carl, can you pull this into 0.3, please? /D -- Dirk Hohndel Intel Open Source Technology Center

Re: [PATCH] Add 'G' keybinding to folder and search view that triggers external poll

2010-04-23 Thread Dirk Hohndel
On Fri, 23 Apr 2010 08:20:40 -0700, Carl Worth cwo...@cworth.org wrote: On Fri, 23 Apr 2010 10:09:03 +0200, Sebastian Spaeth sebast...@sspaeth.de wrote: Hehe. Very useful indeed. There is one more thing: Would it be possible to provide user feedback while this is running (synchronously,

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Jameson Rollins
On Fri, 23 Apr 2010 10:13:37 -0700, Dirk Hohndel hohn...@infradead.org wrote: On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson d...@dme.org wrote: Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by default. Excellent - Carl, can you pull this into 0.3,

Re: [PATCH] test: Set fixed time zone

2010-04-23 Thread Dirk Hohndel
On Fri, 23 Apr 2010 09:36:45 +0200, Michal Sojka sojk...@fel.cvut.cz wrote: When the test suite is run in a different time zone that where Carl lives, some tests may fail depending on the time when the test suite is run. For example, just now I get: By setting a fixed time zone in the test

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Servilio Afre Puentes
On 23 April 2010 13:13, Dirk Hohndel hohn...@infradead.org wrote: On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson d...@dme.org wrote: Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by default. Excellent - Carl, can you pull this into 0.3, please? +1

Re: [PATCH] Add 'G' keybinding to folder and search view that triggers external poll

2010-04-23 Thread Dirk Hohndel
On Fri, 23 Apr 2010 10:09:03 +0200, Sebastian Spaeth sebast...@sspaeth.de wrote: On 2010-04-22, Dirk Hohndel wrote: I appreciate how nicely you can say I liked the idea and then completely rewrote the crap elisp that you submitted :-) Hehe. Very useful indeed. There is one more thing:

Re: Updated elisp FCC patches (was: Fcc, Maildir, and Emacs message-mode)

2010-04-23 Thread Dirk Hohndel
On Fri, 23 Apr 2010 12:08:31 +0200, Sebastian Spaeth sebast...@sspaeth.de wrote: My last mail on this issue: I squashed the recent 7 patch series into 4 nicer ones. Rather than resending the patch series, here are the 4 commits from my repo at g...@github.com:spaetz/notmuch-all-feature.git

[PATCH] Add NEWS updates for my last batch of patches

2010-04-23 Thread Dirk Hohndel
in the future I'll include those with my patches. Hope it's ok to do this as one single patch for this series. Signed-off-by: Dirk Hohndel hohn...@infradead.org --- NEWS | 35 +++ 1 files changed, 35 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index

Re: [PATCH] notmuch new --new-tags=tags...

2010-04-23 Thread Carl Worth
On Mon, 12 Apr 2010 07:59:14 -0400, Jameson Rollins jroll...@finestructure.net wrote: On Mon, 12 Apr 2010 10:00:37 +0200, Sebastian Spaeth sebast...@sspaeth.de wrote: On 2010-04-10, Anthony Towns wrote: The attached patch makes notmuch new --new-tags=unread,new set the unread and new

Re: [Announce] notmuch release 0.2 now available

2010-04-23 Thread Carl Worth
On Sat, 17 Apr 2010 09:49:01 +1000, Alex Ghitza aghi...@gmail.com wrote: notmuch-0.2 built fine, but one test failed during 'make test'. I have put the indicated directory up at http://aghitza.org/files/test.16707 Hi Alex, Thanks for reporting this. Unfortunately, at the time, the test

Re: [PATCH] Reintroduce patch to quote args in notmuch-show to facilitate remote use

2010-04-23 Thread Carl Worth
On Wed, 21 Apr 2010 17:22:08 -0400, Jesse Rosenthal jrosent...@jhu.edu wrote: This reintroduces the patch committed in 9193455fa1, which was reverted during the upgrade to the JSON emacs UI. Thanks. This is pushed. -Carl pgpyDg6MmumS6.pgp Description: PGP signature

[PATCH 1/4] Add elisp file for FCC to maildir solution

2010-04-23 Thread Sebastian Spaeth
From: Jesse Rosenthal jrosent...@jhu.edu File grabbed from http://jkr.acm.jhu.edu/jkr-maildir.el but not integrated yet. Signed-off-by: Sebastian Spaeth sebast...@sspaeth.de --- As stated by Dirk, patches are preferred by mail. So here is the updated patch series of 4 patches by mail again.

[PATCH 4/4] Integrate notmuch-fcc mechansim

2010-04-23 Thread Sebastian Spaeth
I have gone wild and added a defcustom notmuch-fcc-dirs. Depending on the value of that variable we will not do any maildir fcc at all (nil, the default), or it is of the format ((defaultsentbox) (full name em...@address . Work/sentbox) (full name2 ema...@address2 . Work2/sentbox)) The outbox

[PATCH 2/4] Integrate notmuch-maildir-fcc into notmuch

2010-04-23 Thread Sebastian Spaeth
Require notmuch-maildir-fcc and also install it. Rename all jkr/* functions to notmuch-maildir-fcc-* Signed-off-by: Sebastian Spaeth sebast...@sspaeth.de --- emacs/Makefile.local |3 ++- emacs/notmuch-maildir-fcc.el | 42 ++ emacs/notmuch.el

Re: Notmuch success: Xapian database corrupt

2010-04-23 Thread Carl Worth
On Thu, 22 Apr 2010 07:19:58 +, John Fremlin j...@fremlin.org wrote: After the encouraging message from Sebastian. I deleted the .notmuch/xapian dir and started again. It went off a good rate (300+ files/sec) and here was the final score Processed 494764 total files in 2h 54m 41s (47

Re: add delete keybinding

2010-04-23 Thread Carl Worth
On Wed, 21 Apr 2010 16:16:02 -0700, Dirk Hohndel hohn...@infradead.org wrote: I'm sending this mostly as an RFC - I use this and like it, but people seem to have strong feelings as to how they want to deal with deleting email (or for some people, how they don't want to do that at all). I

Re: [PATCH] emacs: Remove `notmuch-search-authors-width' and fix the use of `notmuch-search-result-format' accordingly

2010-04-23 Thread Carl Worth
On Fri, 23 Apr 2010 13:22:24 -0400, Jameson Rollins jroll...@finestructure.net wrote: On Fri, 23 Apr 2010 11:24:09 +0100, David Edmondson d...@dme.org wrote: Jamie, could you test this patch please? My main concern is that it makes a small assumption about the value of

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Carl Worth
On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson d...@dme.org wrote: Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by default. A nice improvement, definitely. But I don't hear anyone actually wanting a configuration value here. Would anyone complain if I

Re: [PATCH] emacs/notmuch-show.el: Add `notmuch-show-toggle-all' bound to M-RET

2010-04-23 Thread Carl Worth
On Thu, 22 Apr 2010 09:24:03 +0100, David Edmondson d...@dme.org wrote: `notmuch-show-toggle-all' changes the visibility all of the messages in the current thread. By default it makes all of the messages visible. With a prefix argument, it makes them all not visible. This is a better default,

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Jameson Rollins
On Fri, 23 Apr 2010 12:26:43 -0700, Carl Worth cwo...@cworth.org wrote: A nice improvement, definitely. But I don't hear anyone actually wanting a configuration value here. Would anyone complain if I just made these all visible by default? Would anyone complain if I removed the code to

RE: [notmuch] Bulk message tagging

2010-04-23 Thread Mark Anderson
On Wed, 21 Apr 2010 18:02:59 -0500, Carl Worth cwo...@cworth.org wrote: On Sat, 17 Apr 2010 20:32:27 +0200, Arian Kuschki arian.kusc...@googlemail.com wrote: So one could query with sysconf and break things up into multiple commands as needed. Doesn't xargs do exactly this? Almost.

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread David Edmondson
I like the current behaviour, but changing the default would be fine. On Friday, April 23, 2010, Carl Worth cwo...@cworth.org wrote: On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson d...@dme.org wrote: Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by

Re: [PATCH] emacs: Re-arrange message sending code

2010-04-23 Thread Carl Worth
On Thu, 22 Apr 2010 10:03:32 +0100, David Edmondson d...@dme.org wrote: Define a new `mail-user-agent' (`notmuch-user-agent') and use it by default. Re-arrange various routines that send mail to use this (compose, reply, forward). Insert a `User-Agent:' header by default. Very nice! I've

Re: [PATCH] emacs: Push the cursor to point-max on `n' or `N' at the end of a thread

2010-04-23 Thread Carl Worth
On Thu, 22 Apr 2010 12:20:16 +0100, David Edmondson d...@dme.org wrote: Sebastian pointed out that the pre-JSON UI would move the cursor to the end of the buffer if `n' or `N' is hit when on the last (unread) message. Mimic that behaviour in the new UI. Quite a lovely improvement. Thanks! This

Re: [PATCH] test: Set fixed time zone

2010-04-23 Thread Carl Worth
On Fri, 23 Apr 2010 09:36:45 +0200, Michal Sojka sojk...@fel.cvut.cz wrote: When the test suite is run in a different time zone that where Carl lives, some tests may fail depending on the time when the test suite is run. For example, just now I get: ... By setting a fixed time zone in the test

Re: [PATCH] emacs: Add notmuch-hello.el, a friendly frontend to notmuch

2010-04-23 Thread Carl Worth
On Fri, 23 Apr 2010 13:13:01 -0700, Carl Worth cwo...@cworth.org wrote: * I'd like the saved searches to appear before the recent searches I think. Actually, this might be OK since the recent searches go away when restarting emacs. * I would *love* a simple way to import my existing

Re: test suite for fancy From guessing

2010-04-23 Thread Carl Worth
On Wed, 21 Apr 2010 14:55:56 -0700, Dirk Hohndel hohn...@infradead.org wrote: This adds five tests for the five main cases in the fancy from guessing. It assumes that you have applied id:1271451102-11336-1-git-send-email-hohn...@infradead.org which will get you the latest fancy From guessing.

Re: [PATCH] Reordering of thread authors to list matching authors first

2010-04-23 Thread Carl Worth
On Wed, 21 Apr 2010 20:58:27 -0700, Dirk Hohndel hohn...@infradead.org wrote: When displaying threads as result of a search it makes sense to list those authors first who match the search. The matching authors are separated from the non-matching ones with a '|' instead of a ',' It seems a

Re: [PATCH] Fix linker error from insufficient LDFLAGS

2010-04-23 Thread Carl Worth
On Thu, 22 Apr 2010 18:20:27 -0400, Ben Gamari bgamari.f...@gmail.com wrote: It seems that LDFLAGS have recently been reorganized, along with the introduction of a notmuch-shared rule. Unfortunately, the LDFLAGS used in notmuch-shared don't include CONFIGURE_LDFLAGS. This caused linking to

  1   2   >