Re: [PATCH 0/4] Quoting HTML-only emails in replies redux

2012-01-15 Thread Aaron Ecay
On Sun, 15 Jan 2012 09:26:59 +, David Edmondson wrote: > Given that we're now doing a bunch of work in emacs as part of the reply > setup, why not just grab the content of the original message from the > show buffer and quote that? > > The last time that approach was discussed Carl was agains

Emacs: Crypto: How to get automatic encryption?

2012-01-15 Thread Gregor Zattler
s this only one without further ado.] Ciao; Gregor -- 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/20120115/d8750632/attachment.pgp>

[PATCH] emacs: Don't attempt to colour tags in `notmuch-show-mode'.

2012-01-15 Thread David Bremner
On Tue, 27 Dec 2011 16:47:14 +, David Edmondson wrote: > The tags were coloured using text properties. Unfortunately that text > (the header line) also has an overlay, which overrides the text > properties. There's not point in applying text properties that will > never be seen. Pushed. d

[PATCH v2] emacs: Cycle through notmuch buffers rather than jumping to the last.

2012-01-15 Thread David Bremner
On Wed, 28 Dec 2011 08:29:58 +, David Edmondson wrote: > As suggested by j4ni in #notmuch, rename > `notmuch-jump-to-recent-buffer' as `notmuch-cycle-notmuch-buffers' and > have it behave accordingly. pushed, d

[PATCH] test: don't bail out of `run_emacs' too early when missing prereqs

2012-01-15 Thread David Bremner
On Sat, 14 Jan 2012 10:09:37 +0100, Pieter Praet wrote: > When running the Emacs tests in verbose mode, only the first missing > prereq is reported because the `run_emacs' function is short-circuited > early: pushed. d

[PATCH v2] emacs: Cycle through notmuch buffers rather than jumping to the last.

2012-01-15 Thread Austin Clements
Quoth David Edmondson on Jan 15 at 11:55 am: > On Sat, 14 Jan 2012 10:29:31 +0100, Pieter Praet wrote: > > Might I ask, to what key(chord) have you bound this ? Due to its > > usefulness, I'm inclined to bind it to [SPC], but on second though, > > that might be a bit on the intense side... > > C

[PATCH 2/2] cli: pick the user's address in a group list as from address

2012-01-15 Thread Tomi Ollila
On Sat, 14 Jan 2012 23:49:50 +0200, Jani Nikula wrote: > Messages received to a group list were not replied to using the from > address in the list. Fix it. > > Signed-off-by: Jani Nikula > --- Both LGTM. Tomi

[PATCH v3 04/10] notmuch-dump: add --format=(notmuch|sup)

2012-01-15 Thread David Bremner
On Sun, 15 Jan 2012 15:35:11 -0500, Austin Clements wrote: > We definitely need a round-trip-able dump format. Did you consider > using JSON to allow for future flexibility (e.g., expansion of what we > store in the database) and so we don't have to invent our own > encodings? A JSON format wou

Re: [PATCH] emacs: Don't attempt to colour tags in `notmuch-show-mode'.

2012-01-15 Thread David Bremner
On Tue, 27 Dec 2011 16:47:14 +, David Edmondson wrote: > The tags were coloured using text properties. Unfortunately that text > (the header line) also has an overlay, which overrides the text > properties. There's not point in applying text properties that will > never be seen. Pushed. d __

Re: [PATCH v2] emacs: Cycle through notmuch buffers rather than jumping to the last.

2012-01-15 Thread Austin Clements
Quoth David Edmondson on Jan 15 at 11:55 am: > On Sat, 14 Jan 2012 10:29:31 +0100, Pieter Praet wrote: > > Might I ask, to what key(chord) have you bound this ? Due to its > > usefulness, I'm inclined to bind it to [SPC], but on second though, > > that might be a bit on the intense side... > > C

Re: [PATCH v2] emacs: Cycle through notmuch buffers rather than jumping to the last.

