make fails with gmime-2.6.4

2012-01-16 Thread Jameson Graef Rollins
nt was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/8fdf9924/attachment.pgp>

[PATCH 2/3] emacs: whitespace-cleanup and indent-region for emacs/*.el files

2012-01-16 Thread Austin Clements
Cleanup is the type of pain that should only be suffered once, so I'd be much happier with this if there was an accompanying git hook that prevented more mis-formatted code from slipping in. Quoth Tomi Ollila on Jan 16 at 11:04 am: > diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el >

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

2012-01-16 Thread Jani Nikula
On Mon, 16 Jan 2012 15:51:30 -0500, Aaron Ecay wrote: > On Mon, 16 Jan 2012 11:22:27 +, David Edmondson wrote: > > On Mon, 16 Jan 2012 12:17:03 +0100, Pieter Praet > > wrote: > > > Alas, those text properties actually *were* effective (and I liked them :) > > > > Well, not in emacs 24. My

[PATCH 1/3] emacs/*.el: changed one-char comment prefix ';' to two: ';;'

2012-01-16 Thread Austin Clements
Quoth Tomi Ollila on Jan 16 at 11:04 am: > In order for emacs (indent-region) to (re)indent emacs lisp > properly there needs to be at least 2 comment characters (;;). > --- > > $ perl -ne 's/;;.*//; print $_ if /(^|[^;]);([^;]|$)/' *.el | less > > shows that there is (only) need to change

[PATCH v2] Document external dependencies in the test suite

2012-01-16 Thread Jani Nikula
On Mon, 16 Jan 2012 22:09:55 +0100, Xavier Maillard wrote: > Hi, > > On Mon, 16 Jan 2012 13:52:47 -0500, Ethan Glasser-Camp > wrote: > > [...] > > > +Prerequisites > > +- > > +Some tests require external dependencies to run. Without them, they > > +will be skipped, or (rarely)

[PATCH] v2 [RFC] emacs: merge overhauled `notmuch-cycle-notmuch-buffers' into `notmuch'

2012-01-16 Thread Jani Nikula
On Mon, 16 Jan 2012 17:46:55 +0100, Pieter Praet wrote: > Make `notmuch-cycle-notmuch-buffers' more Lispy and merge into `notmuch', > eliminating the need to hog yet another keybind. What does "merge" mean here? Will it still be possible for me to hit one key to unconditionally get to

[PATCH] Add pseudo-compatibility with gmime 2.6

2012-01-16 Thread Kazuo Teramoto
mail/notmuch/attachments/20120116/45eec978/attachment-0001.pgp>

[PATCH] Add pseudo-compatibility with gmime 2.6

2012-01-16 Thread Austin Clements
Quoth Thomas Jost on Jan 17 at 12:56 am: > There are lots of API changes in gmime 2.6 crypto handling. By adding > preprocessor directives, it is however possible to add gmime 2.6 compatibility > while preserving compatibility with gmime 2.4 too. Awesome. Comments inline below. > This is mostly

Partial words on notmuch search?

2012-01-16 Thread Andrei Popescu
n it simply, you don't understand it well enough. (Albert Einstein) -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 490 bytes Desc: Digital signature URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/e56d0b28/attachment.pgp>

[PATCH 2/2] notmuch-dev: Update the master branch when requested.

2012-01-16 Thread Xavier Maillard
On Mon, 16 Jan 2012 09:07:52 +, David Edmondson wrote: > On Fri, 13 Jan 2012 22:03:17 +0100, Xavier Maillard > wrote: > > Have you planned to cook something about this in notmuch wiki ? > > That sounds like something I could use to test patches more easily than > > I do currently. > > I'll

