Re: [PATCH] cli: factor out config handling code to get/set lists.

2012-01-12 Thread Pieter Praet
On Mon, 12 Dec 2011 14:26:21 -0400, David Bremner wrote: > On Sun, 11 Dec 2011 20:41:53 +0400, Dmitry Kurochkin > wrote: > > Hi David. > > > > On Sun, 11 Dec 2011 12:07:51 -0400, David Bremner wrote: > > > From: David Bremner > > > > > > Two new internal routines are created _config_get_list

Re: revised patch for gmime init, with test.

2012-01-12 Thread Pieter Praet
On Sat, 31 Dec 2011 23:22:46 -0400, David Bremner wrote: > It turns out that our existing (trivial) python test is enough to > catch this bug, but the corpus needs to be augmented. This > augmentation is a bit intrusive so I'm thinking of cherry-picking only > the actual fix to the release branch

[PATCH 3/3] emacs: colorize buttonized 'id:' links depending on the target message's state

2012-01-12 Thread Pieter Praet
* emacs/notmuch-show.el (notmuch-show-buttonized-link-present), * emacs/notmuch-show.el (notmuch-show-buttonized-link-present-and-unread), * emacs/notmuch-show.el (notmuch-show-buttonized-link-missing): new faces for buttonized id: links. * emacs/notmuch-show.el (notmuch-show-found-target-p): ad

[PATCH 2/3] emacs: `notmuch-show-buttonize-links' only `notmuch-show's a message if it exists

2012-01-12 Thread Pieter Praet
* emacs/notmuch-show.el (notmuch-show-found-target-p): new predicate function that uses notmuch(1) 'count' to see if a query turns up any results. * emacs/notmuch-show.el (notmuch-show-if-found): new function that only shows a message/thread if present in the database and otherwise returns an

[PATCH 1/3] emacs: s/buttonise/buttonize/g

2012-01-12 Thread Pieter Praet
"Worldwide, -ize endings prevail in scientific writing and are commonly used by many international organizations, such as the ISO and the WHO. The European Union switched from -ize to -ise some years ago in its English language publications, and this resulted in the coexistence of the -ize spelling

Re: [PATCH v2] emacs: call notmuch-show instead of notmuch-search in buttonised id: links

2012-01-12 Thread Pieter Praet
On Thu, 22 Dec 2011 20:59:12 -0800, Jameson Graef Rollins wrote: > Since message-ids necessarily match just a single message, there's no > reason to do a search for the id before viewing the actual message; > the search just becomes an extra screen to click through. Clicking on > an id: links no

Re: RFC: use the XDG Base Directory Specification for configuration data

