nomuch_addresses.py

2012-02-17 Thread Tomi Ollila
On Fri, 17 Feb 2012 21:33:41 +0100, Sebastian Spaeth wrote: > On Fri, 17 Feb 2012 20:58:50 +0200, Tomi Ollila wrote: > > Thank you for promoting my stuff. > > > > Latest version is 2.0 and name is now nottoomuch-addresses.sh > > It is available at: > > > > http://www.iki.fi/too/nottoomuch/ >

[PATCH v5.2 7/7] emacs: Use the new JSON reply format and message-cite-original

2012-02-17 Thread Austin Clements
Quoth Adam Wolfe Gordon on Feb 17 at 7:22 pm: > On Fri, Feb 17, 2012 at 13:00, Austin Clements wrote: > > One general comment that affects a lot of things in this patch: I > > think you should use the same JSON parsing settings that > > notmuch-query-get-threads uses. ?Besides consistency and mor

[PATCH v5.2 3/7] reply: Add a JSON reply format.

2012-02-17 Thread Austin Clements
Quoth Adam Wolfe Gordon on Feb 17 at 7:06 pm: > On Fri, Feb 17, 2012 at 10:04, Austin Clements wrote: > > The first two patches LGTM. ?A few nits in this one. > > Thanks for the review. A couple of points to discuss below; everything > else I'll change for the next version. > > >> +void > >> +f

nomuch_addresses.py

2012-02-17 Thread Philippe LeCavalier
On Fri, 17 Feb 2012 22:46:49 +0200, Tomi Ollila wrote: > On Fri, 17 Feb 2012 21:33:41 +0100, Sebastian Spaeth SSpaeth.de> wrote: > > On Fri, 17 Feb 2012 20:58:50 +0200, Tomi Ollila > > wrote: > > > Thank you for promoting my stuff. > > > > > > Latest version is 2.0 and name is now nottoomuch-a

nomuch_addresses.py

2012-02-17 Thread Sebastian Spaeth
so that people can actually find it without searching the email lists. Sebastian -- 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

Re: nomuch_addresses.py

2012-02-17 Thread Tomi Ollila
On Fri, 17 Feb 2012 22:04:12 -0500, Philippe LeCavalier wrote: > On Fri, 17 Feb 2012 22:46:49 +0200, Tomi Ollila wrote: > > On Fri, 17 Feb 2012 21:33:41 +0100, Sebastian Spaeth > > wrote: > > > On Fri, 17 Feb 2012 20:58:50 +0200, Tomi Ollila > > > wrote: > > > > Thank you for promoting my st

nomuch_addresses.py

2012-02-17 Thread Tomi Ollila
On Fri, 17 Feb 2012 02:28:52 +0100, Daniel Schoepe wrote: > On Thu, 16 Feb 2012 14:12:36 -0500, Philippe LeCavalier plecavalier.com> wrote: > > Hi again. > > > > I'm trying to setup tab completion for addresses. I appeared to me that > > the simplest solution was notmuch_addresses so I grabbed

[PATCH] emacs: Add configurable wrapping width for notmuch-wash-wrap-long-lines

2012-02-17 Thread Daniel Schoepe
t was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120217/9b572b5d/attachment.pgp>

[PATCH] emacs: Add configurable wrapping width for notmuch-wash-wrap-long-lines

2012-02-17 Thread Daniel Schoepe
This introduces a variable to control after how many characters a line is wrapped by notmuch-wash-wrap-long-lines (still wrapping at the window width if it is lower). --- emacs/notmuch-wash.el | 36 ++-- 1 files changed, 26 insertions(+), 10 deletions(-) diff --g

Re: [PATCH v5.2 7/7] emacs: Use the new JSON reply format and message-cite-original

2012-02-17 Thread Austin Clements
Quoth Adam Wolfe Gordon on Feb 17 at 7:22 pm: > On Fri, Feb 17, 2012 at 13:00, Austin Clements wrote: > > One general comment that affects a lot of things in this patch: I > > think you should use the same JSON parsing settings that > > notmuch-query-get-threads uses.  Besides consistency and mor

Re: [PATCH v5.2 3/7] reply: Add a JSON reply format.

2012-02-17 Thread Austin Clements
Quoth Adam Wolfe Gordon on Feb 17 at 7:06 pm: > On Fri, Feb 17, 2012 at 10:04, Austin Clements wrote: > > The first two patches LGTM.  A few nits in this one. > > Thanks for the review. A couple of points to discuss below; everything > else I'll change for the next version. > > >> +void > >> +f

