[notmuch] Notmuch-side support for git send-email --notmuch id:

2009-11-25 Thread Jed Brown
These two patches provide support for features like the one in the subject line. Along with these two patches, I have one for git format-patch (which I will submit upstream soon) that uses the output from notmuch reply --format=headers-only to build a reply with proper referencing, To, and Cc

[notmuch] [PATCH 1/2] notmuch-reply.c: accept the --format=default default option.

2009-11-25 Thread Jed Brown
This factors actual generation of the reply out of notmuch_reply_command into notmuch_reply_format_default(), in preparation for other --format= options. Signed-off-by: Jed Brown --- notmuch-reply.c | 121 +++--- 1 files changed, 78

[notmuch] [PATCH 2/2] notmuch-reply.c: implement notmuch_reply_format_headers_only

2009-11-25 Thread Jed Brown
This command only generates References, To, and Cc headers. The purpose is primarily for use in git send-email --notmuch id: to get proper threading and address the relevant parties. Hooks for other SCMs may come later. Signed-off-by: Jed Brown --- notmuch-reply.c | 70

[notmuch] [PATCH 2/2] search-tags: Add support for search-terms.

2009-11-25 Thread Jan Janak
This patch adds support for search-terms to 'notmuch search-tags'. If no search-term is provided then the command returns a list of all tags from the database. If the user provides one or more search-terms as arguments then the command collects tags from matching messages only. This could be

[notmuch] [PATCH 1/2] lib: New function to collect tags from a list of messages.

2009-11-25 Thread Jan Janak
This patch adds a new function that can be used to collect a list of unique tags from a list of messages. 'notmuch search-tags' uses the function to get a list of tags from messages matching a search-term, but it has the potential to be used elsewhere so we put it in the lib. Signed-off-by: Jan

[notmuch] [PATCH] notmuch.el: When removing tags, offer only those a msg/thread has set.

2009-11-25 Thread Jan Janak
When removing a tag from a message or thread, build a completion buffer which contains only tags that the message or thread has really set. Signed-off-by: Jan Janak --- notmuch.el |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/notmuch.el b/notmuch.el index

[notmuch] [PATCH -v4] notmuch.el: Add face support to search and show mode

