[PATCH v6 6/6] test: add tests for notmuch search --first and --maxitems

2011-11-13 Thread Jani Nikula
Signed-off-by: Jani Nikula --- test/notmuch-test|1 + test/search-limiting | 71 ++ 2 files changed, 72 insertions(+), 0 deletions(-) create mode 100755 test/search-limiting diff --git a/test/notmuch-test b/test/notmuch-test index

[PATCH v6 5/6] test: add tests for notmuch count

2011-11-13 Thread Jani Nikula
Signed-off-by: Jani Nikula --- test/count| 40 test/notmuch-test |1 + 2 files changed, 41 insertions(+), 0 deletions(-) create mode 100755 test/count diff --git a/test/count b/test/count new file mode 100755 index 000..300b171 ---

[PATCH v6 4/6] cli: add support for --output parameter in notmuch count

2011-11-13 Thread Jani Nikula
Add support for --output=messages (which remains the default) and --output=threads to notmuch count. Signed-off-by: Jani Nikula --- NEWS|5 + notmuch-count.c | 18 -- notmuch.1 | 29 + notmuch.c | 20

[PATCH v6 3/6] cli: drop unused code from notmuch count

2011-11-13 Thread Jani Nikula
Remove unused code within #if 0 blocks from notmuch count. Signed-off-by: Jani Nikula --- notmuch-count.c | 32 1 files changed, 0 insertions(+), 32 deletions(-) diff --git a/notmuch-count.c b/notmuch-count.c index 0d700a9..a35be40 100644 ---

[PATCH v6 2/6] cli: add options --first and --maxitems to notmuch search

2011-11-13 Thread Jani Nikula
Add options --first=[-]N and --maxitems=M to notmuch search to determine the first result and maximum number of results to display. Option --maxitems=M limits the maximum number of results to display to M. Option --first=[-]N skips the first N results; with the leading '-' skip until the Nth

[PATCH v6 1/6] lib: add function to get the number of threads matching a search

2011-11-13 Thread Jani Nikula
Add function notmuch_query_count_threads() to get the number of threads matching a search. This is done by performing a search and figuring out the number of unique thread IDs in the matching messages, a significantly heavier operation than notmuch_query_count_messages(). Signed-off-by: Jani

[PATCH v6 0/6] lib/cli: limit number of messages in search results

2011-11-13 Thread Jani Nikula
v6, now with the missing 'notmuch help' documentation spotted by jrollins. Previous version at id:"cover.1321212673.git.jani at nikula.org". BR, Jani. Jani Nikula (6): lib: add function to get the number of threads matching a search cli: add options --first and --maxitems to notmuch search

[PATCH] Store "from" and "subject" headers in the database.

2011-11-13 Thread Jameson Graef Rollins
ication/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/2013/fe58b8d2/attachment-0001.pgp>

[PATCH v5 6/6] test: add tests for notmuch search --first and --maxitems

2011-11-13 Thread Jani Nikula
Signed-off-by: Jani Nikula --- test/notmuch-test|1 + test/search-limiting | 71 ++ 2 files changed, 72 insertions(+), 0 deletions(-) create mode 100755 test/search-limiting diff --git a/test/notmuch-test b/test/notmuch-test index

[PATCH v5 5/6] test: add tests for notmuch count

2011-11-13 Thread Jani Nikula
Signed-off-by: Jani Nikula --- test/count| 40 test/notmuch-test |1 + 2 files changed, 41 insertions(+), 0 deletions(-) create mode 100755 test/count diff --git a/test/count b/test/count new file mode 100755 index 000..300b171 ---

[PATCH v5 4/6] cli: add support for --output parameter in notmuch count

2011-11-13 Thread Jani Nikula
Add support for --output=messages (which remains the default) and --output=threads to notmuch count. Signed-off-by: Jani Nikula --- NEWS|5 + notmuch-count.c | 18 -- notmuch.1 | 29 + 3 files changed, 46 insertions(+), 6

[PATCH v5 3/6] cli: drop unused code from notmuch count

