[PATCH 6/6] cli: add top level --config=FILE option

Let the user specify the config file on the command line. --- notmuch.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/notmuch.c b/notmuch.c index f4bfeaa..71cd0d6 100644 --- a/notmuch.c +++ b/notmuch.c @@ -258,6 +258,7 @@ main (int argc, char *argv[]) char *talloc_

[PATCH 5/6] cli: move config open/close to main() from subcommands

This allows specifying config file as a top level argument to notmuch, and generally makes it possible to override config file options in main(), without having to touch the subcommands. This also makes notmuch config the talloc context for subcommands. --- notmuch-client.h | 30 +++---

[PATCH 4/6] cli: plug main notmuch command into subcommand machinery

This also allows the main notmuch command to have arguments (though they are not used). --- notmuch.c | 26 +- 1 file changed, 17 insertions(+), 9 deletions(-) diff --git a/notmuch.c b/notmuch.c index c47b998..67d5772 100644 --- a/notmuch.c +++ b/notmuch.c @@ -34,7 +34,1

[PATCH 3/6] cli: abstract subcommand finding into a new function

Clean up code. --- notmuch.c | 68 +++-- 1 file changed, 35 insertions(+), 33 deletions(-) diff --git a/notmuch.c b/notmuch.c index b413b53..c47b998 100644 --- a/notmuch.c +++ b/notmuch.c @@ -70,6 +70,18 @@ static command_t commands[] = {

[PATCH 2/6] cli: make notmuch_config_open() "is new" parameter input only

Use the notmuch_config_is_new() function instead. --- notmuch-client.h |2 +- notmuch-config.c | 32 +++- notmuch-count.c |2 +- notmuch-dump.c |2 +- notmuch-new.c|2 +- notmuch-reply.c |2 +- notmuch-restore.c|

[PATCH 1/6] cli: keep track of whether the config is newly created

Add notmuch_config_is_new() accessor function to check this. --- notmuch-client.h |3 +++ notmuch-config.c | 11 +++ 2 files changed, 14 insertions(+) diff --git a/notmuch-client.h b/notmuch-client.h index 5f28836..07367e0 100644 --- a/notmuch-client.h +++ b/notmuch-client.h @@ -256

[PATCH 0/6] notmuch cli config changes

Hi all, the goal here is to add support for --config=FILE option at the notmuch top level (e.g. 'notmuch --config=FILE search foo'). In order to achieve this neatly, I ended up moving config open/close to main() from subcommands. This isn't a bad thing, because all notmuch commands opened the confi

[PATCH 2/2] NEWS: News for 0.15.2

From: David Bremner Another couple of build fixes. --- NEWS | 14 ++ 1 file changed, 14 insertions(+) diff --git a/NEWS b/NEWS index 97f2305..3ba5a50 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,17 @@ +Notmuch 0.15.2 (2013-01-29) +=== + +Build fixes +---

[PATCH 1/2] test: delay watchdog checks in emacs.

From: David Bremner Instead of checking immediately for the watched process, delay a minute, or in the case that process-attributes returns nil, for two minutes. This is intended to cope with the case that process-attributes is unimplimented, and returns always returns nil. In this case, the wat

another bug fix release: 0.15.2, in progress.

I plan to do another quick bug fix release, with aidecoe's parallel build fix and a patch originally from Tomi to make the Emacs tests not commit suicide right away on certain OSes. I didn't include the parallel building patch here since it's already in master.

[PATCH 1/2] man: document NOTMUCH_TALLOC_REPORT environment variable

Jameson Graef Rollins writes: > > Not for this series, but it might be nice to document usage of the > NOTMUCH_DEBUG_QUERY env var as well. > Not a bad idea. This should (also?) be documented in the library documentation, since it is handled there, and also available e.g. to bindings users. d

Re: another bug fix release: 0.15.2, in progress.

On Wed, Jan 30 2013, da...@tethera.net wrote: > I plan to do another quick bug fix release, with aidecoe's parallel > build fix and a patch originally from Tomi to make the Emacs tests not > commit suicide right away on certain OSes. LGTM. > I didn't include the parallel building patch here sinc

another bug fix release: 0.15.2, in progress.

I plan to do another quick bug fix release, with aidecoe's parallel build fix and a patch originally from Tomi to make the Emacs tests not commit suicide right away on certain OSes. I didn't include the parallel building patch here since it's already in master. ___

[PATCH 2/2] NEWS: News for 0.15.2

From: David Bremner Another couple of build fixes. --- NEWS | 14 ++ 1 file changed, 14 insertions(+) diff --git a/NEWS b/NEWS index 97f2305..3ba5a50 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,17 @@ +Notmuch 0.15.2 (2013-01-29) +=== + +Build fixes +---

[PATCH 1/2] test: delay watchdog checks in emacs.

From: David Bremner Instead of checking immediately for the watched process, delay a minute, or in the case that process-attributes returns nil, for two minutes. This is intended to cope with the case that process-attributes is unimplimented, and returns always returns nil. In this case, the wat

minor fixes for talloc leak logging

On Tue, Jan 29 2013, david at tethera.net wrote: > These are the remaining unapplied patches from > > id:1358619958-21209-1-git-send-email-david at tethera.net > > They are rebased to omit the actual command line argument, as suggested by > > id:87d2wzwmd9.fsf at servo.finestructure.n

It's good to be "back"!

e: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20130129/3705c869/attachment.pgp>

Reply all - issue

sting thread-breaking? There wasn't the last time I looked at the code closely, (but admittedly, that was a while ago). -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available UR

[PATCH 5/6] cli: move config open/close to main() from subcommands

This allows specifying config file as a top level argument to notmuch, and generally makes it possible to override config file options in main(), without having to touch the subcommands. This also makes notmuch config the talloc context for subcommands. --- notmuch-client.h | 30 +++---

[PATCH 6/6] cli: add top level --config=FILE option

Let the user specify the config file on the command line. --- notmuch.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/notmuch.c b/notmuch.c index f4bfeaa..71cd0d6 100644 --- a/notmuch.c +++ b/notmuch.c @@ -258,6 +258,7 @@ main (int argc, char *argv[]) char *talloc_

[PATCH 4/6] cli: plug main notmuch command into subcommand machinery

This also allows the main notmuch command to have arguments (though they are not used). --- notmuch.c | 26 +- 1 file changed, 17 insertions(+), 9 deletions(-) diff --git a/notmuch.c b/notmuch.c index c47b998..67d5772 100644 --- a/notmuch.c +++ b/notmuch.c @@ -34,7 +34,1

[PATCH 3/6] cli: abstract subcommand finding into a new function

Clean up code. --- notmuch.c | 68 +++-- 1 file changed, 35 insertions(+), 33 deletions(-) diff --git a/notmuch.c b/notmuch.c index b413b53..c47b998 100644 --- a/notmuch.c +++ b/notmuch.c @@ -70,6 +70,18 @@ static command_t commands[] = {

[PATCH 2/6] cli: make notmuch_config_open() "is new" parameter input only

Use the notmuch_config_is_new() function instead. --- notmuch-client.h |2 +- notmuch-config.c | 32 +++- notmuch-count.c |2 +- notmuch-dump.c |2 +- notmuch-new.c|2 +- notmuch-reply.c |2 +- notmuch-restore.c|

[PATCH 1/6] cli: keep track of whether the config is newly created

Add notmuch_config_is_new() accessor function to check this. --- notmuch-client.h |3 +++ notmuch-config.c | 11 +++ 2 files changed, 14 insertions(+) diff --git a/notmuch-client.h b/notmuch-client.h index 5f28836..07367e0 100644 --- a/notmuch-client.h +++ b/notmuch-client.h @@ -256

[PATCH 0/6] notmuch cli config changes

Hi all, the goal here is to add support for --config=FILE option at the notmuch top level (e.g. 'notmuch --config=FILE search foo'). In order to achieve this neatly, I ended up moving config open/close to main() from subcommands. This isn't a bad thing, because all notmuch commands opened the confi

Re: [PATCH 1/2] man: document NOTMUCH_TALLOC_REPORT environment variable

Jameson Graef Rollins writes: > > Not for this series, but it might be nice to document usage of the > NOTMUCH_DEBUG_QUERY env var as well. > Not a bad idea. This should (also?) be documented in the library documentation, since it is handled there, and also available e.g. to bindings users. d _

Re: [PATCH 1/2] man: document NOTMUCH_TALLOC_REPORT environment variable

On Tue, Jan 29 2013, da...@tethera.net wrote: > +.B NOTMUCH_TALLOC_REPORT > +Location to write a talloc memory usage report. See > +.B talloc_enable_leak_report_full > +in \fBtalloc\fR(3) > +for more information. Not for this series, but it might be nice to document usage of the NOTMUCH_DEBUG_QUER

[PATCH 1/2] man: document NOTMUCH_TALLOC_REPORT environment variable

nt usage of the NOTMUCH_DEBUG_QUERY env var as well. jamie. -- 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/20130129/fdfd19b8/attachment.pgp>

[PATCH 2/2] CLI: add simple error handling for talloc logging

From: David Bremner This really should have been there before. I think it's better to do the actual operation and then possibly fail writing the memory log, but it would not be too hard to change it to abort earlier. --- notmuch.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions

[PATCH 1/2] man: document NOTMUCH_TALLOC_REPORT environment variable

From: David Bremner --- man/man1/notmuch.1 |8 1 file changed, 8 insertions(+) diff --git a/man/man1/notmuch.1 b/man/man1/notmuch.1 index 0805be8..55e2d16 100644 --- a/man/man1/notmuch.1 +++ b/man/man1/notmuch.1 @@ -147,6 +147,14 @@ behavior of notmuch. .B NOTMUCH_CONFIG Specifie

minor fixes for talloc leak logging

These are the remaining unapplied patches from id:1358619958-21209-1-git-send-email-david at tethera.net They are rebased to omit the actual command line argument, as suggested by id:87d2wzwmd9.fsf at servo.finestructure.net

Re: minor fixes for talloc leak logging

On Tue, Jan 29 2013, da...@tethera.net wrote: > These are the remaining unapplied patches from > > id:1358619958-21209-1-git-send-email-da...@tethera.net > > They are rebased to omit the actual command line argument, as suggested by > > id:87d2wzwmd9@servo.finestructure.net LGTM

[PATCH 2/2] CLI: add simple error handling for talloc logging

From: David Bremner This really should have been there before. I think it's better to do the actual operation and then possibly fail writing the memory log, but it would not be too hard to change it to abort earlier. --- notmuch.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions

[PATCH 1/2] man: document NOTMUCH_TALLOC_REPORT environment variable

From: David Bremner --- man/man1/notmuch.1 |8 1 file changed, 8 insertions(+) diff --git a/man/man1/notmuch.1 b/man/man1/notmuch.1 index 0805be8..55e2d16 100644 --- a/man/man1/notmuch.1 +++ b/man/man1/notmuch.1 @@ -147,6 +147,14 @@ behavior of notmuch. .B NOTMUCH_CONFIG Specifie

minor fixes for talloc leak logging

These are the remaining unapplied patches from id:1358619958-21209-1-git-send-email-da...@tethera.net They are rebased to omit the actual command line argument, as suggested by id:87d2wzwmd9@servo.finestructure.net ___ notmuch mailing