2012-01-15 Thread David Bremner
On Wed, 28 Dec 2011 08:29:58 +, David Edmondson wrote: > As suggested by j4ni in #notmuch, rename > `notmuch-jump-to-recent-buffer' as `notmuch-cycle-notmuch-buffers' and > have it behave accordingly. pushed, d ___ notmuch mailing list notmuch@not

Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs

2012-01-15 Thread David Bremner
On Sat, 14 Jan 2012 10:09:37 +0100, Pieter Praet wrote: > When running the Emacs tests in verbose mode, only the first missing > prereq is reported because the `run_emacs' function is short-circuited > early: pushed. d ___ notmuch mailing list notmuch@

[PATCH] test: don't bail out of `run_emacs' too early when missing prereqs

2012-01-15 Thread Dmitry Kurochkin
On Sat, 14 Jan 2012 10:07:41 +0100, Pieter Praet wrote: > On Thu, 12 Jan 2012 21:34:29 +0400, Dmitry Kurochkin gmail.com> wrote: > > On Thu, 12 Jan 2012 18:16:59 +0100, Pieter Praet > > wrote: > > > When running the Emacs tests in verbose mode, only the first missing > > > prereq is reported be

[PATCH] test: don't bail out of `run_emacs' too early when missing prereqs

2012-01-15 Thread Dmitry Kurochkin
On Sat, 14 Jan 2012 10:09:37 +0100, Pieter Praet wrote: > When running the Emacs tests in verbose mode, only the first missing > prereq is reported because the `run_emacs' function is short-circuited > early: > > #+begin_example > emacs: Testing emacs interface > missing prerequisites:

Partial words on notmuch search?

2012-01-15 Thread mailingli...@nawaz.org
Andrei Popescu writes: > After reading all the docs and a few experiments I don't see how to do > searches on partial words. If I missed the fine manual mentioning that > I'd be glad to RTFM, just please point me to the right FM :) That's OK - I asked the same question once. Use '*'. For exa

[RFC] vim plugin rewrite II

2012-01-15 Thread Anton Khirnov
On Sat, 14 Jan 2012 22:15:07 -0400, David Bremner wrote: > On Sat, 14 Jan 2012 08:54:43 +0100, anton at khirnov.net wrote: > > > > The advantages over current vim client are still the following: > > * sending and displaying/saving attachments > > * much better unicode support > > * tag name and

Re: Partial words on notmuch search?

2012-01-15 Thread mailinglists
Andrei Popescu writes: > After reading all the docs and a few experiments I don't see how to do > searches on partial words. If I missed the fine manual mentioning that > I'd be glad to RTFM, just please point me to the right FM :) That's OK - I asked the same question once. Use '*'. For exa

Re: [PATCH v3 04/10] notmuch-dump: add --format=(notmuch|sup)

2012-01-15 Thread David Bremner
On Sun, 15 Jan 2012 15:35:11 -0500, Austin Clements wrote: > We definitely need a round-trip-able dump format. Did you consider > using JSON to allow for future flexibility (e.g., expansion of what we > store in the database) and so we don't have to invent our own > encodings? A JSON format wou

[PATCH v3 04/10] notmuch-dump: add --format=(notmuch|sup)

2012-01-15 Thread Austin Clements
Quoth David Bremner on Jan 14 at 9:40 pm: > From: David Bremner > > sup is the old format, and remains the default. > > Each line of the notmuch format is "msg_id tag tag...tag" where each > space seperated token is 'hex-encoded' to remove troubling characters. > In particular this format won't

[PATCH] Fix dependency generation for compat, test, and util

2012-01-15 Thread Austin Clements
This adds source files in compat, test, and util to SRCS so that the top-level Makefile.local will generate dependency files for them. --- compat/Makefile.local |2 ++ test/Makefile.local |1 + util/Makefile.local |1 + 3 files changed, 4 insertions(+), 0 deletions(-) diff --git a

[PATCH] Fix dependency generation for CLI sources

2012-01-15 Thread Austin Clements
Previously, the dependency file list was generated before the CLI sources were added to SRCS, so dependency files weren't generated for CLI sources. This moves that code to after the CLI sources are added. --- Makefile.local |8 1 files changed, 4 insertions(+), 4 deletions(-) diff

Re: Emacs: Crypto: How to get automatic encryption?

2012-01-15 Thread Gregor Zattler
Hi David, notmuch developers, On Wed, 04 Jan 2012 08:48:43 +, David Edmondson wrote: > On Wed, 04 Jan 2012 08:36:39 +, David Edmondson wrote: > > (if (and encrypt (dme:message-determine-encryption)) > > Oops. `encrypt' is set to `t' earlier if the code decides that I want to > en