[PATCH 2/3] emacs: whitespace-cleanup and indent-region for emacs/*.el files

2012-01-16 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/20120116/e249d5f0/attachment.pgp>

[PATCH v2] Document external dependencies in the test suite

2012-01-16 Thread Xavier Maillard
Hi, On Mon, 16 Jan 2012 13:52:47 -0500, Ethan Glasser-Camp wrote: [...] > +Prerequisites > +- > +Some tests require external dependencies to run. Without them, they > +will be skipped, or (rarely) marked failed. Please install these, so > +that you know if you break anything. > +

[PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions

2012-01-16 Thread Dmitry Kurochkin
Hello. Since this patches got in, I have yet to send a single email to the address(es) I intend to :( I am really used to the bindings and this change is a pain. From IRC discussion, it seems like I am not alone here. There was a proposal to change reply bindings to 'ra' and 'rs'. When 'r' is

[PATCH] Document the dependency on dtach in the test suite

2012-01-16 Thread Dmitry Kurochkin
On Mon, 16 Jan 2012 12:16:08 -0500, Ethan Glasser-Camp wrote: > From: Ethan Glasser-Camp > > dtach (and before that, screen) are used to run emacs with more > verisimilitude, but this is only mentioned in the NEWS. Mention this > in the test/README file, to make it easier to find out why a

[PATCH 1/2] test: Add "test_expect_emacs_t" to assist with emacs testing.

2012-01-16 Thread Dmitry Kurochkin
Hi David. On Mon, 16 Jan 2012 16:47:32 +, David Edmondson wrote: > "test_expect_emacs_t" expects a single argument. If this is `t' then > the test passes. Otherwise the argument provides the details of the > test failure to be reported. Isn't this function the same as something like

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

2012-01-16 Thread Mark Walters
(My apologies, owing to various errors on my part the patch has ended up in a separate thread: id:"1326749910-30437-1-git-send-email-markwalters1009 at gmail.com") On Mon, 16 Jan 2012 11:31:16 -0800, Jameson Graef Rollins wrote: > > Keymap on a button is: s for save, v for view and o for view

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

2012-01-16 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

Partial words on notmuch search?

2012-01-16 Thread Austin Clements
Quoth Andrei Popescu on Jan 16 at 10:21 pm: > This is also interesting: > $ notmuch count 'debian' > 65888 > $ notmuch count 'dEbian' > 65888 > $ notmuch count 'Debian' > 65887 The first two will match stemmed versions of "debian" such as "debian's" and "debianed". However, starting a term with

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

2012-01-16 Thread Austin Clements
Quoth Mark Walters on Jan 16 at 9:44 pm: > On Mon, 16 Jan 2012 11:31:16 -0800, Jameson Graef Rollins finestructure.net> wrote: > > > Keymap on a button is: s for save, v for view and o for view with > > > other program. Default (i.e. enter or mouse button) is save but is > > > easily

[PATCH 1/2] test: add known broken test for reply from address in named group list

2012-01-16 Thread David Bremner
On Sat, 14 Jan 2012 23:49:49 +0200, Jani Nikula wrote: > If a message was received to the user's address that was in a named > group list, notmuch reply does not use that address for picking the > from address. Pushed. d

[PATCH v3 0/2] Automatic tag-based exclusion

2012-01-16 Thread David Bremner
On Sat, 14 Jan 2012 19:17:32 -0500, Austin Clements wrote: > This fixes the symbol visibility warning Jamie pointed out. Third time the charm. pushed. Please (somebody) consider writing NEWS and man page updates. d

[RFC] vim plugin rewrite II

2012-01-16 Thread Anton Khirnov
On Mon, 16 Jan 2012 09:34:59 -0800, Jameson Graef Rollins wrote: > On Sun, 15 Jan 2012 16:34:08 +0100, Anton Khirnov > wrote: > > Yes, it's still synchronous. I suppose making it asynchronous shouldn't > > be all that hard, i just never found enough time and motivation for that > > (it was

[PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions

2012-01-16 Thread David Edmondson
tion/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/06824ed7/attachment.pgp>

[PATCH 1/2] test: Add "test_expect_emacs_t" to assist with emacs testing.

2012-01-16 Thread David Edmondson
hink? Agreed. I'll do it. -- 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/20120116/c118d6ea/attachment.pgp>

[PATCH 2/2] test: Add tests for `notmuch-show-test-clean-address'.

2012-01-16 Thread David Edmondson
available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/1f30ad7d/attachment.pgp>

Partial words on notmuch search?

2012-01-16 Thread David Bremner
On Mon, 16 Jan 2012 22:21:03 +0200, Andrei Popescu wrot> > $ notmuch count 'Debian' > 65888 > $ notmuch count '*ebian' > 3 > $ notmuch count '?ebian' > 3 > $ notmuch count 'ebian' > 3 > $ notmuch count '0ebian' You have to delve into the mysteries of xapian query parsing unfortunately. The

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

2012-01-16 Thread David Bremner
On Mon, 16 Jan 2012 15:51:30 -0500, Aaron Ecay wrote: > On Mon, 16 Jan 2012 11:22:27 +, David Edmondson wrote: > > On Mon, 16 Jan 2012 12:17:03 +0100, Pieter Praet > > wrote: > > > Alas, those text properties actually *were* effective (and I liked them :) > > > > Well, not in emacs 24. My

[PATCH v2 5/4] emacs: Add customization for the first line of quotes.

2012-01-16 Thread Adam Wolfe Gordon
Add a customization option, notmuch-mua-reply-quoth, which controls the first line of the reply body (typically, "On %date%, %from% wrote:"). This allows users who like other styles or correspond in other languages to set an appropriate line using any of the quoted message's headers. --- Due to

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

2012-01-16 Thread Pieter Praet
* emacs/notmuch-show.el (notmuch-show-buttonized-link-colors): new defcustom, allows toggling colorization of buttonized links. * emacs/notmuch-show.el (notmuch-show-buttonized-link-present), * emacs/notmuch-show.el (notmuch-show-buttonized-link-present-and-unread), * emacs/notmuch-show.el

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

2012-01-16 Thread Pieter Praet
On Mon, 16 Jan 2012 11:45:13 +, David Edmondson wrote: > On Thu, 12 Jan 2012 18:23:45 +0100, Pieter Praet wrote: > > + 'face (cond > > + ((notmuch-show-found-target-p message-id t) > > + > >

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

2012-01-16 Thread Pieter Praet
On Mon, 16 Jan 2012 11:45:13 +, David Edmondson wrote: > On Thu, 12 Jan 2012 18:23:45 +0100, Pieter Praet wrote: > > + 'face (cond > > + ((notmuch-show-found-target-p message-id t) > > + > >

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

2012-01-16 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 2/3] emacs: `notmuch-show-buttonize-links' only `notmuch-show's a message if it exists

2012-01-16 Thread Pieter Praet
On Mon, 16 Jan 2012 11:43:25 +, David Edmondson wrote: > On Thu, 12 Jan 2012 18:23:44 +0100, Pieter Praet wrote: > > + (make-text-button string-start string-end > > 'action `(lambda (arg) > > - (notmuch-show ,(match-string-no-properties

[PATCH] test: cli: getting/setting/removing config values

2012-01-16 Thread Pieter Praet
On Mon, 16 Jan 2012 07:37:13 -0400, David Bremner wrote: > On Mon, 16 Jan 2012 11:31:11 +0100, Pieter Praet wrote: > > > > Hmmm, `dnsdomainname' returns "(none)" here; Does it work for you? > > > > Running `domainname' instead seems to do the right thing though... > > > > I make no claims

[PATCH] v2 [RFC] emacs: merge overhauled `notmuch-cycle-notmuch-buffers' into `notmuch'

2012-01-16 Thread Pieter Praet
Make `notmuch-cycle-notmuch-buffers' more Lispy and merge into `notmuch', eliminating the need to hog yet another keybind. --- emacs/notmuch.el | 40 ++-- 1 files changed, 14 insertions(+), 26 deletions(-) diff --git a/emacs/notmuch.el b/emacs/notmuch.el

[PATCH] [RFC] emacs: merge overhauled `notmuch-cycle-notmuch-buffers' into `notmuch'

2012-01-16 Thread Pieter Praet
On Mon, 16 Jan 2012 11:47:03 +, David Edmondson wrote: > On Mon, 16 Jan 2012 12:28:06 +0100, Pieter Praet wrote: > > Perhaps we could use `ido-completion-help', like Org-mode does with > > `org-iswitchb' ? OTOH, that would require an extra [RET]. Bah... > > I'm a no-ido person. > > > >

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

2012-01-16 Thread Aaron Ecay
On Mon, 16 Jan 2012 08:39:30 +, David Edmondson wrote: > On Mon, 16 Jan 2012 02:38:38 -0500, Aaron Ecay wrote: > > - Greater flexibility in the construction of address lists. For example, > > there are some email lists where I want replies to list mail to go only > > to the list, not

[PATCH 1/2] test: Add "test_expect_emacs_t" to assist with emacs testing.

2012-01-16 Thread David Edmondson
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/20120116/4041a02b/attachment.pgp>

[PATCH 2/2] emacs: Another special case for `notmuch-show-clean-address'.

2012-01-16 Thread David Edmondson
Remove backslashes. --- Including a new test case. emacs/notmuch-show.el | 14 +- emacs/notmuch-test.el |6 -- 2 files changed, 13 insertions(+), 7 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index 02819f3..6b9d1b8 100644 ---

[PATCH 1/2] emacs: Avoid `mail-header-parse-address' in `notmuch-show-clean-address'.

2012-01-16 Thread David Edmondson
`mail-header-parse-address' expects un-decoded mailbox parts, which is not what we have at this point. Replace it with simple string deconstruction. --- Fix the failing test in the previous set. emacs/notmuch-show.el | 48 +++- 1 files changed, 35

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

2012-01-16 Thread David Edmondson
Nice. +1. -- 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/20120116/d6e88e8a/attachment-0001.pgp>

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

2012-01-16 Thread David Edmondson
+1. -- 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/20120116/f4b4bfe0/attachment.pgp>

[PATCH] v2 [RFC] emacs: merge overhauled `notmuch-cycle-notmuch-buffers' into `notmuch'

2012-01-16 Thread David Edmondson
(progn > + (switch-to-buffer next-buffer) > + (bury-buffer old-buffer)) > + (notmuch-hello Integrating with `notmuch' is fine with me. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 b

[PATCH] emacs: Truncate lines and do not enable visual-line-mode in notmuch-show buffers.

2012-01-16 Thread Aaron Ecay
On Mon, 16 Jan 2012 14:13:22 +, David Edmondson wrote: > Enable the truncation of lines in `notmuch-show-mode' to avoid visual > noise caused by the wrapping of the header lines. > > Don't enable `visual-line-mode' because it disables line truncation. > > The benefits of `visual-line-mode'

[PATCH 2/2] test: Add tests for `notmuch-show-test-clean-address'.

2012-01-16 Thread David Edmondson
--- Add three tests. The third one currently fails. emacs/notmuch-test.el | 87 + test/emacs|6 +++ 2 files changed, 93 insertions(+), 0 deletions(-) create mode 100644 emacs/notmuch-test.el diff --git a/emacs/notmuch-test.el

[PATCH 1/2] test: Add "test_expect_emacs_t" to assist with emacs testing.

2012-01-16 Thread David Edmondson
"test_expect_emacs_t" expects a single argument. If this is `t' then the test passes. Otherwise the argument provides the details of the test failure to be reported. --- For use in the following set of tests and later for other emacs related tests. test/test-lib.sh | 20

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

2012-01-16 Thread Aaron Ecay
On Mon, 16 Jan 2012 17:57:33 +0100, Pieter Praet wrote: > * emacs/notmuch-show.el (notmuch-show-buttonized-link-colors): > new defcustom, allows toggling colorization of buttonized links. > > * emacs/notmuch-show.el (notmuch-show-buttonized-link-present), > * emacs/notmuch-show.el

[PATCH] emacs: Improved printing support.

2012-01-16 Thread Aaron Ecay
I don?t have (easy access to) a printer, so I haven?t actually exercised the printing functionality of this patch. In general, the patch LGTM. Specific comments below. On Tue, 3 Jan 2012 14:12:26 +, David Edmondson wrote: > Add various functions to print notmuch messages and tie them

[PATCH v2] Document external dependencies in the test suite

2012-01-16 Thread Ethan Glasser-Camp
On 01/16/2012 04:09 PM, Xavier Maillard wrote: > Looks good but just a nit: what is the meaning of the '1' ? It's a manpage-like indication that these are standalone programs and not, say, libraries or system calls :) Not that I expect there to be library dependencies for just the test suite,

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

2012-01-16 Thread Aaron Ecay
On Mon, 16 Jan 2012 11:22:27 +, David Edmondson wrote: > On Mon, 16 Jan 2012 12:17:03 +0100, Pieter Praet wrote: > > Alas, those text properties actually *were* effective (and I liked them :) > > Well, not in emacs 24. My reading of the source was that overlays were > intended to stomp on

[PATCH] debian: add a recommends for w3m-el or w3m-el-snapshot to notmuch-emacs

2012-01-16 Thread David Edmondson
is be 'obsolete'? -- 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/20120116/de0bd52a/attachment.pgp>

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

2012-01-16 Thread Adam Wolfe Gordon
On Mon, Jan 16, 2012 at 15:31, Aaron Ecay wrote: > One advantage of supercite is that it allows non-English speakers to set > up the ?On X, X wrote? line as they prefer. ?Notmuch?s current approach > (a hard-coded C string) is the opposite of internationalized. ?So it > would be nice to support

[PATCH 1/4] emacs: unify search mechanisms

2012-01-16 Thread Dmitry Kurochkin
On Mon, 16 Jan 2012 11:35:37 +, David Edmondson wrote: > On Sun, 25 Dec 2011 08:14:52 +0400, Dmitry Kurochkin gmail.com> wrote: > > Before the change, there were two ways to do search in Emacs UI: > > search widget in notmuch-hello buffer and `notmuch-search' > > function bound to "s".

[PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions

2012-01-16 Thread David Bremner
On Mon, 16 Jan 2012 22:06:15 +0400, Dmitry Kurochkin wrote: > > If others like it, how about reverting to the old bindings until the new > ones are implemented? > Flip-flopping the bindings will just cause more confusion, in my opinion. It is easy (and documented in the wiki) to customize the

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

2012-01-16 Thread Tomi Ollila
On Mon, 16 Jan 2012 08:49:03 +, David Edmondson wrote: > On Sun, 15 Jan 2012 12:58:40 -0500, Austin Clements > wrote: > > This, I suspect, brings us back to what may have been Dmitry's original > concern. If we codify the current behaviour then we're actually > _forcing_ clients to use

[PATCH v2 3/3] search: Support automatic tag exclusions

2012-01-16 Thread Jeremy Nickurak
On Mon, Jan 16, 2012 at 12:28, Austin Clements wrote: > Quoth David Edmondson on Jan 16 at ?9:12 am: >> Having "deleted" and "spam" as default settings in the configuration >> file might be more reasonable. > > Sorry, I'm confused. ?Are you saying deleted;spam should or should not > be the

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

2012-01-16 Thread Austin Clements
Quoth Jameson Graef Rollins on Jan 16 at 11:31 am: > On Sun, 15 Jan 2012 12:16:36 +, Mark Walters gmail.com> wrote: > > Define a keymap for attachment buttons to allow multiple actions. > > Define 3 possible actions: > > save attachment: exactly as currently, > > view attachment: uses

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

2012-01-16 Thread Tomi Ollila
On Mon, 16 Jan 2012 11:56:40 +0100, Pieter Praet wrote: > Less code, same results, without sacrificing readability. > > --- +1 Tomi

[PATCH v2 3/3] search: Support automatic tag exclusions

2012-01-16 Thread Austin Clements
Quoth David Edmondson on Jan 16 at 9:12 am: > On Sat, 14 Jan 2012 15:40:26 -0800, Jameson Graef Rollins finestructure.net> wrote: > > This patch looks fine. Philosophical UI discussion to follow: > > > > On Fri, 13 Jan 2012 18:07:04 -0500, Austin Clements > > wrote: > > > +if

[PATCH v2 3/3] search: Support automatic tag exclusions

2012-01-16 Thread Jameson Graef Rollins
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/20120116/81f89072/attachment.pgp>

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

2012-01-16 Thread Austin Clements
LGTM. Quoth Pieter Praet on Jan 16 at 11:38 am: > - Add `notmuch-send' to the `message' group, as per Austin's > recommendation [1]. This is actually the reverse of what I was suggesting (though is also reasonable). Since notmuch uses message-mode, I was saying that the notmuch group (or the

[PATCH] emacs: Truncate lines and do not enable visual-line-mode in notmuch-show buffers.

2012-01-16 Thread David Edmondson
Enable the truncation of lines in `notmuch-show-mode' to avoid visual noise caused by the wrapping of the header lines. Don't enable `visual-line-mode' because it disables line truncation. The benefits of `visual-line-mode' were that it wrapped long lines in received messages. With

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

2012-01-16 Thread Jameson Graef Rollins
available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/afe4c7cb/attachment.pgp>

[PATCH v2] Document external dependencies in the test suite

2012-01-16 Thread Ethan Glasser-Camp
From: Ethan Glasser-Camp Add an explicit note to the README explaining what programs are necessary and the perhaps-surprising behavior of skipping tests if they aren't present. Signed-off-by: Ethan Glasser-Camp --- v2 suggested by Dmitry Kurochkin: document all the

The overloading of show (was Re: [PATCH] Output unmodified Content-Type header value for JSON format.)

2012-01-16 Thread David Edmondson
-- 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/20120116/029db09b/attachment-0001.pgp>

[PATCH] emacs-hello: Do not calculate the count of the messages in

2012-01-16 Thread Daniel Schoepe
e Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/f3717455/attachment.pgp>

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

2012-01-16 Thread Pieter Praet
On Mon, 16 Jan 2012 11:22:27 +, David Edmondson wrote: > On Mon, 16 Jan 2012 12:17:03 +0100, Pieter Praet wrote: > > Alas, those text properties actually *were* effective (and I liked them :) > > Well, not in emacs 24. My reading of the source was that overlays were > intended to stomp on

[PATCH] [RFC] emacs: merge overhauled `notmuch-cycle-notmuch-buffers' into `notmuch'

2012-01-16 Thread Pieter Praet
On Mon, 16 Jan 2012 12:28:06 +0100, Pieter Praet wrote: > On Mon, 16 Jan 2012 11:06:25 +, David Edmondson wrote: > > On Mon, 16 Jan 2012 11:42:31 +0100, Pieter Praet > > wrote: > > > CLisp'ier version of `notmuch-cycle-notmuch-buffers', merged into > > > `notmuch', > > > eliminating the

[PATCH] [RFC] emacs: merge overhauled `notmuch-cycle-notmuch-buffers' into `notmuch'

2012-01-16 Thread Pieter Praet
On Mon, 16 Jan 2012 11:06:25 +, David Edmondson wrote: > On Mon, 16 Jan 2012 11:42:31 +0100, Pieter Praet wrote: > > CLisp'ier version of `notmuch-cycle-notmuch-buffers', merged into `notmuch', > > eliminating the need to hog yet another keybind. > > The lisp-ier-ness changes are good. > >

[PATCH 1/4] emacs: unify search mechanisms

2012-01-16 Thread Jani Nikula
On Mon, 16 Jan 2012 15:39:14 +0400, Dmitry Kurochkin wrote: > On Mon, 16 Jan 2012 11:35:37 +, David Edmondson wrote: > > On Sun, 25 Dec 2011 08:14:52 +0400, Dmitry Kurochkin > gmail.com> wrote: > > > Before the change, there were two ways to do search in Emacs UI: > > > search widget in

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

2012-01-16 Thread Pieter Praet
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. > --- >

[PATCH] Document the dependency on dtach in the test suite

2012-01-16 Thread Ethan Glasser-Camp
From: Ethan Glasser-Camp dtach (and before that, screen) are used to run emacs with more verisimilitude, but this is only mentioned in the NEWS. Mention this in the test/README file, to make it easier to find out why a bunch of tests are failing/skipped. Signed-off-by:

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

2012-01-16 Thread Daniel Schoepe
vailable URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/ef4cf53a/attachment.pgp>

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

2012-01-16 Thread Pieter Praet
On Mon, 16 Jan 2012 10:56:03 +, David Edmondson wrote: > Given the discussion that followed this patch I'd like to mark it as > 'obsolete'. > > Any objections? None whatsoever :) Peace -- Pieter

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

2012-01-16 Thread Pieter Praet
Less code, same results, without sacrificing readability. --- v2: Fixed indentation oversight @ `notmuch-hello-insert-tags'. emacs/notmuch-address.el |6 +++--- emacs/notmuch-hello.el | 22 +++--- emacs/notmuch-show.el| 12 ++-- emacs/notmuch.el |

[PATCH] debian: add a recommends for w3m-el or w3m-el-snapshot to notmuch-emacs

2012-01-16 Thread David Bremner
On Mon, 16 Jan 2012 15:50:45 +, David Edmondson wrote: > On Wed, 7 Dec 2011 22:13:06 -0400, David Bremner > wrote: > > From: David Bremner > > > > Installing w3m-el provides a nicer out of the box experience for > > viewing html in notmuch, and the overhead is not too bad (about 5M > >

[PATCH 1/4] emacs: unify search mechanisms

2012-01-16 Thread David Edmondson
lication/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/72e6a33d/attachment.pgp>

[PATCH] [RFC] emacs: merge overhauled `notmuch-cycle-notmuch-buffers' into `notmuch'

2012-01-16 Thread David Edmondson
mpt to call the function, it doesn't complain? (I haven't tested.) -- 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/20120116/23d8da12/attachment.pgp>

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

2012-01-16 Thread David Edmondson
for a missing or read message). -- 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/20120116/b16fd3ac/attachment.pgp>

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

2012-01-16 Thread David Edmondson
re Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/874115a1/attachment.pgp>

[PATCH] [RFC] emacs: merge overhauled `notmuch-cycle-notmuch-buffers' into `notmuch'

2012-01-16 Thread Pieter Praet
CLisp'ier version of `notmuch-cycle-notmuch-buffers', merged into `notmuch', eliminating the need to hog yet another keybind. FYI: causes 2 compile warnings due to requiring the cl package at runtime. --- emacs/notmuch.el | 34 -- 1 files changed, 8

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

2012-01-16 Thread Pieter Praet
On Sun, 15 Jan 2012 17:56:37 +0400, Dmitry Kurochkin wrote: > 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

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

2012-01-16 Thread David Edmondson
-- 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/20120116/492672ef/attachment.pgp>

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

2012-01-16 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 - External Commands - Appearance As an added benefit, defcustom keyword args are now

[PATCH 1/4] emacs: unify search mechanisms

2012-01-16 Thread David Edmondson
on-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/72b34e3c/attachment.pgp>

[PATCH v3 0/2] Automatic tag-based exclusion

2012-01-16 Thread Jameson Graef Rollins
ype: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/dd6a9821/attachment.pgp>

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

2012-01-16 Thread Pieter Praet
On Sat, 14 Jan 2012 13:08:28 -0500, Austin Clements wrote: > Quoth Pieter Praet on Jan 14 at 10:04 am: > > 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 > >

[PATCH v2 3/3] search: Support automatic tag exclusions

2012-01-16 Thread Jameson Graef Rollins
n-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/06a6a0e1/attachment.pgp>

[PATCH] emacs-hello: Do not calculate the count of the messages in

2012-01-16 Thread David Edmondson
le Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/4b6f40fd/attachment-0001.pgp>

[PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions

2012-01-16 Thread Jameson Graef Rollins
attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/682e9754/attachment.pgp>

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

2012-01-16 Thread Jameson Graef Rollins
ailable Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/11ed866f/attachment.pgp>

[PATCH] test: cli: getting/setting/removing config values

2012-01-16 Thread Pieter Praet
On Sat, 14 Jan 2012 08:16:30 -0400, David Bremner wrote: > On Sat, 14 Jan 2012 09:57:56 +0100, Pieter Praet wrote: > > Full test coverage for getting, setting and removing options in > > notmuch(1)'s config file ($NOTMUCH_CONFIG or $HOME/.notmuch-config). > > > > --- > > + > > +# FIXME: Not the

[PATCH] emacs: Improved printing support.

2012-01-16 Thread David Edmondson
L: <http://notmuchmail.org/pipermail/notmuch/attachments/20120116/2ee82c6f/attachment.pgp>

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

2012-01-16 Thread Jameson Graef Rollins
we should. 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/20120116/c83bdf52/attachment.pgp>

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

2012-01-16 Thread Pieter Praet
On Thu, 12 Jan 2012 18:04:10 +0100, Pieter Praet wrote: > On Wed, 16 Nov 2011 23:47:53 +0100, Pieter Praet wrote: > > On Sun, 13 Nov 2011 13:16:48 -0800, Jameson Graef Rollins > finestructure.net> wrote: > > > This patch breaks out much of the functionality of > > >

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

2012-01-16 Thread David Edmondson
ect when you add/remove tags? -- 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/20120116/945dde66/attachment.pgp>

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

2012-01-16 Thread Adam Wolfe Gordon
Using the new JSON reply format allows emacs to quote HTML parts nicely by using mm-display-part to turn them into displayable text, then quoting them. This is very useful for users who regularly receive HTML-only email. The behavior for messages that contain plain text parts should be unchanged,

[PATCH v2 3/4] man: Update notmuch-reply man page for JSON format.

2012-01-16 Thread Adam Wolfe Gordon
--- man/man1/notmuch-reply.1 |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/man/man1/notmuch-reply.1 b/man/man1/notmuch-reply.1 index 5160ece..ea7f87b 100644 --- a/man/man1/notmuch-reply.1 +++ b/man/man1/notmuch-reply.1 @@ -43,6 +43,11 @@ include .BR default

[PATCH v2 2/4] reply: Add a JSON reply format.

2012-01-16 Thread Adam Wolfe Gordon
This new JSON format for replies includes headers generated for a reply message as well as the headers and all text parts of the original message. Using this data, a client can intelligently create a reply. For example, the emacs client will be able to create replies with quoted HTML parts by

[PATCH v2 1/4] test: Add broken test for the new JSON reply format.

2012-01-16 Thread Adam Wolfe Gordon
--- test/multipart |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/test/multipart b/test/multipart index f83526b..f5ebf04 100755 --- a/test/multipart +++ b/test/multipart @@ -589,6 +589,13 @@ Non-text part: text/html EOF test_expect_equal_file OUTPUT EXPECTED

  1   2   3   >