[PATCH] lib: fix clang build

2013-08-18 Thread Jani Nikula
Long story short, fix build on recent (3.2+) clang. The long story for posterity follows. gcc 4.6 added new warnings about structs with greater visibility than their fields. The warnings were silenced by adjusting visibility in commit d5523ead90b6be2b07d4af745b8ed9b980a6b9f1 Author: Carl Worth

Compiling fails

2013-08-18 Thread Jani Nikula
On Thu, 18 Apr 2013, David Bremner wrote: > Simonas Kazlauskas writes: > >> Of course `#pragma GCC` has to mean something and might be a reason why >> compiling with `CXX=clang++` fails. > > Yep. clang is not currently supported. I'm not sure how difficult it > would be to get working. For

cli: add --include-html option to notmuch show

2013-08-18 Thread Jani Nikula
On Sat, 17 Aug 2013, John Lenz wrote: > On Sun Aug 4 14:47 -0500 2013, Tomi Ollila wrote: >> The next question is should we have new option as >> >> --include-html >> >> or as >> >> --include-html=(true|false) >> >> or even >> >> --body=(true|false|text-and-html) >> >> See --exclude

[PATCH v2 00/11] contrib: pick: keybindings

2013-08-18 Thread Mark Walters
This v2 of the patch set id:1373047878-20822-1-git-send-email-markwalters1009 at gmail.com to fix the two things mentioned by Tomi in id:m2ppuwt7db.fsf at guru.guru-group.fi : That is the comment for notmuch-show-get-prop now has an XXX and the keybinding for activating a pick show window button

[PATCH v2 01/11] contrib: pick: override notmuch-show-get-prop

2013-08-18 Thread Mark Walters
We override notmuch-show-get-prop so that many of the show functions can be used in notmuch-pick without modification. The main use is that it means notmuch-show-get-message-id `works' in pick. Thus we get all the stash functions and several other `for free' in pick. ---

[PATCH v2 02/11] contrib: pick: Link in notmuch-show-pipe-message

2013-08-18 Thread Mark Walters
Since we can now use show functions directly in pick we can drop pick-pipe-message. --- contrib/notmuch-pick/notmuch-pick.el | 31 ++- 1 files changed, 2 insertions(+), 29 deletions(-) diff --git a/contrib/notmuch-pick/notmuch-pick.el

[PATCH v2 03/11] contrib: pick: Link in attachment functions straight from notmuch-show

2013-08-18 Thread Mark Walters
We can use the attachment functions straight from notmuch-show. notmuch-show-view-all-mime-parts might be deprecated so we either want to undeprecate it or not have this binding. --- contrib/notmuch-pick/notmuch-pick.el |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git

[PATCH v2 04/11] contrib: pick: Link in stash map straight from notmuch-show

2013-08-18 Thread Mark Walters
These functions all now work straight from their notmuch-show implementation so link them in. Stash functionality was one of the key missing things in notmuch-pick. --- contrib/notmuch-pick/notmuch-pick.el |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git

[PATCH v2 05/11] contrib: pick: add in to-message-window function

2013-08-18 Thread Mark Walters
--- contrib/notmuch-pick/notmuch-pick.el | 13 + 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el index 89e6d4b..d00d324 100644 --- a/contrib/notmuch-pick/notmuch-pick.el +++

[PATCH v2 06/11] contrib: pick: add button press helper

2013-08-18 Thread Mark Walters
We will want to be able to activate buttons not in the current buffer (ie in the message pane) so it is helpful to have a way of activating a button without signalling error if there is no button. --- contrib/notmuch-pick/notmuch-pick.el |8 1 files changed, 8 insertions(+), 0

[PATCH v2 07/11] contrib: pick: pass tab through to the message pane

2013-08-18 Thread Mark Walters
This makes tab move to next button in the message pane and binds button activate (in message pane) to "e". This means that is easy to toggle hidden parts or hidden citations etc in the message pane. --- contrib/notmuch-pick/notmuch-pick.el |8 1 files changed, 8 insertions(+), 0

[PATCH v2 08/11] contrib: pick: close window function

2013-08-18 Thread Mark Walters
--- contrib/notmuch-pick/notmuch-pick.el | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el index bd506ca..ba252d5 100644 --- a/contrib/notmuch-pick/notmuch-pick.el +++

[PATCH v2 09/11] contrib: pick: make help close the message pane first

2013-08-18 Thread Mark Walters
Previously pressing "?" for help when the message pane was open meant the help window was very small. Close the message pane before displaying help. --- contrib/notmuch-pick/notmuch-pick.el |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git

[PATCH v2 11/11] contrib: pick: use close-message-pane for reply etc

