Re: [PATCH] test:Improve test behaviors when --root is used

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 16:50:47 -0400, Austin Clements amdra...@mit.edu wrote: This looks great (modulo one bug, below). I've wanted to run the tests on tmpfs before, but was too lazy to actually fix the tests. Given how easy it is to accidentally use .., I wonder if there's a way to force

Re: Debian package not building

2011-06-28 Thread Pieter Praet
On Fri, 24 Jun 2011 16:34:02 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: Non-text part: multipart/mixed Non-text part: multipart/signed Hey, folks. As of today I am for some reason no longer able to build the Notmuch Debian package. I'm using the same build technique I have

Re: [PATCH] test:Folder tags shouldn't match after removal of file in given folder

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 11:12:24 -0600, Mark Anderson ma.sk...@gmail.com wrote: Test for bug. Current stemming support for notmuch adds extra terms to the DB which aren't removed when the file renames are detected. When folder tags are added to a message, Xapian terms for both XFOLDER and

Re: Race condition for '*' command

2011-06-28 Thread Pieter Praet
On Sun, 26 Jun 2011 10:00:41 +0100, Robin Green gree...@greenrd.org wrote: On Sat, 25 Jun 2011 16:57:50 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: On Sat, 25 Jun 2011 23:18:52 +0100, Robin Green gree...@greenrd.org wrote: A race condition in the '*' command was noted

Re: Drafts being tagged as inbox

2011-06-28 Thread Robin Green
On Tue, 28 Jun 2011 08:34:44 +0200, Pieter Praet pie...@praet.org wrote: notmuch tag -inbox +draft -- folder:${where_your_drafts_reside} My notmuch help does not mention folder:, and indeed folder:drafts as a search term is just treated as the words folder and drafts. Perhaps this is implemented

Re: [PATCH] fix sum moar typos

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 14:17:27 -0700, Carl Worth cwo...@cworth.org wrote: Non-text part: multipart/signed On Fri, 24 Jun 2011 22:04:13 +0200, Pieter Praet pie...@praet.org wrote: You however, seem to be incredibly disciplined in this respect, so statistically speaking, you're destined to become

Re: Drafts being tagged as inbox

2011-06-28 Thread Pieter Praet
On Tue, 28 Jun 2011 08:00:53 +0100, Robin Green gree...@greenrd.org wrote: On Tue, 28 Jun 2011 08:34:44 +0200, Pieter Praet pie...@praet.org wrote: notmuch tag -inbox +draft -- folder:${where_your_drafts_reside} My notmuch help does not mention folder:, and indeed folder:drafts as a search

Re: Notmuch scripts

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 21:31:26 -0400, Ben Gamari bgamari.f...@gmail.com wrote: [SNIP] It would be nice, however, if the emacs interface supported hiding tags matching certain patterns (say /\..+/). This should be possible as of last month (eb4e0ea2), by courtesy of Daniel Schoepe [1]. - Ben

Re: bug in emacs-ui ?

2011-06-28 Thread Jani Nikula
Carl Worth cworth@... writes: On Wed, 22 Jun 2011 08:50:23 +0200, Sebastien Binet seb.binet@... wrote: On Tue, 21 Jun 2011 15:09:20 -0700, Carl Worth cworth@... wrote: Perhaps this is an emacs bug? could be, but, I'm using Debian's emacs 23.3.1 and have not encountered the

Re: Notmuch scripts

2011-06-28 Thread Ben Gamari
On Tue, 28 Jun 2011 10:36:59 +0200, Pieter Praet pie...@praet.org wrote: On Mon, 27 Jun 2011 21:31:26 -0400, Ben Gamari bgamari.f...@gmail.com wrote: [SNIP] It would be nice, however, if the emacs interface supported hiding tags matching certain patterns (say /\..+/). This should be