2012-01-12 Thread Pieter Praet
On Wed, 21 Dec 2011 11:35:37 -0800, Jameson Graef Rollins wrote: > On Wed, 21 Dec 2011 07:44:16 -0400, David Bremner wrote: > [SNIP] > > > On my machine all of the programs (except nottoomuch ;) ) that use > > ~/.config are gui based, and a bunch of them store binary blobs (or not > > plain tex

Re: Code Style Bikeshed 1/n: Break before Brace or Brace before Break?

2012-01-12 Thread Pieter Praet
On Sun, 18 Dec 2011 20:18:03 +0400, Dmitry Kurochkin wrote: > On Sun, 18 Dec 2011 18:12:51 +0200, Tomi Ollila wrote: > > On Sun, 18 Dec 2011 19:50:52 +0400, Dmitry Kurochkin > > wrote: > > > +1 for the former. > > > > +1 > > > > > How about indenting with spaces+tabs vs indenting with spaces

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

2012-01-12 Thread Pieter Praet
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: [0] emacs(1) skipping test: [0] Basic notmuch-hello view in ema

Re: [PATCH v3 00/7] test: (hopefully) better test prerequisites

2012-01-12 Thread Pieter Praet
On Sun, 27 Nov 2011 22:36:12 +0400, Dmitry Kurochkin wrote: > Changes: > > v3 since v2: > > * rebased the remaining patches on current master > > v2 since v1: > > * Add test_require_external_prereq function to explicitly check for > external dependencies, use it in test_emacs. > > * Indent

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

2012-01-12 Thread Pieter Praet
On Tue, 22 Nov 2011 22:40:21 -0500, Austin Clements wrote: > Quoth Jameson Graef Rollins on Nov 20 at 12:10 pm: > > The open question seems to be how we handle the content encoding > > parameters. My argument is that those should either be used by notmuch > > to properly encode the content for th

[PATCH v2] emacs: logically group def{custom,face}s

2012-01-12 Thread Pieter Praet
To allow for expansion whilst keeping everything tidy and organized, move all defcustom/defface variables to the following subgroups, defined in notmuch-lib.el: - Hello - Search - Show - Send - Crypto - Hooks - Appearance - External Commands As an added benefit, defcustom keyword args are now con

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

2012-01-12 Thread Pieter Praet
On Sun, 20 Nov 2011 22:35:42 +0400, Dmitry Kurochkin wrote: > On Sat, 19 Nov 2011 08:59:29 -0400, David Bremner wrote: > > On Sat, 19 Nov 2011 08:18:41 +0400, Dmitry Kurochkin > > wrote: > > > Before the change, notmuch used g_mime_content_type_to_string(3) > > > function to output Content-Typ

Re: [PATCH 0/6] Rebase of Pieter's "set test prereqs"

2012-01-12 Thread Pieter Praet
On Thu, 17 Nov 2011 11:14:11 +0100, Thomas Jost wrote: > On Wed, 16 Nov 2011 21:50:17 +0100, Pieter Praet wrote: > > On Wed, 16 Nov 2011 15:33:49 +0100, Thomas Jost > > wrote: > > > Hello list, > > > > > > This is another rebased version of Pieter's series to add GPG and Emacs > > > as test >

[PATCH 2/2] notmuch-reply.c: uncrustify

2012-01-12 Thread David Bremner
On Wed, 11 Jan 2012 22:57:39 -0500, Austin Clements wrote: > > From: David Bremner > > > > This patch shows the raw result of running uncrustify on notmuch-reply.c. > > The re-indenting of "format_reply" would probably not be desirable. > > The good news is that that structure is on its way out

Re: [PATCH 2/2] NEWS: fix some old typos and trailing whitespace

2012-01-12 Thread Pieter Praet
On Thu, 17 Nov 2011 08:21:59 +0200, Jani Nikula wrote: > On Nov 16, 2011 11:39 PM, "Pieter Praet" wrote: > > > > On Wed, 16 Nov 2011 22:54:35 +0200, Tomi Ollila > wrote: > > > On Wed, 16 Nov 2011 21:37:50 +0100, Pieter Praet > wrote: > > > > No further explanation needed. > > > > --- > > > > N

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

2012-01-12 Thread Pieter Praet
On Wed, 16 Nov 2011 23:47:53 +0100, Pieter Praet wrote: > On Sun, 13 Nov 2011 13:16:48 -0800, Jameson Graef Rollins > wrote: > > 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

Re: 0.11 is frozen, please update NEWS

2012-01-12 Thread Pieter Praet
On Sun, 25 Dec 2011 15:55:43 -0400, David Bremner wrote: > > Well, it's -19C outside, so guess that is appropriate. > > I have merged master to release in preparation for release. There should > only be additions to NEWS or fixes for serious bugs for branch release > until 0.11 is out the door.

Re: [PATCH] make (kill-emacs) from emacsclient work with emacs 23.(1|2)

2012-01-12 Thread Austin Clements
Quoth Tomi Ollila on Jan 12 at 10:51 am: > On Wed, 11 Jan 2012 23:13:44 -0500, Austin Clements wrote: > > This is important to fix, but this solution seems needlessly > > roundabout. What about using an after-advice and simply delq'ing > > whatever the offending hook is? That wouldn't even need

Re: [PATCH 2/2] notmuch-reply.c: uncrustify

2012-01-12 Thread Austin Clements
Quoth David Bremner on Jan 12 at 9:08 am: > @@ -425,7 +420,7 @@ notmuch_search_command (void *ctx, int argc, char *argv[]) > int limit = -1; /* unlimited */ > > enum { NOTMUCH_FORMAT_JSON, NOTMUCH_FORMAT_TEXT } > - format_sel = NOTMUCH_FORMAT_TEXT; > +format_sel = NOTMUCH_FORMA

Re: [PATCH] notmuch/emacs: Observe the charset of encoded parts, where known.

2012-01-12 Thread Dmitry Kurochkin
On Thu, 12 Jan 2012 14:42:49 +, David Edmondson wrote: > On Thu, 12 Jan 2012 18:17:44 +0400, Dmitry Kurochkin > wrote: > > I think there is a record of useful features and fixes that were not > > accepted to notmuch because of some implementation issues. And > > interested people were using

Re: [PATCH] notmuch/emacs: Observe the charset of encoded parts, where known.

2012-01-12 Thread David Edmondson
On Thu, 12 Jan 2012 18:17:44 +0400, Dmitry Kurochkin wrote: > I think there is a record of useful features and fixes that were not > accepted to notmuch because of some implementation issues. And > interested people were using them in private repos for years. (I do not > say that it is always t

Re: [PATCH] notmuch/emacs: Observe the charset of encoded parts, where known.

2012-01-12 Thread Dmitry Kurochkin
Hi David. On Thu, 12 Jan 2012 12:00:14 +, David Edmondson wrote: > On Wed, 11 Jan 2012 22:34:45 +0400, Dmitry Kurochkin > wrote: > > -1 > > Which puts me back to 0 :-( > > > One particular issue with your patch is that it adds (incorrect) charset > > parameter to plain/text parts which ar

Re: [PATCH 2/2] notmuch-reply.c: uncrustify

2012-01-12 Thread Tomi Ollila
On Thu, 12 Jan 2012 09:08:18 -0400, David Bremner wrote: > > > All of the other format fixes look very reasonable to me. I think you > > managed to find one of the least canonical source files in the tree. > > Was that intentional? > > Yes, I expect there would be less changes elsewhere. Althou

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

2012-01-12 Thread David Edmondson
On Thu, 12 Jan 2012 15:36:07 +0200, Tomi Ollila wrote: > On Thu, 12 Jan 2012 12:07:40 +, David Edmondson wrote: > > On Thu, 12 Jan 2012 12:33:58 +0400, Dmitry Kurochkin > > wrote: > > > I did not follow the rest of the discussion, so sorry if I missed > > > something obvious. But why can't

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

2012-01-12 Thread Tomi Ollila
On Thu, 12 Jan 2012 12:07:40 +, David Edmondson wrote: > On Thu, 12 Jan 2012 12:33:58 +0400, 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

[PATCH v2] notmuch/emacs: Observe the charset of text/html parts, where known.

2012-01-12 Thread David Edmondson
Add the charset of text/html parts to the JSON output of 'notmuch -show' when it is known. Observe the encoding when rendering such parts in emacs. --- Add the charset only for text/html parts to avoid incorrectly declaring the charset for JSON inline text/plain parts (as they have already been co

Re: [PATCH 2/2] notmuch-reply.c: uncrustify

2012-01-12 Thread Jani Nikula
On Thu, 12 Jan 2012 09:08:18 -0400, David Bremner wrote: > The corresponding diff for notmuch-search.c follows > - > - notmuch_filenames_destroy( filenames ); > + > + notmuch_filenames_destroy ( filenames ); Can you make it remove the extra spaces after ( and before ) to

Re: [PATCH 2/2] notmuch-reply.c: uncrustify

2012-01-12 Thread David Bremner
On Wed, 11 Jan 2012 22:57:39 -0500, Austin Clements wrote: > > From: David Bremner > > > > This patch shows the raw result of running uncrustify on notmuch-reply.c. > > The re-indenting of "format_reply" would probably not be desirable. > > The good news is that that structure is on its way out

Re: [PATCH v3 4/4] test: add tests for "notmuch reply" --reply-to=sender

2012-01-12 Thread Tomi Ollila
On Wed, 11 Jan 2012 22:50:32 -0500, Austin Clements wrote: > I'm not sure what the point of the last test is, but this patch > covered everything I could think of, so LGTM. Maybe checking that long subject line (possibly split/join sometimes in the process) keeps it's original form (?) LGTM (and

Re: Regarding notmuch and Fedora 16

2012-01-12 Thread Tomi Ollila
On Tue, 03 Jan 2012 22:36:59 +, Darren McGuicken wrote: > On Tue, 03 Jan 2012 17:09:39 -0500, Peter Portante > wrote: > > I am interested in using notmuch from within emacs, but have not been > > able to get the latest version of notmuch (0.10.2) to compile under > > Fedora 16. > > Looks

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

2012-01-12 Thread David Edmondson
On Thu, 12 Jan 2012 12:33:58 +0400, 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 w3m.el

Re: [PATCH] notmuch/emacs: Observe the charset of encoded parts, where known.

2012-01-12 Thread David Edmondson
On Wed, 11 Jan 2012 22:34:45 +0400, Dmitry Kurochkin wrote: > -1 Which puts me back to 0 :-( > One particular issue with your patch is that it adds (incorrect) charset > parameter to plain/text parts which are converted to UTF-8 in JSON > output. Patches that break things should obviously not

Re: [PATCH] make (kill-emacs) from emacsclient work with emacs 23.(1|2)

2012-01-12 Thread Tomi Ollila
On Wed, 11 Jan 2012 23:13:44 -0500, Austin Clements wrote: > This is important to fix, but this solution seems needlessly > roundabout. What about using an after-advice and simply delq'ing > whatever the offending hook is? That wouldn't even need a version > check. delq could work -- thanks for

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

2012-01-12 Thread Dmitry Kurochkin
Hi Adam. On Mon, 9 Jan 2012 19:10:48 -0700, Adam Wolfe Gordon wrote: > Hi David, > > Thanks for the review. Most of the things you've suggested are easy > changes, and I think obvious improvements, so I'll change them for the > next version. A bit of discussion on the more involved things belo

[PATCH v3 2/4] emacs: add support for replying just to the sender

2012-01-12 Thread Jani Nikula
On Wed, 11 Jan 2012 16:15:30 -0500, Aaron Ecay wrote: > On Wed, 11 Jan 2012 08:20:56 +, David Edmondson wrote: > > On Tue, 10 Jan 2012 21:54:02 +0200, Jani Nikula wrote: > > > -(defun notmuch-mua-reply (query-string &optional sender) > > > +(defun notmuch-mua-reply (query-string &optional se

<    1   2