2011-11-13 Thread Jani Nikula
Remove unused code within #if 0 blocks from notmuch count. Signed-off-by: Jani Nikula --- notmuch-count.c | 32 1 files changed, 0 insertions(+), 32 deletions(-) diff --git a/notmuch-count.c b/notmuch-count.c index 0d700a9..a35be40 100644 ---

[PATCH v5 1/6] lib: add function to get the number of threads matching a search

2011-11-13 Thread Jani Nikula
Add function notmuch_query_count_threads() to get the number of threads matching a search. This is done by performing a search and figuring out the number of unique thread IDs in the matching messages, a significantly heavier operation than notmuch_query_count_messages(). Signed-off-by: Jani

[PATCH v5 0/6] lib/cli: limit number of messages in search results

2011-11-13 Thread Jani Nikula
Hi, output limiting patches v5, with only documentation changes since the previous version id:"cover.1320615322.git.jani at nikula.org". The changes are as suggested by Austin Clements: id:"87ehxcufr0.fsf at awakening.csail.mit.edu" id:"87bosgufcf.fsf at awakening.csail.mit.edu" BR, Jani.

[PATCH] add `tag:' prefix to `--output=tags' results

2011-11-13 Thread Jani Nikula
On Sat, 12 Nov 2011 10:17:30 -0500, David Bremner wrote: > On Thu, 30 Jun 2011 10:20:29 +0200, Pieter Praet wrote: > > Alter `do_search_tags()' to prepend each result with `tag:', > > and update affected test. > > > > This makes its output consistent with `do_search_threads()' and > >

[PATCH v6 2/6] cli: add options --first and --maxitems to notmuch search

2011-11-13 Thread Jameson Graef Rollins
--- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/2013/621b66d0/attachment.pgp>

[PATCH] Link libutil using filenmae, rather than using -l.

2011-11-13 Thread David Bremner
On Sun, 13 Nov 2011 12:05:03 -0500, Tom Prince wrote: > glibc includes a libutil, so if the wrong -L options get passed, we > will pick up glibc's version, rather than our own. pushed, d

[PATCH 13/13] test: make smtp-dummy work with Emacs 24

2011-11-13 Thread David Bremner
On Mon, 3 Oct 2011 18:47:27 +0200, Thomas Jost wrote: > In Emacs 24, a space is expected after a SMTP response code. If we don't > respect > that, smtpmail-send-it will wait forever. pushed. d

[PATCH] add `tag:' prefix to `--output=tags' results

2011-11-13 Thread Jameson Graef Rollins
Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/2013/b65d33e4/attachment.pgp>

[PATCH 00/13] Test prereqs and screen-based Emacs tests

2011-11-13 Thread David Bremner
On Tue, 01 Nov 2011 20:54:49 +0100, Pieter Praet wrote: > > I'll be commenting on these modified commits where needed, and have > re-submitted my original series (rebased to current master) in a new > thread [1]. > I'm having trouble sorting out which of these series should be considered for

[PATCH] emacs: breakout notmuch-show-advance functionality from notmuch-show-advance-and-archive

2011-11-13 Thread Jameson Graef Rollins
This patch breaks out much of the functionality of notmuch-show-advance-and-archive into a new function: notmuch-show-advance. This new function does all the advancing through a show buffer that notmuch-show-advance-and-archive did, without all the invasive thread archiving. The return value of

No subject

2011-11-13 Thread Jameson Graef Rollins
I really hate the default behavior of the space bar in notmuch-show; I don't want it to pop out of the thread that i'm currently view, and I especially don't want it to archive threads. The space bar should just be for viewing, not altering. The following patch breaks out the food functionality

Winter is coming [upcoming release 0.10]

2011-11-13 Thread Austin Clements
Quoth David Bremner on Nov 13 at 10:19 am: > - id:"1320599856-24078-1-git-send-email-amdragon at mit.edu" is probably > not a candidate. I'm not ready to push database format changing > patches without some feedback from cworth. It also needs some more > review in general. This was