2009-11-25 Thread Aneesh Kumar K.V
This add two faces, notmuch-show-subject-face and notmuch-tag-face. The first face is used to show the subject line in the notmuch-show-mode and the second one to show tags in the notmuch-search-mode. We can selectively highlight each tag by setting notmuch-tag-face-alist as below (defface

[notmuch] [PATCH -v2] notmuch.el: Use variable notmuch-search-oldest-first to decide the search order

2009-11-25 Thread Aneesh Kumar K.V
Make sure we use notmuch-search-oldest-first to decide the how the search result should be displayed. This helps to set the value to nil and have latest mail shown first. This also fix the notmuch-folder mode to use the right search order when we select a folder. Also the notmuch command is fixed

[notmuch] [PATCH] notmuch-new: Fix notmuch new to look at files within symbolic links

2009-11-25 Thread Aneesh Kumar K.V
We look at the modified time of the database and the directory to decide whether we need to look at only the subdirectories. ie, if directory modified time is < database modified time then we have already looking at all the files withing the directory. So we just need to iterate through the

[notmuch] vim interface pull request

2009-11-25 Thread Bart Trojanowski
Carl, as you already know, I've spent some time working on a vim-based interface to notmuch. I currently only depend on the 'march:[01]' patches that we talked about briefly on irc. Everything else is isolated to the 'vim' directory and should not effect anyone else. My efforts are cataloged

[notmuch] [PATCH 4/4] notmuch-new: New cmdline option --tag=.

2009-11-25 Thread Karl Wiberg
On Tue, Nov 24, 2009 at 11:10 PM, Jan Janak wrote: > The list of tags to be applied by 'notmuch new' can be configured in > the configuration file. This command line option can be used to > override the list of tags from the coonfiguration file on the command > line. You may repeat the option

[notmuch] [PATCH 1/2] add functions to toggle the nearest backwards button of a given type

2009-11-25 Thread Alexander Botero-Lowry
It was noted that though local expansion was nice, it was also pretty nice to have an easy way to just open the headers without moving to the header button. So this exposes a number of symbols which search backwards in the buffer for the nearest button of the desire type and push it ---

[notmuch] [PATCH 2/2] add some very rudimentary support for handling html parts

2009-11-25 Thread Alexander Botero-Lowry
I was planning to be committing super awesome mime-handling support that would make notmuch rival every mail program ever for pure mime awesomeness. It turns out, that mime is confusing and hard, and the mm-* functions do awesome things like provide different forms based on the number and

[notmuch] notmuch new: Memory problem

2009-11-25 Thread Dominik Epple
Hello, I repeated the procedure (mb2md, notmuch new), but before, I saved all those large emails with backup logs into a separate folder which i deleted before "notmuch new". Then, "notmoch new" works as expected. So the problem stems indeed from too many too large files being present. (I

[notmuch] [PATCH] Silence compiler warning by initializing a variable.

2009-11-25 Thread Jeffrey C. Ollie
If Xapian threw an exception on notmuch_query_count_messages the count variable could be used uninitialized. Initialize count to solve the problem. Signed-off-by: Jeffrey C. Ollie --- lib/query.cc |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/query.cc

[notmuch] [PATCH -v4] notmuch.el: Add face support to search and show mode

2009-11-25 Thread Jeffrey Ollie
On Tue, Nov 24, 2009 at 11:24 PM, Alexander Botero-Lowry wrote: > > This way of detecting the tags seems ok, but I think it would be nicer > if it could be done even more deterministically. :) One idea that be > neat is to have a --format=sexp for notmuch search, which exports sexps > (probably

[notmuch] [PATCH -v4] notmuch.el: Add face support to search and show mode

2009-11-25 Thread Bart Trojanowski
* Jeffrey Ollie [091125 09:33]: > On Tue, Nov 24, 2009 at 11:24 PM, Alexander Botero-Lowry > wrote: > > > > This way of detecting the tags seems ok, but I think it would be nicer > > if it could be done even more deterministically. :) One idea that be > > neat is to have a --format=sexp for

[notmuch] [PATCH -v4] notmuch.el: Add face support to search and show mode

2009-11-25 Thread Aneesh Kumar K. V
On Tue, 24 Nov 2009 21:24:07 -0800, Alexander Botero-Lowry wrote: > On Wed, 25 Nov 2009 10:28:00 +0530, "Aneesh Kumar K.V" linux.vnet.ibm.com> wrote: > > This add two faces, notmuch-show-subject-face and > > notmuch-tag-face. The first face is used to show the subject > > line in the

[notmuch] Using notmuch from Ruby, Python, et al.

2009-11-25 Thread Scott Robinson
I'm hoping this thread will provide a guide for my efforts. I'm writing a web frontend for notmuch in Ruby. But, the output from the notmuch command-line is ambiguous. Therefore, I've started two branches experimenting with different ways of providing hooks. Which of these approaches are likely

[notmuch] Using notmuch from Ruby, Python, et al.

2009-11-25 Thread Jeffrey Ollie
On Wed, Nov 25, 2009 at 10:16 AM, Scott Robinson wrote: > > == Shared Library > > I have modified the build to output a shared library. It's a small change > relying on libtool. > > However, the archives indicate Carl dislikes libtool? Ugh, please, let's not go down the autoconf/automake/libtool

[notmuch] Using notmuch from Ruby, Python, et al.

2009-11-25 Thread Scott Robinson
... Name: libtoolize-notmuch.diff Type: application/octet-stream Size: 3996 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20091125/7d8741a7/attachment.obj>

[notmuch] Using notmuch from Ruby, Python, et al.

2009-11-25 Thread Jeffrey Ollie
On Wed, Nov 25, 2009 at 10:16 AM, Scott Robinson wrote: > > To accomplish this, I used the cJSON library sans parsing logic. I wasn't familiar with cJSON so I went over to take a look. I haven't really examined the code, but I really don't like the fact that you are expected to copy the cJSON

[notmuch] [PATCH] notmuch: Add Maildir directory name as tag name for messages

2009-11-25 Thread Carl Worth
On Sun, 22 Nov 2009 10:33:53 +0100, Michiel Buddingh' wrote: > On the positive side, this allows us to map these flags onto tags, > at least for indexing (the patch at the bottom implements this), and, > if I can pry you away from your principles, later for modification > as well. Hi Michiel,

[notmuch] Using notmuch from Ruby, Python, et al.

2009-11-25 Thread Jeffrey Ollie
On Wed, Nov 25, 2009 at 11:40 AM, Scott Robinson wrote: > > As the json work is a bit more substantial, is it preferable to start a > [PATCH] > thread or link to a git repo? Probably a new [PATCH] thread so that the changes can be discussed. -- Jeff Ollie

[notmuch] Using notmuch from Ruby, Python, et al.

2009-11-25 Thread Scott Robinson
Excerpts from Jeffrey Ollie's message of Wed Nov 25 09:46:47 -0800 2009: > On Wed, Nov 25, 2009 at 10:16 AM, Scott Robinson > wrote: > > > > To accomplish this, I used the cJSON library sans parsing logic. > > I wasn't familiar with cJSON so I went over to take a look. I haven't > really

[notmuch] [PATCH 4/4] notmuch-new: New cmdline option --tag=.

2009-11-25 Thread Jan Janak
Hi Karl, On Wed, Nov 25, 2009 at 7:21 AM, Karl Wiberg wrote: > On Tue, Nov 24, 2009 at 11:10 PM, Jan Janak wrote: >> The list of tags to be applied by 'notmuch new' can be configured in >> the configuration file. This command line option can be used to >> override the list of tags from the

[notmuch] [PATCH 2/2] Recognize reply --format=headers-only-git

2009-11-25 Thread Jed Brown
This currently means the same thing as --format=headers-only, but this name gives more freedom to change --format=headers-only without breaking existing versions of git. --- notmuch-reply.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/notmuch-reply.c

[notmuch] Using notmuch from Ruby, Python, et al.

2009-11-25 Thread Jeffrey Ollie
On Wed, Nov 25, 2009 at 11:58 AM, Scott Robinson wrote: > > I selected cJSON because it was simple for integration. Given how libsha1.c is > included in lib/, I figured adding a depedency wouldn't be very popular. Oh, I noticed libsha1.c as well, and it bugs the heck out of me. I just haven't

[notmuch] Using notmuch from Ruby, Python, et al.

2009-11-25 Thread Scott Robinson
Excerpts from Jeffrey Ollie's message of Wed Nov 25 10:03:05 -0800 2009: > On Wed, Nov 25, 2009 at 11:58 AM, Scott Robinson > wrote: > > Would it be better to link with a previously existing library? > > IMHO yes. I was looking at json-glib because it seems to be the only > C JSON API in

[notmuch] [PATCH] notmuch-new: Option to disable tags from the configuration file.

2009-11-25 Thread Jan Janak
This patch slightly changes behavior of the command line option --tag. Any tags specified by the user on the command line will be added *in addition* to tags configured in the configuration file. This behavior can be changed with the new command line option --no-config-tags. The user can use this

[notmuch] Search results now start appearing "instantly"

2009-11-25 Thread Carl Worth
as fixed). -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/20091125/bfdac07d/attachment-0001.pgp>