[PATCH 4/4] emacs: Use the new JSON reply format.

2012-01-15 Thread Adam Wolfe Gordon
Hi Dmitry, On Thu, Jan 12, 2012 at 01:33, Dmitry Kurochkin wrote: > I did not follow the rest of the discussion, so sorry if I missed > something obvious. ?But why can't we render HTML parts in replies the > same way we do in notmuch-show (using `mm-display-part')? ?That should > not introduce a

Partial words on notmuch search?

2012-01-15 Thread Andrei Popescu
Hi everybody, I'm using notmuch 0.10.2 from Debian backports with Stefano Zacchiroli's mutt integration (but I've also tried the command line interface). After reading all the docs and a few experiments I don't see how to do searches on partial words. If I missed the fine manual mentioning that

Re: [PATCH 4/4] emacs: Use the new JSON reply format.

2012-01-15 Thread Adam Wolfe Gordon
Hi Dmitry, On Thu, Jan 12, 2012 at 01:33, Dmitry Kurochkin wrote: > I did not follow the rest of the discussion, so sorry if I missed > something obvious.  But why can't we render HTML parts in replies the > same way we do in notmuch-show (using `mm-display-part')?  That should > not introduce a

[PATCH] Output unmodified Content-Type header value for JSON format.

2012-01-15 Thread Austin Clements
On Sun, 15 Jan 2012 11:52:40 +, David Edmondson wrote: > > Technically the IRC discussion was about not including *any* part > > content in the JSON output, and always using show --format=raw or > > similar to retrieve desired parts. Currently, notmuch includes part > > content in the JSON on

Re: [PATCH v3 04/10] notmuch-dump: add --format=(notmuch|sup)

2012-01-15 Thread Austin Clements
Quoth David Bremner on Jan 14 at 9:40 pm: > From: David Bremner > > sup is the old format, and remains the default. > > Each line of the notmuch format is "msg_id tag tag...tag" where each > space seperated token is 'hex-encoded' to remove troubling characters. > In particular this format won't

[PATCH] Fix dependency generation for compat, test, and util

2012-01-15 Thread Austin Clements
This adds source files in compat, test, and util to SRCS so that the top-level Makefile.local will generate dependency files for them. --- compat/Makefile.local |2 ++ test/Makefile.local |1 + util/Makefile.local |1 + 3 files changed, 4 insertions(+), 0 deletions(-) diff --git a

[PATCH] Fix dependency generation for CLI sources

2012-01-15 Thread Austin Clements
Previously, the dependency file list was generated before the CLI sources were added to SRCS, so dependency files weren't generated for CLI sources. This moves that code to after the CLI sources are added. --- Makefile.local |8 1 files changed, 4 insertions(+), 4 deletions(-) diff

[PATCH 1/1] Make buttons for attachments allow viewing as well as saving

2012-01-15 Thread Mark Walters
Define a keymap for attachment buttons to allow multiple actions. Define 3 possible actions: save attachment: exactly as currently, view attachment: uses mailcap entry, view attachment with user chosen program Keymap on a button is: s for save, v for view and o for view with other prog

[PATCH 0/1] emacs: allow viewing as well saving individual attachments