[PATCH] Link libutil using filenmae, rather than using -l.

2011-11-13 Thread Tom Prince
glibc includes a libutil, so if the wrong -L options get passed, we will pick up glibc's version, rather than our own. --- lib/Makefile.local |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/Makefile.local b/lib/Makefile.local index cb53781..54c4dea 100644 ---

[PATCH v4 4/6] cli: add support for --output parameter in notmuch count

2011-11-13 Thread Austin Clements
Code looks good. Maybe put messages before threads in the documentation (both in the | list and in the descriptions), since that's the default? The output documentation for search does that. "--output" is an unfortunate name for this option since the *output* of count is always a count, but this

[PATCH v4 2/6] cli: add options --first and --maxitems to notmuch search

2011-11-13 Thread Austin Clements
Code LGTM. Some documentation nits below. On Sun, 6 Nov 2011 23:47:11 +0200, Jani Nikula wrote: > Add options --first=[-]N and --maxitems=M to notmuch search to determine > the first result and maximum number of results to display. > > Option --maxitems=M limits the maximum number of results

[PATCH] Don't link libnotmuch if libutil isn't linked in properly.

2011-11-13 Thread Tom Prince
For some reason, on my machine, the link is picking up /usr/lib/libutil.so instead of util/libutil.a. This causes there to be undefined symbols in libnotmuch, making it unuseable. This patch causes the link to fail instead. --- lib/Makefile.local |2 +- 1 files changed, 1 insertions(+), 1

[PATCH 2/2] emacs: add notmuch-show-worker function for specifying crypto processing directly

2011-11-13 Thread Jameson Graef Rollins
The main reason to introduce this new unexposed function is to allow the buffer redisplay crypto switch to behaving in a more expected way. The prefix to notmuch-show-redisplay buffer now switches the crypto processing of the current show buffer, as opposed to switching the logic of the

[PATCH 1/2] emacs: add documentation for notmuch-show crypto-switch option

2011-11-13 Thread Jameson Graef Rollins
--- emacs/notmuch-show.el |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index d395d87..95af32a 100644 --- a/emacs/notmuch-show.el +++ b/emacs/notmuch-show.el @@ -824,7 +824,11 @@ non-nil. The optional BUFFER-NAME

No subject

2011-11-13 Thread Jameson Graef Rollins
So this is incredibly confusing to me, but git is supposedly doing something clever that allows these patches to apply in my repo but not in a clean one. In any event, I'm resending the patches after being applied to my master, which should resolve any conflict and allow them to apply. jamie.

[PATCH] test: do not hide test_emacs errors