[notmuch] [PATCH] notmuch-new: Test if directory looks like Maildir before skipping tmp.

2009-11-25 Thread Jan Janak
'notmuch new' skips directory entries with the name 'tmp'. This is to prevent notmuch from processing possibly incomplete Maildir messages stored in that directory. This patch attempts to refine the feature. If "tmp" entry is found, it first checks if the containing directory looks like a Maildir

[notmuch] [PATCH] notmuch-new: Option to disable tags from the configuration file.

2009-11-25 Thread Bart Trojanowski
Jan, I think your use of STRNCMP_LITERAL here is wrong... > + } else if (STRNCMP_LITERAL (argv[i], "--no-config-tags") == 0) { > + add_config_tags = 0; it will happily match "--no-config-tags-xxx". Can I also suggest including --no-config-tags in the 'notmuch help'

[notmuch] [PATCH 2/4] notmuch: Config option to specify tags to be applied by 'notmuch new'.

2009-11-25 Thread Bart Trojanowski
Jan, I really want this feature to get in, so I am going to do my best to review your code :) Here are some more sticking points... > +char ** > +notmuch_config_get_new_tags (notmuch_config_t *config, size_t *length); If you are not giving over control of the pointer to the caller please

[notmuch] [PATCH] notmuch-new: Test if directory looks like Maildir before skipping tmp.