2012-01-15 Thread Mark Walters
Hello This is a resubmission of (a tidied version) of id:"87mxehqhbl.fsf at r102.config". I have modified the emacs interface for handling attachments by adding a keymap to the attachment button. For example pressing v when on an attachment button views the attachment (using the mailcap method) a

[PATCH v2] emacs: Cycle through notmuch buffers rather than jumping to the last.

2012-01-15 Thread David Edmondson
n a couple of major modes, but not enough to bother me so far. -- 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/20120115/2fe429df/attachment.pgp>

Re: [PATCH 2/2] cli: pick the user's address in a group list as from address

2012-01-15 Thread Tomi Ollila
On Sat, 14 Jan 2012 23:49:50 +0200, Jani Nikula wrote: > Messages received to a group list were not replied to using the from > address in the list. Fix it. > > Signed-off-by: Jani Nikula > --- Both LGTM. Tomi ___ notmuch mailing list notmuch@notmuch

[PATCH] emacs: globally replace non-branching "(if (not ..." with "(unless ..."

2012-01-15 Thread David Edmondson
-commit hooks to detect it. Footnotes: [1] Though I've done it myself. -- 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/notm

[PATCH] Output unmodified Content-Type header value for JSON format.

2012-01-15 Thread David Edmondson
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/20120115/c37bd4f5/attachment-0001.pgp>

[PATCH v3 09/10] random-dump.c: new test-binary to generate dump files

2012-01-15 Thread Jani Nikula
; 0) > + exit (1); > + > +pad_mid = ((int) log10 (num_lines) + 1); > +pad_tag = ((int) log10 (max_tags)) + 1; > + > +srandom (seed); > + > +int line; > +for (line = 0; line < num_lines; line++) { > + > + printf ("%0*d-", pad_mid, line); > + > + random_message_id (ctx, message_id_len); > + > + int num_tags = random () % (max_tags + 1); > + > + int j; > + for (j = 0; j < num_tags; j++) { > + printf (" %0*d-", pad_tag, j); > + random_tag (ctx, tag_len); > + } > + putchar ('\n'); > +} > + > +talloc_free (ctx); > + > +return 0; > +} > -- > 1.7.7.3 > > ___ > notmuch mailing list > notmuch at notmuchmail.org > http://notmuchmail.org/mailman/listinfo/notmuch -- next part -- An HTML attachment was scrubbed... URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120115/48d4809f/attachment-0001.html>

Re: [PATCH] Output unmodified Content-Type header value for JSON format.

2012-01-15 Thread Austin Clements
On Sun, 15 Jan 2012 11:52:40 +, David Edmondson wrote: > > Technically the IRC discussion was about not including *any* part > > content in the JSON output, and always using show --format=raw or > > similar to retrieve desired parts. Currently, notmuch includes part > > content in the JSON on

[PATCH v3 09/10] random-dump.c: new test-binary to generate dump files