2011-11-13 Thread Dmitry Kurochkin
On Sat, 12 Nov 2011 09:57:53 -0500, David Bremner wrote: > On Fri, 1 Jul 2011 06:23:48 +0400, Dmitry Kurochkin gmail.com> wrote: > > Do not redirect test_emacs stderr to /dev/null. Test_emacs uses > > emacsclient(1) now and it does not print unwanted messages (like > > those from `message') to

[PATCH] test: attempt to send QUIT to smtp-dummy in case mail send failed

2011-11-13 Thread Dmitry Kurochkin
On Sat, 12 Nov 2011 23:07:38 +0200, Tomi Ollila wrote: > On Sat, 12 Nov 2011 10:41:28 -0500, Austin Clements > wrote: > > > > Actually, since sending mail is synchronous, there shouldn't be any > > issues with buffering or timing. If Emacs successfully sends the > > message, it will wait for

the present and future of patch tracking for notmuch.

2011-11-13 Thread David Bremner
Size: 315 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/2013/1d9a8c50/attachment.pgp>

[PATCH] test/atomicity: change shebang to '#!/usr/bin/env bash'

2011-11-13 Thread Tomi Ollila
--- test/atomicity |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/test/atomicity b/test/atomicity index 3464238..ad7d4a3 100755 --- a/test/atomicity +++ b/test/atomicity @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/env bash test_description='atomicity' . ./test-lib.sh --

[PATCH] test: attempt to send QUIT to smtp-dummy in case mail send failed

2011-11-13 Thread Tomi Ollila
On Sun, 13 Nov 2011 11:01:24 +0400, Dmitry Kurochkin wrote: > On Sat, 12 Nov 2011 23:07:38 +0200, Tomi Ollila wrote: > > SO, in our cases there are 2 options: > > > > > > 1) add line > > { echo QUIT > /dev/tcp/127.0.0.1/25025; } 2>/dev/null > > before > > wait ${smtp_dummy_pid} > >

Winter is coming [upcoming release 0.10]

2011-11-13 Thread David Bremner
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/2013/12149687/attachment.pgp>

[PATCH] test: do not hide test_emacs errors

2011-11-13 Thread David Bremner
On Sun, 13 Nov 2011 11:06:43 +0400, Dmitry Kurochkin wrote: > > Is this patch still relevant now that we use dtach to run the tests? > > > > It is still relevant. If test_emacs prints any errors, we should see > them. If it does not, we should remove the redirection. pushed, d

[PATCH] exec emacs at the end of run_emacs script

2011-11-13 Thread David Bremner
On Fri, 11 Nov 2011 01:43:03 +0200, Tomi Ollila wrote: > In the last line of run_emacs, exec the emacs process. > With one fork less the process list is (also) neater. pushed.

Re: [PATCH] test: attempt to send QUIT to smtp-dummy in case mail send failed

2011-11-13 Thread Tomi Ollila
On Sun, 13 Nov 2011 11:01:24 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: On Sat, 12 Nov 2011 23:07:38 +0200, Tomi Ollila tomi.oll...@iki.fi wrote: SO, in our cases there are 2 options: 1) add line { echo QUIT /dev/tcp/127.0.0.1/25025; } 2/dev/null before

Re: [PATCH] test: do not hide test_emacs errors

2011-11-13 Thread David Bremner
On Sun, 13 Nov 2011 11:06:43 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: Is this patch still relevant now that we use dtach to run the tests? It is still relevant. If test_emacs prints any errors, we should see them. If it does not, we should remove the redirection.

Re: [PATCH] add `tag:' prefix to `--output=tags' results

2011-11-13 Thread Jani Nikula
On Sat, 12 Nov 2011 10:17:30 -0500, David Bremner da...@tethera.net wrote: On Thu, 30 Jun 2011 10:20:29 +0200, Pieter Praet pie...@praet.org wrote: Alter `do_search_tags()' to prepend each result with `tag:', and update affected test. This makes its output consistent with

Winter is coming [upcoming release 0.10]

2011-11-13 Thread David Bremner
Hi All. According to our sortof-schedule, we should release 0.10 soonish. I'd like to freeze for a few days first, so that means we have time for maybe one or two more non-trivial patches before the freeze. - id:1320846275-28520-1-git-send-email-amdra...@mit.edu is a candidate, but needs

the present and future of patch tracking for notmuch.

2011-11-13 Thread David Bremner
As some of you probably know, I have been using nmbug [1], now in git master, to help track the state of notmuch patches. At this point I think I am about ready to declare the experimental deployment of nmbug a success and make it more official. ## present? - Currently this is running off a

[PATCH] Don't link libnotmuch if libutil isn't linked in properly.

2011-11-13 Thread Tom Prince
For some reason, on my machine, the link is picking up /usr/lib/libutil.so instead of util/libutil.a. This causes there to be undefined symbols in libnotmuch, making it unuseable. This patch causes the link to fail instead. --- lib/Makefile.local |2 +- 1 files changed, 1 insertions(+), 1

Re: [PATCH v4 2/6] cli: add options --first and --maxitems to notmuch search

2011-11-13 Thread Austin Clements
Code LGTM. Some documentation nits below. On Sun, 6 Nov 2011 23:47:11 +0200, Jani Nikula j...@nikula.org wrote: Add options --first=[-]N and --maxitems=M to notmuch search to determine the first result and maximum number of results to display. Option --maxitems=M limits the maximum number

Re: [PATCH v4 4/6] cli: add support for --output parameter in notmuch count

2011-11-13 Thread Austin Clements
Code looks good. Maybe put messages before threads in the documentation (both in the | list and in the descriptions), since that's the default? The output documentation for search does that. --output is an unfortunate name for this option since the *output* of count is always a count, but this

[PATCH] Link libutil using filenmae, rather than using -l.

2011-11-13 Thread Tom Prince
glibc includes a libutil, so if the wrong -L options get passed, we will pick up glibc's version, rather than our own. --- lib/Makefile.local |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/Makefile.local b/lib/Makefile.local index cb53781..54c4dea 100644 ---

Re: [PATCH] Don't link libnotmuch if libutil isn't linked in properly.

2011-11-13 Thread Tom Prince
Perhaps the g++ step in symbol-hiding should in fact be a test. Right now, that step failing isn't captured by the test-suite. Tom ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

Re: Winter is coming [upcoming release 0.10]

2011-11-13 Thread Austin Clements
Quoth David Bremner on Nov 13 at 10:19 am: - id:1320599856-24078-1-git-send-email-amdra...@mit.edu is probably not a candidate. I'm not ready to push database format changing patches without some feedback from cworth. It also needs some more review in general. This was originally

Re: Configure mail responses headers to use Gcc anddddd not Fcc header in Emacs

2011-11-13 Thread Olivier Berger
Olivier Berger olivier.ber...@it-sudparis.eu writes: Hi. I'm using notmuch together with a local dovecot imap server, and with Gnus in Emacs. When sending mails from notmuch, I'd like to be able to have a nnimap+localdovecot:Sentmail added for instance, instead of a Fcc header. Is this

Re: [PATCH 00/13] Test prereqs and screen-based Emacs tests

2011-11-13 Thread David Bremner
On Tue, 01 Nov 2011 20:54:49 +0100, Pieter Praet pie...@praet.org wrote: I'll be commenting on these modified commits where needed, and have re-submitted my original series (rebased to current master) in a new thread [1]. I'm having trouble sorting out which of these series should be

Re: [PATCH 13/13] test: make smtp-dummy work with Emacs 24

2011-11-13 Thread David Bremner
On Mon, 3 Oct 2011 18:47:27 +0200, Thomas Jost schno...@schnouki.net wrote: In Emacs 24, a space is expected after a SMTP response code. If we don't respect that, smtpmail-send-it will wait forever. pushed. d ___ notmuch mailing list

Re: [PATCH] Link libutil using filenmae, rather than using -l.

2011-11-13 Thread David Bremner
On Sun, 13 Nov 2011 12:05:03 -0500, Tom Prince tom.pri...@ualberta.net wrote: glibc includes a libutil, so if the wrong -L options get passed, we will pick up glibc's version, rather than our own. pushed, d ___ notmuch mailing list

[no subject]

2011-11-13 Thread Jameson Graef Rollins
So this is incredibly confusing to me, but git is supposedly doing something clever that allows these patches to apply in my repo but not in a clean one. In any event, I'm resending the patches after being applied to my master, which should resolve any conflict and allow them to apply. jamie.

[PATCH 1/2] emacs: add documentation for notmuch-show crypto-switch option

2011-11-13 Thread Jameson Graef Rollins
--- emacs/notmuch-show.el |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index d395d87..95af32a 100644 --- a/emacs/notmuch-show.el +++ b/emacs/notmuch-show.el @@ -824,7 +824,11 @@ non-nil. The optional BUFFER-NAME

[PATCH v5 1/6] lib: add function to get the number of threads matching a search

2011-11-13 Thread Jani Nikula
Add function notmuch_query_count_threads() to get the number of threads matching a search. This is done by performing a search and figuring out the number of unique thread IDs in the matching messages, a significantly heavier operation than notmuch_query_count_messages(). Signed-off-by: Jani

[PATCH v5 2/6] cli: add options --first and --maxitems to notmuch search

2011-11-13 Thread Jani Nikula
Add options --first=[-]N and --maxitems=M to notmuch search to determine the first result and maximum number of results to display. Option --maxitems=M limits the maximum number of results to display to M. Option --first=[-]N skips the first N results; with the leading '-' skip until the Nth

[PATCH v5 3/6] cli: drop unused code from notmuch count

2011-11-13 Thread Jani Nikula
Remove unused code within #if 0 blocks from notmuch count. Signed-off-by: Jani Nikula j...@nikula.org --- notmuch-count.c | 32 1 files changed, 0 insertions(+), 32 deletions(-) diff --git a/notmuch-count.c b/notmuch-count.c index 0d700a9..a35be40 100644 ---

[PATCH v5 6/6] test: add tests for notmuch search --first and --maxitems

2011-11-13 Thread Jani Nikula
Signed-off-by: Jani Nikula j...@nikula.org --- test/notmuch-test|1 + test/search-limiting | 71 ++ 2 files changed, 72 insertions(+), 0 deletions(-) create mode 100755 test/search-limiting diff --git a/test/notmuch-test

[PATCH v6 1/6] lib: add function to get the number of threads matching a search

2011-11-13 Thread Jani Nikula
Add function notmuch_query_count_threads() to get the number of threads matching a search. This is done by performing a search and figuring out the number of unique thread IDs in the matching messages, a significantly heavier operation than notmuch_query_count_messages(). Signed-off-by: Jani

[PATCH v6 3/6] cli: drop unused code from notmuch count

2011-11-13 Thread Jani Nikula
Remove unused code within #if 0 blocks from notmuch count. Signed-off-by: Jani Nikula j...@nikula.org --- notmuch-count.c | 32 1 files changed, 0 insertions(+), 32 deletions(-) diff --git a/notmuch-count.c b/notmuch-count.c index 0d700a9..a35be40 100644 ---

[PATCH v6 4/6] cli: add support for --output parameter in notmuch count

2011-11-13 Thread Jani Nikula
Add support for --output=messages (which remains the default) and --output=threads to notmuch count. Signed-off-by: Jani Nikula j...@nikula.org --- NEWS|5 + notmuch-count.c | 18 -- notmuch.1 | 29 + notmuch.c |

[PATCH v6 5/6] test: add tests for notmuch count

2011-11-13 Thread Jani Nikula
Signed-off-by: Jani Nikula j...@nikula.org --- test/count| 40 test/notmuch-test |1 + 2 files changed, 41 insertions(+), 0 deletions(-) create mode 100755 test/count diff --git a/test/count b/test/count new file mode 100755 index

[no subject]

2011-11-13 Thread Jameson Graef Rollins
I really hate the default behavior of the space bar in notmuch-show; I don't want it to pop out of the thread that i'm currently view, and I especially don't want it to archive threads. The space bar should just be for viewing, not altering. The following patch breaks out the food functionality

[PATCH] emacs: breakout notmuch-show-advance functionality from notmuch-show-advance-and-archive

2011-11-13 Thread Jameson Graef Rollins
This patch breaks out much of the functionality of notmuch-show-advance-and-archive into a new function: notmuch-show-advance. This new function does all the advancing through a show buffer that notmuch-show-advance-and-archive did, without all the invasive thread archiving. The return value of

Re: [PATCH v2] emacs: Let the user choose where to compose new mails

2011-11-13 Thread Jameson Graef Rollins
Hey, Thomas et. al. I've tested this patch and it works great! The customization interface is very clean now, and the newly created windows/frames close automatically when the message is sent. Very nice. Thanks so much, Thomas, for this nice ui feature. The commit message is a little sparse,

[PATCH 0/6] cli getoptification

2011-11-13 Thread Jani Nikula
Hi, this set switches to getopt_long for parsing command line options where applicable, and hopefully makes David happy. ;) This is on top of id:cover.1321217854.git.j...@nikula.org. This was a fairly quick exercise, so watch out for bugs. getopt_long does have subtle differences from the