Re: [PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Austin Clements
Quoth myself on Jun 27 at 11:49 pm: Quoth Dmitry Kurochkin on Jun 28 at 5:03 am: EMACSDONE=$TEST_DIRECTORY/emacsdone mkfifo $EMACSDONE coproc emacs --batch --eval '(while t (eval (read)) (write-region \n nil '$EMACSDONE' t 0))' EMACSFD=${COPROC[1]} test_emacs() { echo $1 $EMACSFD

Re: [PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Dmitry Kurochkin
On Tue, 28 Jun 2011 12:22:57 -0400, Austin Clements amdra...@mit.edu wrote: Quoth myself on Jun 27 at 11:49 pm: Quoth Dmitry Kurochkin on Jun 28 at 5:03 am: EMACSDONE=$TEST_DIRECTORY/emacsdone mkfifo $EMACSDONE coproc emacs --batch --eval '(while t (eval (read)) (write-region \n nil

Re: Race condition for '*' command

2011-06-28 Thread Mark Anderson
On Tue, 28 Jun 2011 08:49:06 +0200, Pieter Praet pie...@praet.org wrote: On Sun, 26 Jun 2011 10:00:41 +0100, Robin Green gree...@greenrd.org wrote: On Sat, 25 Jun 2011 16:57:50 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: On Sat, 25 Jun 2011 23:18:52 +0100, Robin Green

[PATCH 0/4] emacs: find non-matching mails, plus small tweaks

2011-06-28 Thread Jani Nikula
Hi, I've found the following features useful with Emacs. Please have a look. And be gentle; these are my first lines of lisp I'm sharing in public, and perhaps not quite as lispy as I'd like them to be... :) Jani Jani Nikula (4): emacs: Add functions and bindings to archive and mark thread as

[PATCH 1/4] emacs: Add functions and bindings to archive and mark thread as read

2011-06-28 Thread Jani Nikula
Add mark as read versions of the archive thread functions to archive and and mark each message in thread as read by removing the inbox and unread tags from the messages. Also add default keybindings A and X for them: shifted a and x also mark as read. --- emacs/notmuch-show.el | 25

[PATCH 2/4] emacs: Add option to make adding saved searches append, not prepend

2011-06-28 Thread Jani Nikula
Add new customization option notmuch-add-saved-search-appends to determine whether new saved searches should be appended to or inserted in front of saved searches. The default remains insert in front. --- emacs/notmuch-hello.el | 10 -- 1 files changed, 8 insertions(+), 2 deletions(-)

[PATCH 3/4] emacs: Add pseudo saved search to match mail that no saved search matches

2011-06-28 Thread Jani Nikula
Add a pseudo saved search that matches all the mail that no other saved search matches. Add new customization option notmuch-saved-searches-nomatch to enable and name the pseudo saved search. --- emacs/notmuch-hello.el | 30 +++--- 1 files changed, 23 insertions(+), 7

[PATCH 4/4] emacs: Add pseudo tag to match all messages that have no tags

2011-06-28 Thread Jani Nikula
Add a pseudo tag that matches all the messages that have no tags. Add new customization option notmuch-tags-nomatch to enable and name the pseudo tag. --- emacs/notmuch-hello.el | 18 -- 1 files changed, 16 insertions(+), 2 deletions(-) diff --git a/emacs/notmuch-hello.el

Re: [PATCH] libnotmuch: fix typos in CLEAN setting, add file

2011-06-28 Thread Carl Worth
On Sat, 25 Jun 2011 09:45:58 -0300, da...@tethera.net wrote: Here is an updated version. Thanks. I've committed this now, (waiting to be pushed until I fix my build---the symbols stuff---so I can actually run make test again). I'm not sure the best way to do a test of the cleaning; maybe we

Re: [PATCH] libnotmuch: build symbols list without relying on gcc -aux-info.

2011-06-28 Thread Carl Worth
On Sun, 26 Jun 2011 23:42:25 -0300, da...@tethera.net wrote: Here is another way of doing this, probably better unless you are very attached to the idea of declaring the API in the header file. Thanks for both of these, David. I've gone ahead and pushed the first, (generating the symbol list

Re: [PATCH 2/2] search --output=files: Output all filenames for each matching message

2011-06-28 Thread Carl Worth
On Fri, 24 Jun 2011 17:26:55 -0600, Mark Anderson ma.sk...@gmail.com wrote: Messages in the database can have multiple files associated with a single message-id, but until now only one filename for each message has been reported by notmuch search --output=files Hi Mark, Thanks for

Re: [PATCH 2/2] search --output=files: Output all filenames for each matching message

2011-06-28 Thread Carl Worth
On Tue, 28 Jun 2011 12:12:52 -0700, Carl Worth cwo...@cworth.org wrote: I think the only thing I would like to see in addition is an update to the documentation to make it explicit that --output=files will emit all filenames for a message that has more than one filename. Actually, I went to go

Re: Debian package not building

2011-06-28 Thread Carl Worth
On Fri, 24 Jun 2011 16:34:02 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: Hey, folks. As of today I am for some reason no longer able to build the Notmuch Debian package. I'm using the same build technique I have been using for a while (git-buildpackage). The tail of the

Re: [PATCH] libnotmuch: build symbols list without relying on gcc -aux-info.

2011-06-28 Thread David Bremner
On Tue, 28 Jun 2011 12:04:39 -0700, Carl Worth cwo...@cworth.org wrote: I assume by better you mean that this one is less fragile to issues like the precise formatting of our notmuch.h header file. Yes, that's what I meant. Meanwhile, though, our API *is* declared in the header file, (that's

Re: Race condition for '*' command

2011-06-28 Thread Carl Worth
On Tue, 28 Jun 2011 11:36:10 -0600, Mark Anderson ma.sk...@gmail.com wrote: On Tue, 28 Jun 2011 08:49:06 +0200, Pieter Praet pie...@praet.org wrote: On Sun, 26 Jun 2011 10:00:41 +0100, Robin Green gree...@greenrd.org wrote: On Sat, 25 Jun 2011 16:57:50 -0700, Jameson Graef Rollins

Re: [PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Carl Worth
On Tue, 28 Jun 2011 20:42:42 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: I would like to hear what other (Carl in particular) think about this. If the consensus is for your approach, I would be happy to implement it. In general, I love the whole series, thanks! I'm looking

Re: notmuch Digest, Vol 20, Issue 57

2011-06-28 Thread Carl Worth
On Mon, 27 Jun 2011 09:43:41 +0200, Sander Boer sanderb...@mauc.nl wrote: For instance, is this a possibility sync gmail tags with notmuch tags ? As fas as I am aware Gmail exposes its tags through imap folders, I am not too thrilled about the mixing of paradigms (folders vs tags) and it would

Re: [PATCH] test:Improve test behaviors when --root is used

2011-06-28 Thread Carl Worth
On Mon, 27 Jun 2011 17:03:03 -0600, Mark Anderson markr.ander...@amd.com wrote: On Mon, 27 Jun 2011 15:50:47 -0500, Austin Clements amdra...@mit.edu wrote: This looks great (modulo one bug, below). I've wanted to run the tests on tmpfs before, but was too lazy to actually fix the tests.

Re: Drafts being tagged as inbox

2011-06-28 Thread Carl Worth
On Mon, 27 Jun 2011 22:50:25 +0100, Robin Green gree...@greenrd.org wrote: My drafts (created using emacs message mode) are being tagged as inbox. However, they are not tagged as drafts. This is because emacs message mode does not store drafts in Maildir format, so there are no Maildir flags

Re: [PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Dmitry Kurochkin
On Tue, 28 Jun 2011 13:10:58 -0700, Carl Worth cwo...@cworth.org wrote: On Tue, 28 Jun 2011 20:42:42 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: I would like to hear what other (Carl in particular) think about this. If the consensus is for your approach, I would be happy to

Re: [PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Dmitry Kurochkin
On Tue, 28 Jun 2011 13:10:58 -0700, Carl Worth cwo...@cworth.org wrote: Non-text part: multipart/signed On Tue, 28 Jun 2011 20:42:42 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: I would like to hear what other (Carl in particular) think about this. If the consensus is for your

[PATCH v2] test:Improve test behaviors when --root is used

2011-06-28 Thread Mark Anderson
Change add_email_corpus, emacs_deliver_message and tests to use $TEST_DIRECTORY instead of '..'. This improves the behavior of the usage of --root=dir, as the assumption of what '..' means will usually be incorrect. Document -root option in README and update valgrind to work with -root.

Re: notmuch Digest, Vol 20, Issue 57

2011-06-28 Thread Sander Boer
Carl Worth cwo...@cworth.org writes: [snip] First, it's important to understand that any friction here comes from Gmail exposing its tags as folders, (which in turn could be the lack of availability of a more tag-aware protocol than imap). I agree that this is the key issue here. I was

[PATCH v3] test:Improve test behaviors when --root is used

2011-06-28 Thread Mark Anderson
Change add_email_corpus, emacs_deliver_message and tests to use $TEST_DIRECTORY instead of '..'. This improves the behavior of the usage of --root=dir, as the assumption of what '..' means will usually be incorrect. Document -root option in README and update valgrind to work with -root. ---

Re: [PATCH 10/10] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Dmitry Kurochkin
On Tue, 28 Jun 2011 15:14:04 -0700, Carl Worth cwo...@cworth.org wrote: On Tue, 28 Jun 2011 08:45:11 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: Before the change, every Emacs test ran in a separate Emacs instance. Starting Emacs many times wastes considerable time and it

Re: notmuch Digest, Vol 20, Issue 57

2011-06-28 Thread Mark Anderson
On Tue, 28 Jun 2011 23:43:52 +0200, Sander Boer sanb...@gmail.com wrote: Carl Worth cwo...@cworth.org writes: Hopefully it's clear enough that you could do the above in a script that loops over all of your existing tags. And if you were doing a one-time switch from Gmail to notmuch

Re: [PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Carl Worth
On Wed, 29 Jun 2011 00:47:37 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: more robust against leaving daemon's around for some reason, etc. Not sure I agree with this. I'm sorry. I wasn't clear. I wasn't advocating one solution over the other. I was just giving an example of

Re: [PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Carl Worth
On Wed, 29 Jun 2011 00:58:41 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: BTW Carl, while we continue our debate, you may consider applying the first 9 patches from the series :) Yes, I already did that. And I would have even installed all 10 (with the debate still going) except

Re: notmuch Digest, Vol 20, Issue 57

2011-06-28 Thread Mark Anderson
On Tue, 28 Jun 2011 23:43:52 +0200, Sander Boer sanb...@gmail.com wrote: Carl Worth cwo...@cworth.org writes: I was hoping that google somehow was able to expose the tags in the All Mail folder, like the headers that are gmail specific: X-pstn-nxpr and X-pstn-nxp (which contains a hash)

Re: [PATCH v3] test:Improve test behaviors when --root is used

2011-06-28 Thread Carl Worth
On Tue, 28 Jun 2011 16:11:32 -0600, Mark Anderson ma.sk...@gmail.com wrote: Change add_email_corpus, emacs_deliver_message and tests to use $TEST_DIRECTORY instead of '..'. ... Document -root option in README and update valgrind to work with -root. Thanks for the features, Mark. These should

Re: notmuch Digest, Vol 20, Issue 57

2011-06-28 Thread Carl Worth
On Tue, 28 Jun 2011 16:38:30 -0600, Mark Anderson ma.sk...@gmail.com wrote: I had briefly considered adding another output format file, just to get a single file for each message in the db, but the file/files distinction feels a bit niggling. Perhaps it should be changed to files and

Re: [PATCH 10/10] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Carl Worth
On Wed, 29 Jun 2011 02:22:31 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: Ouch. I never saw these. Do you get these in crypto tests only? Yes. I just found the bug. The crypto tests use emacs_deliver_message to generate signed/encrypted messages, but that function doesn't call

[PATCH 1/3] test: `notmuch-show-advance-and-archive' with invisible signature

2011-06-28 Thread Dmitry Kurochkin
Add Emacs test to check that `notmuch-show-advance-and-archive' works for the last message in thread with invisible signature. --- This patch series fixes the bug reported by Sebastien in [1]. I was able to reproduce it and confirm that the second patch from this series fixes the problem.

[PATCH 2/3] emacs: improve hidden signatures handling in notmuch-show-advance-and-archive

2011-06-28 Thread Dmitry Kurochkin
Use `previous-single-char-property-change' instead of going through each character by hand and testing it's visibility. This fixes `notmuch-show-advance-and-archive' to work for the last message in thread with hidden signature. --- emacs/notmuch-show.el | 17 + 1 files changed,

[PATCH 3/3] emacs: remove no longer used functions from notmuch-show.el

2011-06-28 Thread Dmitry Kurochkin
Remove `notmuch-show-move-past-invisible-backward' and `notmuch-show-move-past-invisible-forward' functions which are unused. --- emacs/notmuch-show.el |8 1 files changed, 0 insertions(+), 8 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index

Re: bug in emacs-ui ?

2011-06-28 Thread Dmitry Kurochkin
On Tue, 28 Jun 2011 21:38:17 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: Hi Jani. On Tue, 28 Jun 2011 10:35:48 + (UTC), Jani Nikula j...@nikula.org wrote: Carl Worth cworth@... writes: On Wed, 22 Jun 2011 08:50:23 +0200, Sebastien Binet seb.binet@... wrote:

Re: [PATCH 1/2] Add part filename and content-id in notmuch show output if available.

2011-06-28 Thread Carl Worth
On Sun, 29 May 2011 02:03:47 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: Also, the patch adds Content-id to text output format of notmuch show. I don't think I've raised this point before, but I've been tempted to setup a git hook that rejects any commits with a paragraph

[PATCH] emacs: remove unused `point-invisible-p' function

2011-06-28 Thread Dmitry Kurochkin
`point-invisible-p' does not work correctly when `invisible' property is a list. There are standard `invisible-p' and related functions that should be used instead. --- emacs/notmuch-lib.el | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git

Re: [PATCH 1/3] test: `notmuch-show-advance-and-archive' with invisible signature

2011-06-28 Thread Dmitry Kurochkin
On Wed, 29 Jun 2011 05:48:50 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: Add Emacs test to check that `notmuch-show-advance-and-archive' works for the last message in thread with invisible signature. --- This patch series fixes the bug reported by Sebastien in [1]. I was

[PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Dmitry Kurochkin
On Mon, 27 Jun 2011 16:02:12 -0400, Austin Clements wrote: > This looks like a great idea! The test suite has been getting irritating > slow. > > A few minor comments: This patch would be clearer if it the > setq-to-let translation were a separate patch. It would also be worth > adding a big

[PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Dmitry Kurochkin
Austin, On Tue, 28 Jun 2011 00:22:41 +0400, Dmitry Kurochkin wrote: > On Mon, 27 Jun 2011 16:02:12 -0400, Austin Clements > wrote: > > This looks like a great idea! The test suite has been getting irritating > > slow. > > > > A few minor comments: This patch would be clearer if it the > >

[PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Austin Clements
Quoth Dmitry Kurochkin on Jun 28 at 5:03 am: > > > The only way I know to > > > reliably kill a child process is to open a pipe to it and have it exit > > > on its own when it reads EOF. Unfortunately, I couldn't find a way to > > > do this with an emacs daemon (it appears daemon mode

[PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Dmitry Kurochkin
On Mon, 27 Jun 2011 23:49:37 -0400, Austin Clements wrote: > Quoth Dmitry Kurochkin on Jun 28 at 5:03 am: > > > > The only way I know to > > > > reliably kill a child process is to open a pipe to it and have it exit > > > > on its own when it reads EOF. Unfortunately, I couldn't find a way to

[PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Austin Clements
Quoth Dmitry Kurochkin on Jun 28 at 7:59 am: > I am sure that would work, but I do not like the complexity. How about > getting back to standard emacsclient and running a watchdog in the > emacs? Like: > > (defun orphan-watchdog (pid) > "Periodically check that the process with id PID is

[PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Dmitry Kurochkin
On Tue, 28 Jun 2011 00:17:42 -0400, Austin Clements wrote: > Quoth Dmitry Kurochkin on Jun 28 at 7:59 am: > > I am sure that would work, but I do not like the complexity. How about > > getting back to standard emacsclient and running a watchdog in the > > emacs? Like: > > > > (defun

[PATCH 02/10] test: do not set `message-signature' in test_emacs

2011-06-28 Thread Dmitry Kurochkin
It is no longer needed since tests are run in a temporary home directory instead of the user's one. --- test/test-lib.sh |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/test/test-lib.sh b/test/test-lib.sh index cc20f41..3ec388c 100755 --- a/test/test-lib.sh +++

[PATCH 03/10] test: cleanup test_emacs

2011-06-28 Thread Dmitry Kurochkin
Move auxiliary function definition and configuration from command line to test-lib.el. --- test/test-lib.el |8 test/test-lib.sh |9 + 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/test/test-lib.el b/test/test-lib.el index 9439996..344a02e 100644 ---

[PATCH 04/10] test: wrap and indent test_emacs calls

2011-06-28 Thread Dmitry Kurochkin
Most test_emacs calls have long arguments that consist of many expressions. Putting them on a single line makes it hard to read and produces poor diff when they are changed. The patch puts every expression in test_emacs calls on a separate line. --- test/emacs | 124

[PATCH 07/10] test: use emacs_deliver_message in Emacs SMTP send test

2011-06-28 Thread Dmitry Kurochkin
Minor changes to expected results of other Emacs tests were needed because the message Date header changed. --- test/emacs | 34 +- 1 files changed, 9 insertions(+), 25 deletions(-) diff --git a/test/emacs b/test/emacs index 4592005..409e033 100755 ---

[PATCH 09/10] test: generate run_emacs script once on test startup

2011-06-28 Thread Dmitry Kurochkin
Instead of generating auxiliary run_emacs script every time test_emacs is run, do it once in the beginning of the test. Also, use absolute paths in the script to make it more robust. --- test/test-lib.sh | 17 +++-- 1 files changed, 11 insertions(+), 6 deletions(-) diff --git

[PATCH 08/10] test: set variables using `let' instead of `setq' in Emacs tests

2011-06-28 Thread Dmitry Kurochkin
Using `setq' for setting variables in Emacs tests affect other tests that may run in the same Emacs environment. Currently it works because each test is run in a separate Emacs instance. But in the future multiple tests will run in a single Emacs instance. The patch changes all variables to use

[PATCH 10/10] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Dmitry Kurochkin
Before the change, every Emacs test ran in a separate Emacs instance. Starting Emacs many times wastes considerable time and it gets worse as the test suite grows. The patch solves this by using a single Emacs server and emacsclient(1) to run multiple tests. Emacs server is started on the first

[PATCH] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Dmitry Kurochkin
Before the change, every Emacs test ran in a separate Emacs instance. Starting Emacs many times wastes considerable time and it gets worse as the test suite grows. The patch solves this by using a single Emacs server and emacsclient(1) to run multiple tests. Emacs server is started on the first

Drafts being tagged as inbox

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 22:50:25 +0100, Robin Green wrote: > My drafts (created using emacs message mode) are being tagged as > inbox. However, they are not tagged as drafts. This is because emacs > message mode does not store drafts in Maildir format, so there are no > Maildir flags (e.g. D for

[PATCH] test:Improve test behaviors when --root is used

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 16:50:47 -0400, Austin Clements wrote: > This looks great (modulo one bug, below). I've wanted to run the > tests on tmpfs before, but was too lazy to actually fix the tests. > > Given how easy it is to accidentally use "..", I wonder if there's a > way to force people to

Debian package not building

2011-06-28 Thread Pieter Praet
On Fri, 24 Jun 2011 16:34:02 -0700, Jameson Graef Rollins wrote: Non-text part: multipart/mixed Non-text part: multipart/signed > Hey, folks. As of today I am for some reason no longer able to build > the Notmuch Debian package. I'm using the same build technique I have > been using for a while

[PATCH] test:Folder tags shouldn't match after removal of file in given folder

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 11:12:24 -0600, Mark Anderson wrote: > > Test for bug. Current stemming support for notmuch adds extra terms > to the DB which aren't removed when the file renames are detected. > > When folder tags are added to a message, Xapian terms for both XFOLDER > and ZXFOLDER are

Race condition for '*' command

2011-06-28 Thread Pieter Praet
On Sun, 26 Jun 2011 10:00:41 +0100, Robin Green wrote: > On Sat, 25 Jun 2011 16:57:50 -0700, Jameson Graef Rollins finestructure.net> wrote: > > On Sat, 25 Jun 2011 23:18:52 +0100, Robin Green > > wrote: > > > A race condition in the '*' command was noted when it was first > > > proposed. It

Drafts being tagged as inbox

2011-06-28 Thread Robin Green
On Tue, 28 Jun 2011 08:34:44 +0200, Pieter Praet wrote: > notmuch tag -inbox +draft -- folder:${where_your_drafts_reside} My notmuch help does not mention "folder:", and indeed folder:drafts as a search term is just treated as the words folder and drafts. Perhaps this is implemented in git head.

[PATCH] fix sum moar typos

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 14:17:27 -0700, Carl Worth wrote: Non-text part: multipart/signed > On Fri, 24 Jun 2011 22:04:13 +0200, Pieter Praet wrote: > > You however, seem to be incredibly disciplined in this respect, > > so statistically speaking, you're destined to become typo-king, > > and you have

Drafts being tagged as inbox

2011-06-28 Thread Pieter Praet
On Tue, 28 Jun 2011 08:00:53 +0100, Robin Green wrote: > On Tue, 28 Jun 2011 08:34:44 +0200, Pieter Praet wrote: > > notmuch tag -inbox +draft -- folder:${where_your_drafts_reside} > > My notmuch help does not mention "folder:", and indeed folder:drafts as a > search term is just treated as the

Notmuch scripts

2011-06-28 Thread Pieter Praet
On Mon, 27 Jun 2011 21:31:26 -0400, Ben Gamari wrote: > [SNIP] It > would be nice, however, if the emacs interface supported hiding tags > matching certain patterns (say /\..+/). This should be possible as of last month (eb4e0ea2), by courtesy of Daniel Schoepe [1]. > > - Ben > >

bug in emacs-ui ?

2011-06-28 Thread Jani Nikula
Carl Worth writes: > > On Wed, 22 Jun 2011 08:50:23 +0200, Sebastien Binet > wrote: > > On Tue, 21 Jun 2011 15:09:20 -0700, Carl Worth wrote: > > > Perhaps this is an emacs bug? > > could be, but, > > > > > I'm using Debian's emacs 23.3.1 and have not encountered the problem > > > described.

Notmuch scripts

2011-06-28 Thread Ben Gamari
On Tue, 28 Jun 2011 10:36:59 +0200, Pieter Praet wrote: > On Mon, 27 Jun 2011 21:31:26 -0400, Ben Gamari > wrote: > > [SNIP] It > > would be nice, however, if the emacs interface supported hiding tags > > matching certain patterns (say /\..+/). > > This should be possible as of last month

[PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Austin Clements
Quoth myself on Jun 27 at 11:49 pm: > Quoth Dmitry Kurochkin on Jun 28 at 5:03 am: > EMACSDONE=$TEST_DIRECTORY/emacsdone > mkfifo $EMACSDONE > coproc emacs --batch --eval '(while t (eval (read)) (write-region "\n" nil > "'$EMACSDONE'" t 0))' > EMACSFD=${COPROC[1]} > > test_emacs() { > echo

[PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Dmitry Kurochkin
On Tue, 28 Jun 2011 12:22:57 -0400, Austin Clements wrote: > Quoth myself on Jun 27 at 11:49 pm: > > Quoth Dmitry Kurochkin on Jun 28 at 5:03 am: > > EMACSDONE=$TEST_DIRECTORY/emacsdone > > mkfifo $EMACSDONE > > coproc emacs --batch --eval '(while t (eval (read)) (write-region "\n" nil > >

Race condition for '*' command

2011-06-28 Thread Mark Anderson
On Tue, 28 Jun 2011 08:49:06 +0200, Pieter Praet wrote: > On Sun, 26 Jun 2011 10:00:41 +0100, Robin Green > wrote: > > On Sat, 25 Jun 2011 16:57:50 -0700, Jameson Graef Rollins > finestructure.net> wrote: > > > On Sat, 25 Jun 2011 23:18:52 +0100, Robin Green > > > wrote: > > > > A race

bug in emacs-ui ?

2011-06-28 Thread Dmitry Kurochkin
Hi Jani. On Tue, 28 Jun 2011 10:35:48 + (UTC), Jani Nikula wrote: > Carl Worth writes: > > > > > On Wed, 22 Jun 2011 08:50:23 +0200, Sebastien Binet > > wrote: > > > On Tue, 21 Jun 2011 15:09:20 -0700, Carl Worth wrote: > > > > Perhaps this is an emacs bug? > > > could be, but, > > >

[PATCH 0/4] emacs: find non-matching mails, plus small tweaks

2011-06-28 Thread Jani Nikula
Hi, I've found the following features useful with Emacs. Please have a look. And be gentle; these are my first lines of lisp I'm sharing in public, and perhaps not quite as lispy as I'd like them to be... :) Jani Jani Nikula (4): emacs: Add functions and bindings to archive and mark thread as

[PATCH 1/4] emacs: Add functions and bindings to archive and mark thread as read

2011-06-28 Thread Jani Nikula
Add "mark as read" versions of the archive thread functions to archive and and mark each message in thread as read by removing the "inbox" and "unread" tags from the messages. Also add default keybindings A and X for them: shifted a and x also mark as read. --- emacs/notmuch-show.el | 25

[PATCH 2/4] emacs: Add option to make adding saved searches append, not prepend

2011-06-28 Thread Jani Nikula
Add new customization option notmuch-add-saved-search-appends to determine whether new saved searches should be appended to or inserted in front of saved searches. The default remains insert in front. --- emacs/notmuch-hello.el | 10 -- 1 files changed, 8 insertions(+), 2 deletions(-)

[PATCH 4/4] emacs: Add pseudo tag to match all messages that have no tags

2011-06-28 Thread Jani Nikula
Add a pseudo tag that matches all the messages that have no tags. Add new customization option notmuch-tags-nomatch to enable and name the pseudo tag. --- emacs/notmuch-hello.el | 18 -- 1 files changed, 16 insertions(+), 2 deletions(-) diff --git a/emacs/notmuch-hello.el

[PATCH] libnotmuch: fix typos in CLEAN setting, add file

2011-06-28 Thread Carl Worth
but it's not a high priority for me. -Carl -- carl.d.worth at intel.com -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110628/d5138233/attachment.pgp>

[PATCH] libnotmuch: build symbols list without relying on gcc -aux-info.

2011-06-28 Thread Carl Worth
-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110628/970e7520/attachment.pgp>

[PATCH 2/2] search --output=files: Output all filenames for each matching message

2011-06-28 Thread Carl Worth
ith clean output. -- carl.d.worth at intel.com -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110628/464837b9/attachment.pgp>

[PATCH 2/2] search --output=files: Output all filenames for each matching message

2011-06-28 Thread Carl Worth
otmuchmail.org/pipermail/notmuch/attachments/20110628/5fabd97a/attachment.pgp>

Debian package not building

2011-06-28 Thread Carl Worth
el.com -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110628/c276c239/attachment.pgp>

[PATCH] libnotmuch: build symbols list without relying on gcc -aux-info.

2011-06-28 Thread David Bremner
pgp-signature Size: 315 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110628/cbd6136f/attachment.pgp>

Race condition for '*' command

2011-06-28 Thread Carl Worth
tachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110628/b2a5ff0f/attachment.pgp>

[PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Carl Worth
.com -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110628/a65483ac/attachment.pgp>

notmuch Digest, Vol 20, Issue 57

2011-06-28 Thread Carl Worth
l -- carl.d.worth at intel.com -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110628/147c3670/attachment-0001.pgp>

[PATCH] test:Improve test behaviors when --root is used

2011-06-28 Thread Carl Worth
rt -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110628/c60d2f39/attachment.pgp>

Drafts being tagged as inbox

2011-06-28 Thread Carl Worth
pplication/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110628/9451691f/attachment.pgp>

Drafts being tagged as inbox

2011-06-28 Thread Carl Worth
we can start pinning down what went wrong. Then you can finalize your git-bisect run with: git bisect reset Good luck! -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available

[PATCH v2] test:Improve test behaviors when --root is used

2011-06-28 Thread Mark Anderson
Change add_email_corpus, emacs_deliver_message and tests to use $TEST_DIRECTORY instead of '..'. This improves the behavior of the usage of --root=, as the assumption of what '..' means will usually be incorrect. Document -root option in README and update valgrind to work with -root.

[PATCH v3] test:Improve test behaviors when --root is used

2011-06-28 Thread Mark Anderson
Change add_email_corpus, emacs_deliver_message and tests to use $TEST_DIRECTORY instead of '..'. This improves the behavior of the usage of --root=, as the assumption of what '..' means will usually be incorrect. Document -root option in README and update valgrind to work with -root. --- This

[PATCH 10/10] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Carl Worth
tes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110628/91bc0251/attachment.pgp>

notmuch Digest, Vol 20, Issue 57

2011-06-28 Thread Mark Anderson
On Tue, 28 Jun 2011 23:43:52 +0200, "Sander Boer" wrote: > Carl Worth writes:> > > > > Hopefully it's clear enough that you could do the above in a script that > > loops over all of your existing tags. > > > > And if you were doing a one-time switch from Gmail to notmuch that would > > be all

[PATCH 2/2] test: use emacsclient(1) for Emacs tests

2011-06-28 Thread Carl Worth
l/notmuch/attachments/20110628/fe35a6ae/attachment.pgp>

notmuch Digest, Vol 20, Issue 57

2011-06-28 Thread Mark Anderson
On Tue, 28 Jun 2011 23:43:52 +0200, "Sander Boer" wrote: > > Carl Worth writes:> > I was hoping that google somehow was able to expose the tags in the "All > Mail" folder, like the headers that are gmail specific: X-pstn-nxpr and > X-pstn-nxp (which contains a > hash) for instance. I don't

  1   2   >