2012-01-15 Thread David Bremner
On Sun, 15 Jan 2012 10:05:15 +0200, Jani Nikula wrote: > On Jan 15, 2012 3:41 AM, "David Bremner" wrote: > > > > From: David Bremner > > > > +notmuch_opt_desc_t options[] = { > > + { NOTMUCH_OPT_INT, &num_lines, "num-lines", 'n', 0 }, > > + { NOTMUCH_OPT_INT, &max_tags, "max-tags

[PATCH 0/4] Quoting HTML-only emails in replies redux

2012-01-15 Thread David Edmondson
e CLI, but it seems that we're already heading in that direction. -- 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/attachm

Re: [RFC] vim plugin rewrite II

2012-01-15 Thread Anton Khirnov
On Sat, 14 Jan 2012 22:15:07 -0400, David Bremner wrote: > On Sat, 14 Jan 2012 08:54:43 +0100, an...@khirnov.net wrote: > > > > The advantages over current vim client are still the following: > > * sending and displaying/saving attachments > > * much better unicode support > > * tag name and sea

Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs

2012-01-15 Thread Dmitry Kurochkin
On Sat, 14 Jan 2012 10:07:41 +0100, Pieter Praet wrote: > On Thu, 12 Jan 2012 21:34:29 +0400, Dmitry Kurochkin > wrote: > > On Thu, 12 Jan 2012 18:16:59 +0100, Pieter Praet wrote: > > > When running the Emacs tests in verbose mode, only the first missing > > > prereq is reported because the `ru

Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs

2012-01-15 Thread Dmitry Kurochkin
On Sat, 14 Jan 2012 10:09:37 +0100, Pieter Praet wrote: > When running the Emacs tests in verbose mode, only the first missing > prereq is reported because the `run_emacs' function is short-circuited > early: > > #+begin_example > emacs: Testing emacs interface > missing prerequisites:

Re: [PATCH v3 09/10] random-dump.c: new test-binary to generate dump files

2012-01-15 Thread David Bremner
On Sun, 15 Jan 2012 10:05:15 +0200, Jani Nikula wrote: > On Jan 15, 2012 3:41 AM, "David Bremner" wrote: > > > > From: David Bremner > > > > +notmuch_opt_desc_t options[] = { > > + { NOTMUCH_OPT_INT, &num_lines, "num-lines", 'n', 0 }, > > + { NOTMUCH_OPT_INT, &max_tags, "max-tags

[PATCH 1/1] Make buttons for attachments allow viewing as well as saving

2012-01-15 Thread Mark Walters
Define a keymap for attachment buttons to allow multiple actions. Define 3 possible actions: save attachment: exactly as currently, view attachment: uses mailcap entry, view attachment with user chosen program Keymap on a button is: s for save, v for view and o for view with other prog

[PATCH 0/1] emacs: allow viewing as well saving individual attachments

2012-01-15 Thread Mark Walters
Hello This is a resubmission of (a tidied version) of id:"87mxehqhbl.fsf@r102.config". I have modified the emacs interface for handling attachments by adding a keymap to the attachment button. For example pressing v when on an attachment button views the attachment (using the mailcap method) and

Re: [PATCH v2] emacs: Cycle through notmuch buffers rather than jumping to the last.

2012-01-15 Thread David Edmondson
On Sat, 14 Jan 2012 10:29:31 +0100, Pieter Praet wrote: > Might I ask, to what key(chord) have you bound this ? Due to its > usefulness, I'm inclined to bind it to [SPC], but on second though, > that might be a bit on the intense side... C-c= globally. That's clobbered in a couple of major modes

Re: [PATCH] emacs: globally replace non-branching "(if (not ..." with "(unless ..."

2012-01-15 Thread David Edmondson
On Sat, 14 Jan 2012 10:18:46 +0100, Pieter Praet wrote: > Does this really warrant a v2, or might we simply leave it as yet > another victim for Tomi's uncrustify-for-elisp [1] ? Pushing incorrectly indented code should be banned, in my opinion[1]. I'd be tempted to have some pre-commit hooks to

Re: [PATCH] Output unmodified Content-Type header value for JSON format.

2012-01-15 Thread David Edmondson
> Technically the IRC discussion was about not including *any* part > content in the JSON output, and always using show --format=raw or > similar to retrieve desired parts. Currently, notmuch includes part > content in the JSON only for text/*, *except* when it's text/html. I > assume non-text pa

Re: [PATCH 0/4] Quoting HTML-only emails in replies redux

2012-01-15 Thread David Edmondson
Given that we're now doing a bunch of work in emacs as part of the reply setup, why not just grab the content of the original message from the show buffer and quote that? The last time that approach was discussed Carl was against it because it moved the emacs UI away from the behaviour of the CLI,

Re: [PATCH v3 09/10] random-dump.c: new test-binary to generate dump files

2012-01-15 Thread Jani Nikula
On Jan 15, 2012 3:41 AM, "David Bremner" wrote: > > From: David Bremner > > This binary creates a "torture test" dump file for the new dump > format. > --- > test/Makefile.local |4 ++ > test/basic |2 +- > test/random-dump.c | 144 +