[PATCH v5.2 7/7] emacs: Use the new JSON reply format and message-cite-original

2012-02-17 Thread Adam Wolfe Gordon
On Fri, Feb 17, 2012 at 13:00, Austin Clements wrote: > One general comment that affects a lot of things in this patch: I > think you should use the same JSON parsing settings that > notmuch-query-get-threads uses. ?Besides consistency and more > opportunities for code reuse, using lists instead o

[PATCH v5.2 3/7] reply: Add a JSON reply format.

2012-02-17 Thread Adam Wolfe Gordon
On Fri, Feb 17, 2012 at 10:04, Austin Clements wrote: > The first two patches LGTM. ?A few nits in this one. Thanks for the review. A couple of points to discuss below; everything else I'll change for the next version. >> +void >> +format_part_json (const void *ctx, mime_node_t *node, notmuch_bo

Re: nomuch_addresses.py

2012-02-17 Thread Philippe LeCavalier
On Fri, 17 Feb 2012 22:46:49 +0200, Tomi Ollila wrote: > On Fri, 17 Feb 2012 21:33:41 +0100, Sebastian Spaeth > wrote: > > On Fri, 17 Feb 2012 20:58:50 +0200, Tomi Ollila wrote: > > > Thank you for promoting my stuff. > > > > > > Latest version is 2.0 and name is now nottoomuch-addresses.sh >

[PATCH v10 2/2] emacs: Tests for user-defined sections

2012-02-17 Thread Dmitry Kurochkin
From: Daniel Schoepe A new file was added for notmuch-hello tests. --- test/emacs-hello | 47 test/emacs.expected-output/notmuch-hello |3 +- .../notmuch-hello-new-section |4 ++ .../notmuch-hello-no-

[PATCH v10 1/2] emacs: User-defined sections in notmuch-hello

2012-02-17 Thread Dmitry Kurochkin
From: Daniel Schoepe This patch makes the notmuch-hello screen fully customizable by allowing the user to add and remove arbitrary sections. It also provides some convenience functions for constructing sections, e.g. showing the unread message count for each tag. This is done by specifying a lis

[PATCH v10 0/2] emacs: User-defined sections in notmuch-hello

2012-02-17 Thread Dmitry Kurochkin
Found a small bug... Changes: v10: * explicitly return nil from `notmuch-hello-insert-recent-searches', otherwise `indent-rigidly' returns a marker pointing to non-existing buffer and breaks final-target-pos handling in `notmuch-hello' v9: * correctly merge bc267b70b01c79f6bdda52641e9cd757

[PATCH] emacs: Add configurable wrapping width for notmuch-wash-wrap-long-lines

2012-02-17 Thread David Edmondson
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/20120217/ec3b7047/attachment.pgp>

Re: [PATCH v5.2 7/7] emacs: Use the new JSON reply format and message-cite-original

2012-02-17 Thread Adam Wolfe Gordon
On Fri, Feb 17, 2012 at 13:00, Austin Clements wrote: > One general comment that affects a lot of things in this patch: I > think you should use the same JSON parsing settings that > notmuch-query-get-threads uses.  Besides consistency and more > opportunities for code reuse, using lists instead o

Re: [PATCH v5.2 3/7] reply: Add a JSON reply format.

2012-02-17 Thread Adam Wolfe Gordon
On Fri, Feb 17, 2012 at 10:04, Austin Clements wrote: > The first two patches LGTM.  A few nits in this one. Thanks for the review. A couple of points to discuss below; everything else I'll change for the next version. >> +void >> +format_part_json (const void *ctx, mime_node_t *node, notmuch_bo

[PATCH v5.2 4/7] man: Update notmuch-reply man page for JSON format.