2013-08-18 Thread Mark Walters
We can save some code duplication by using the new close-message-pane functionality for reply, forward, and new mail. --- contrib/notmuch-pick/notmuch-pick.el | 43 +++-- 1 files changed, 4 insertions(+), 39 deletions(-) diff --git

[PATCH v2 10/11] contrib: pick: add in binding to view raw message

2013-08-18 Thread Mark Walters
Note this does rely on the fact that we have over-ridden notmuch-show-get-properties --- contrib/notmuch-pick/notmuch-pick.el |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el index

[PATCH v4 7/8] NEWS: cli options for improved duplicate message handling

2013-08-18 Thread Jani Nikula
--- NEWS | 20 1 file changed, 20 insertions(+) diff --git a/NEWS b/NEWS index 722a35d..2e845f4 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,23 @@ + +Notmuch 0.17 (UNRELEASED) += + +Command-Line Interface +-- + +New options to better

[PATCH] test: Canonicalize RFC 2047 encoding and charset

2013-08-18 Thread Austin Clements
RFC 2047 states that the encoding and charset in an encoded word are case-insensitive, so force them to lower case in the reply test. This fixes an issue caused by GMime versions (somewhere between 2.6.10 and 2.6.16), which changed the capitalization of the encoding. --- test/reply |8

[PATCH] test: Canonicalize RFC 2047 encoding and charset

2013-08-18 Thread Jani Nikula
On Sun, 18 Aug 2013, Austin Clements wrote: > RFC 2047 states that the encoding and charset in an encoded word are > case-insensitive, so force them to lower case in the reply test. This > fixes an issue caused by GMime versions (somewhere between 2.6.10 and > 2.6.16), which changed the

[PATCH 1/2] cli: abstract common config get/set code

2013-08-18 Thread Jani Nikula
Pretty straightforward abstraction similar to get/set list. --- notmuch-config.c | 80 -- 1 file changed, 29 insertions(+), 51 deletions(-) diff --git a/notmuch-config.c b/notmuch-config.c index befe9b5..305d213 100644 --- a/notmuch-config.c

[PATCH 2/2] cli: define config getters and setters using a macro

2013-08-18 Thread Jani Nikula
There's plenty of duplicated code in defining the functions for config get/set. Add macros to define the functions. --- This might be a bit too tricky for some people's tastes... let's see! ;) --- notmuch-config.c | 141 ++ 1 file changed, 37

[PATCH] test: Canonicalize RFC 2047 encoding and charset

2013-08-18 Thread Tomi Ollila
On Sun, Aug 18 2013, Austin Clements wrote: > RFC 2047 states that the encoding and charset in an encoded word are > case-insensitive, so force them to lower case in the reply test. This > fixes an issue caused by GMime versions (somewhere between 2.6.10 and > 2.6.16), which changed the

[PATCH 2/2] cli: define config getters and setters using a macro

2013-08-18 Thread Tomi Ollila
On Sun, Aug 18 2013, Jani Nikula wrote: > There's plenty of duplicated code in defining the functions for config > get/set. Add macros to define the functions. > > --- > > This might be a bit too tricky for some people's tastes... let's see! > ;) Yes, a bit tricky. Nevertheless I like these...

cli: add --include-html option to notmuch show

2013-08-18 Thread Tomi Ollila
On Sun, Aug 18 2013, Jani Nikula wrote: > On Sat, 17 Aug 2013, John Lenz wrote: >> On Sun Aug 4 14:47 -0500 2013, Tomi Ollila wrote: >>> The next question is should we have new option as >>> >>> --include-html >>> >>> or as >>> >>> --include-html=(true|false) >>> >>> or even >>> >>>

[PATCH] lib: fix clang build

2013-08-18 Thread Tomi Ollila
On Sun, Aug 18 2013, Jani Nikula wrote: > Long story short, fix build on recent (3.2+) clang. > > The long story for posterity follows. > > gcc 4.6 added new warnings about structs with greater visibility than > their fields. The warnings were silenced by adjusting visibility in > > commit

[PATCH] lib: fix clang build

2013-08-18 Thread Jani Nikula
On Sun, 18 Aug 2013, Tomi Ollila wrote: > On Sun, Aug 18 2013, Jani Nikula wrote: > >> Long story short, fix build on recent (3.2+) clang. >> >> The long story for posterity follows. >> >> gcc 4.6 added new warnings about structs with greater visibility than >> their fields. The warnings were

search order

2013-08-18 Thread David Bremner
Ramakrishnan Muthukrishnan writes: > If I startup notmuch and then do a M-x notmuch-search and then *, I see > the messages with the newest on the top. But if I instead, startup > notmuch and then hit "s", I see that the new messages are at the > bottom. The value of notmuch-search-oldest-first

missing part in emacs UI

2013-08-18 Thread Jameson Graef Rollins
e any ideas what the issue could be? Thanks in advance. 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/20130818/721e7277/attachment.pgp>