[PATCH 1/6] cli: notmuch new: use getopt_long for parsing command line options

2011-11-13 Thread Jani Nikula
Signed-off-by: Jani Nikula j...@nikula.org --- notmuch-client.h |1 + notmuch-new.c| 20 +++- 2 files changed, 16 insertions(+), 5 deletions(-) diff --git a/notmuch-client.h b/notmuch-client.h index b50cb38..eb16f0d 100644 --- a/notmuch-client.h +++ b/notmuch-client.h

[PATCH 2/6] cli: notmuch search: use getopt_long for parsing command line options

2011-11-13 Thread Jani Nikula
Signed-off-by: Jani Nikula j...@nikula.org --- notmuch-search.c | 90 - 1 files changed, 48 insertions(+), 42 deletions(-) diff --git a/notmuch-search.c b/notmuch-search.c index c62a594..4f7384a 100644 --- a/notmuch-search.c +++

[PATCH 3/6] cli: notmuch show: use getopt_long for parsing command line options

2011-11-13 Thread Jani Nikula
Signed-off-by: Jani Nikula j...@nikula.org --- notmuch-show.c | 61 --- 1 files changed, 35 insertions(+), 26 deletions(-) diff --git a/notmuch-show.c b/notmuch-show.c index 603992a..73b4557 100644 --- a/notmuch-show.c +++ b/notmuch-show.c @@