2012-02-17 Thread Austin Clements
The JSON reply format should also get added to devel/schemata (since you're building on my JSON show rewrite patches anyway). Quoth Adam Wolfe Gordon on Feb 15 at 8:12 pm: > --- > man/man1/notmuch-reply.1 |5 + > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/man/man1

[PATCH v5.2 7/7] emacs: Use the new JSON reply format and message-cite-original

2012-02-17 Thread Austin Clements
Quoth Adam Wolfe Gordon on Feb 15 at 8:12 pm: > Use the new JSON reply format to create replies in emacs. Quote HTML > parts nicely by using mm-display-part to turn them into displayable > text, then quoting them with message-cite-original. This is very > useful for users who regularly receive HTM

[PATCH v5.2 1/7] test: Add broken test for the new JSON reply format.

2012-02-17 Thread Austin Clements
One nit, actually. It would good if this test were marked test_subtest_known_broken by this patch and then unmarked broken by the patch that adds the JSON reply format so that the test doesn't outright fail at any point in the history. Quoth Adam Wolfe Gordon on Feb 15 at 8:12 pm: > --- > test/

Re: nomuch_addresses.py

2012-02-17 Thread Tomi Ollila
On Fri, 17 Feb 2012 21:33:41 +0100, Sebastian Spaeth wrote: > On Fri, 17 Feb 2012 20:58:50 +0200, Tomi Ollila wrote: > > Thank you for promoting my stuff. > > > > Latest version is 2.0 and name is now nottoomuch-addresses.sh > > It is available at: > > > > http://www.iki.fi/too/nottoomuch/ >

Re: nomuch_addresses.py

2012-02-17 Thread Sebastian Spaeth
On Fri, 17 Feb 2012 20:58:50 +0200, Tomi Ollila wrote: > Thank you for promoting my stuff. > > Latest version is 2.0 and name is now nottoomuch-addresses.sh > It is available at: > > http://www.iki.fi/too/nottoomuch/ It would be great to add these things to the notmuch wiki so that people can a

[PATCH v5.2 3/7] reply: Add a JSON reply format.

2012-02-17 Thread Austin Clements
The first two patches LGTM. A few nits in this one. Quoth Adam Wolfe Gordon on Feb 15 at 8:12 pm: > This new JSON format for replies includes headers generated for a > reply message as well as the headers of the original message. Using > this data, a client can intelligently create a reply. For

Re: [PATCH v5.2 4/7] man: Update notmuch-reply man page for JSON format.

2012-02-17 Thread Austin Clements
The JSON reply format should also get added to devel/schemata (since you're building on my JSON show rewrite patches anyway). Quoth Adam Wolfe Gordon on Feb 15 at 8:12 pm: > --- > man/man1/notmuch-reply.1 |5 + > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/man/man1

Re: [PATCH v5.2 7/7] emacs: Use the new JSON reply format and message-cite-original

2012-02-17 Thread Austin Clements
Quoth Adam Wolfe Gordon on Feb 15 at 8:12 pm: > Use the new JSON reply format to create replies in emacs. Quote HTML > parts nicely by using mm-display-part to turn them into displayable > text, then quoting them with message-cite-original. This is very > useful for users who regularly receive HTM

[PATCH v9 2/2] emacs: Tests for user-defined sections

2012-02-17 Thread Dmitry Kurochkin
From: Daniel Schoepe A new file was added for notmuch-hello tests. --- test/emacs-hello | 47 test/emacs.expected-output/notmuch-hello |3 +- .../notmuch-hello-new-section |4 ++ .../notmuch-hello-no-

[PATCH v9 1/2] emacs: User-defined sections in notmuch-hello

2012-02-17 Thread Dmitry Kurochkin
From: Daniel Schoepe This patch makes the notmuch-hello screen fully customizable by allowing the user to add and remove arbitrary sections. It also provides some convenience functions for constructing sections, e.g. showing the unread message count for each tag. This is done by specifying a lis

[PATCH v9 0/2] emacs: User-defined sections in notmuch-hello

2012-02-17 Thread Dmitry Kurochkin
Hi all. I have finished reviewing the user-defined sections v8 series at last. I found only few minor issues, some of which are fixed by this v9 series and others can be fixed later or ignored (e.g. renaming notmuch-hello-insert-* functions to notmuch-hello-section-* or something more appropriate)

Re: nomuch_addresses.py

2012-02-17 Thread Tomi Ollila
On Fri, 17 Feb 2012 02:28:52 +0100, Daniel Schoepe wrote: > On Thu, 16 Feb 2012 14:12:36 -0500, Philippe LeCavalier > wrote: > > Hi again. > > > > I'm trying to setup tab completion for addresses. I appeared to me that > > the simplest solution was notmuch_addresses so I grabbed a copy from > >

Re: [PATCH] emacs: Add configurable wrapping width for notmuch-wash-wrap-long-lines

2012-02-17 Thread Daniel Schoepe
On Fri, 17 Feb 2012 18:46:21 +, David Edmondson wrote: > * dan...@schoepe.org [2012-02-17 Fri 18:34] > > This introduces a variable to control after how many characters a line > > is wrapped by notmuch-wash-wrap-long-lines (still wrapping at the > > window width if it is lower). > > What do y

Re: [PATCH] emacs: Add configurable wrapping width for notmuch-wash-wrap-long-lines

2012-02-17 Thread David Edmondson
* dan...@schoepe.org [2012-02-17 Fri 18:34] > This introduces a variable to control after how many characters a line > is wrapped by notmuch-wash-wrap-long-lines (still wrapping at the > window width if it is lower). What do you use this for? pgps2H1ob2IRX.pgp Description: PGP signature

[PATCH] emacs: Add configurable wrapping width for notmuch-wash-wrap-long-lines

2012-02-17 Thread Daniel Schoepe
This introduces a variable to control after how many characters a line is wrapped by notmuch-wash-wrap-long-lines (still wrapping at the window width if it is lower). --- emacs/notmuch-wash.el | 36 ++-- 1 files changed, 26 insertions(+), 10 deletions(-) diff --g

Re: [PATCH v5.2 1/7] test: Add broken test for the new JSON reply format.

2012-02-17 Thread Austin Clements
One nit, actually. It would good if this test were marked test_subtest_known_broken by this patch and then unmarked broken by the patch that adds the JSON reply format so that the test doesn't outright fail at any point in the history. Quoth Adam Wolfe Gordon on Feb 15 at 8:12 pm: > --- > test/

Re: [PATCH v5.2 3/7] reply: Add a JSON reply format.

2012-02-17 Thread Austin Clements
The first two patches LGTM. A few nits in this one. Quoth Adam Wolfe Gordon on Feb 15 at 8:12 pm: > This new JSON format for replies includes headers generated for a > reply message as well as the headers of the original message. Using > this data, a client can intelligently create a reply. For

[PATCH v7 0/3] NEWS and test comment adjustments

2012-02-17 Thread David Bremner
On Wed, 15 Feb 2012 11:17:29 +0200, Tomi Ollila wrote: > This version of 'new.ignore' patch series addresses Austin's last > comments in NEWS and test/new documentation parts. > Pushed, d

[PATCH v2] emacs: Fix display of highlighted line in notmuch-search

2012-02-17 Thread David Bremner
On Tue, 14 Feb 2012 18:09:47 +0100, Michal Sojka wrote: > When notmuch-search-line-faces is used to set background color in search > results, the highlight of the current line is not always displayed > correctly. This patch fixes that by increasing the priority property of > the highlight overlay.

[PATCH v10 2/2] emacs: Tests for user-defined sections

2012-02-17 Thread Dmitry Kurochkin
From: Daniel Schoepe A new file was added for notmuch-hello tests. --- test/emacs-hello | 47 test/emacs.expected-output/notmuch-hello |3 +- .../notmuch-hello-new-section |4 ++ .../notmuch-hello-no-

[PATCH v10 1/2] emacs: User-defined sections in notmuch-hello

2012-02-17 Thread Dmitry Kurochkin
From: Daniel Schoepe This patch makes the notmuch-hello screen fully customizable by allowing the user to add and remove arbitrary sections. It also provides some convenience functions for constructing sections, e.g. showing the unread message count for each tag. This is done by specifying a lis

[PATCH v10 0/2] emacs: User-defined sections in notmuch-hello

2012-02-17 Thread Dmitry Kurochkin
Found a small bug... Changes: v10: * explicitly return nil from `notmuch-hello-insert-recent-searches', otherwise `indent-rigidly' returns a marker pointing to non-existing buffer and breaks final-target-pos handling in `notmuch-hello' v9: * correctly merge bc267b70b01c79f6bdda52641e9cd757

Re: [PATCH v7 0/3] NEWS and test comment adjustments

2012-02-17 Thread David Bremner
On Wed, 15 Feb 2012 11:17:29 +0200, Tomi Ollila wrote: > This version of 'new.ignore' patch series addresses Austin's last > comments in NEWS and test/new documentation parts. > Pushed, d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuc

Re: [PATCH v2] emacs: Fix display of highlighted line in notmuch-search

2012-02-17 Thread David Bremner
On Tue, 14 Feb 2012 18:09:47 +0100, Michal Sojka wrote: > When notmuch-search-line-faces is used to set background color in search > results, the highlight of the current line is not always displayed > correctly. This patch fixes that by increasing the priority property of > the highlight overlay

nomuch_addresses.py

2012-02-17 Thread Daniel Schoepe
y give a useful comparison. [1] id:"m2d3c8e22v.fsf at guru.guru-group.fi" Cheers, Daniel -- 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