2009-11-25 Thread Jan Janak
'notmuch new' skips directory entries with the name 'tmp'. This is to prevent notmuch from processing possibly incomplete Maildir messages stored in that directory. This patch attempts to refine the feature. If "tmp" entry is found, it first checks if the containing directory looks like a Maildir

[notmuch] [PATCH] notmuch-new: Test if directory looks like Maildir before skipping tmp.

2009-11-25 Thread Jan Janak
On 25-11 12:12, Scott Robinson wrote: > Excerpts from Jan Janak's message of Wed Nov 25 11:43:15 -0800 2009: > > +/* Test if the directory looks like a Maildir directory. > > + * > > + * Search through the array of directory entries to see if we can find all > > + * three subdirectories typical

[notmuch] [PATCH] notmuch-new: Option to disable tags from the configuration file.

2009-11-25 Thread Jan Janak
Hi Bart, On 25-11 14:55, Bart Trojanowski wrote: > Jan, > > I think your use of STRNCMP_LITERAL here is wrong... > > > + } else if (STRNCMP_LITERAL (argv[i], "--no-config-tags") == 0) { > > + add_config_tags = 0; > > it will happily match "--no-config-tags-xxx". Ahh, I

[notmuch] [PATCH 2/4] notmuch: Config option to specify tags to be applied by 'notmuch new'.

2009-11-25 Thread Jan Janak
Hi Bart, On 25-11 15:56, Bart Trojanowski wrote: > Jan, > > I really want this feature to get in, so I am going to do my best to > review your code :) > > Here are some more sticking points... > > > +char ** > > +notmuch_config_get_new_tags (notmuch_config_t *config, size_t *length); > > If

[notmuch] [PATCH] notmuch.el: Require message mode to avoid undefined function in replies.

2009-11-25 Thread Eric Anholt
Otherwise, those without keithp's .emacs would end up with reply mode not being entered. Suggested by keithp. --- notmuch.el |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/notmuch.el b/notmuch.el index 551048a..42d397a 100644 --- a/notmuch.el +++ b/notmuch.el @@ -49,6

[notmuch] Gmane

2009-11-25 Thread Ingmar Vanhassel
In case anyone's interested, I've submitted this list to gmane: http://dir.gmane.org/gmane.mail.notmuch.general Maybe someone who was on this list from the very beginning can submit an archive to them, for future reference. -Ingmar -- Exherbo KDE, X.org maintainer

[notmuch] [PATCH 1/3] remove Makefile.config from source to allow for custom config.

2009-11-25 Thread Carl Worth
On Sun, 22 Nov 2009 16:58:34 -0500, Jameson Graef Rollins wrote: > Modified ./configure script to generate a Makefile.config if it > doesn't exist. This will allow people to maintain custom > Makefile.config files without causing conflicts with the repository. It's definitely a good idea to get

[notmuch] [PATCH 2/3] modify notmuch_setup_command to return 1 if config file write fails.

2009-11-25 Thread Carl Worth
On Sun, 22 Nov 2009 16:58:35 -0500, Jameson Graef Rollins wrote: > This fixes a small bug in notmuch_setup_command such that it returned > OK and output the setup message footer even if the config file write > step failed. Thanks for the fix. This is pushed. -Carl

[notmuch] [PATCH 3/3] change config file location to be ~/.notmuch/config

2009-11-25 Thread Carl Worth
On Sun, 22 Nov 2009 16:58:36 -0500, Jameson Graef Rollins wrote: > This change creates a ~/.notmuch config directory where the config > file is stored when created with the "setup" command. The use of a > ~/.notmuch config directory creates one place where all notmuch config > files and data can

[notmuch] [PATCH 3/3] change config file location to be ~/.notmuch/config

2009-11-25 Thread Carl Worth
On Sun, 22 Nov 2009 18:14:20 -0500, Jameson Graef Rollins wrote: > else. (I think there's a whole other thread to be started about > this, because I think the config should also hold the location of > the maildir to be indexed, not the path to the index itself, which > then makes