[PATCH 4/6] cli: notmuch count: use getopt_long for parsing command line options

2011-11-13 Thread Jani Nikula
Signed-off-by: Jani Nikula j...@nikula.org --- notmuch-count.c | 33 ++--- 1 files changed, 18 insertions(+), 15 deletions(-) diff --git a/notmuch-count.c b/notmuch-count.c index 20ce334..4987ca8 100644 --- a/notmuch-count.c +++ b/notmuch-count.c @@ -28,34 +28,37 @@

[PATCH 6/6] test: emacs-large-search-buffer: expected results according to getopt_long

2011-11-13 Thread Jani Nikula
Signed-off-by: Jani Nikula j...@nikula.org --- test/emacs-large-search-buffer |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/test/emacs-large-search-buffer b/test/emacs-large-search-buffer index 6095e9d..1bdffac 100755 --- a/test/emacs-large-search-buffer +++

Re: [PATCH] add `tag:' prefix to `--output=tags' results

2011-11-13 Thread Jameson Graef Rollins
On Sat, 12 Nov 2011 10:17:30 -0500, David Bremner da...@tethera.net wrote: What do people think about this change? Personally I will have change some scripts to not add tag:, but it isn't that big of a deal. I would actually prefer to see the prefixes removed from the messages and threads

Re: [PATCH v6 2/6] cli: add options --first and --maxitems to notmuch search

2011-11-13 Thread Jameson Graef Rollins
On Sun, 13 Nov 2011 23:15:31 +0200, Jani Nikula j...@nikula.org wrote: Option --maxitems=M limits the maximum number of results to display to M. This is bike shedding, but can this option be called --limit instead of maxitems? Somehow that's more intuitive to me. jamie. pgpj1fvibw0LJ.pgp

Re: [PATCH] Store from and subject headers in the database.

2011-11-13 Thread Jameson Graef Rollins
On Sun, 6 Nov 2011 12:17:36 -0500, Austin Clements amdra...@mit.edu wrote: This is a rebase and cleanup of Istvan Marko's patch from id:m3pqnj2j7a@zsu.kismala.com Search retrieves these headers for every message in the search results. Previously, this required opening and parsing every

Re: [PATCH] add `tag:' prefix to `--output=tags' results

2011-11-13 Thread Dmitry Kurochkin
On Sun, 13 Nov 2011 15:00:56 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: On Sat, 12 Nov 2011 10:17:30 -0500, David Bremner da...@tethera.net wrote: What do people think about this change? Personally I will have change some scripts to not add tag:, but it isn't that big of