Re: [PATCH v2] emacs: cl-letf enriched-decode-display-prop for text/encriched display

2017-09-20 Thread David Edmondson
On Wednesday, 2017-09-20 at 08:25:44 +0300, Tomi Ollila wrote: > I felt mentioning this (require 'enriched) important enough to be > stored in the blockchain of notmuch commit history -- knowing such > a subtle behaviour may prevent related bugs somewhere, sometime... Could you add a comment in

Re: [PATCH 0/9] argument parsing fixes and improvements

2017-09-20 Thread Jani Nikula
On Wed, Sep 20, 2017 at 12:43 AM, Daniel Kahn Gillmor wrote: > On Tue 2017-09-19 23:39:20 +0300, Jani Nikula wrote: >> I had some other things in mind, but ended up with this prep series >> instead... The main thing is patch 6 adding --no-argument style negating >>

Re: [PATCH 0/9] argument parsing fixes and improvements

2017-09-20 Thread Daniel Kahn Gillmor
Hi Jani-- On Wed 2017-09-20 12:16:33 +0300, Jani Nikula wrote: > Those are good questions, and up for debate. I was more interested in > getting the mask out behaviour for keyword flag arguments, and the > boolean part came practically free. Or, there was certain synergy in > having them behave

Re: [RFC PATCH] emacs: require minimum version to be 24.1 c-b c-b c-b c-b c-t c-e

2017-09-20 Thread Tomi Ollila
On Wed, Sep 20 2017, Daniel Kahn Gillmor wrote: > On Wed 2017-09-20 13:18:44 -0300, David Bremner wrote: >> Tomi Ollila writes: >>> >>> +(defvar notmuch--minimal-version "42.1") ;; almost indistinguishable from >>> magit >> >> typo? > > I think that's what the "c-b c-b c-b

Re: [RFC PATCH] emacs: require minimum version to be 24.1 c-b c-b c-b c-b c-t c-e

2017-09-20 Thread David Bremner
Tomi Ollila writes: > The rfc part is how this should be done -- this change touches > the earliest entry point when one loads/requires 'notmuch. > > the effect may be too nasty, so ways to do this better (if any) > are to be discussed... We could add it to notmuch-pkg.el,

notmuch.el: "wash" date in message display

2017-09-20 Thread Matt Armstrong
I miss some of the Gnus "wash" functions available for message display. There was one that either reformatted the Date: header into my time zone, or displayed the Date in terms of elapsed time from now (e.g. 1 hour ago, 1 day ago, etc.). I don't remember which, but I'd be happy with either. Has

Re: [RFC PATCH] emacs: require minimum version to be 24.1 c-b c-b c-b c-b c-t c-e

2017-09-20 Thread Daniel Kahn Gillmor
On Wed 2017-09-20 13:18:44 -0300, David Bremner wrote: > Tomi Ollila writes: >> >> +(defvar notmuch--minimal-version "42.1") ;; almost indistinguishable from >> magit > > typo? I think that's what the "c-b c-b c-b c-b c-t c-e" in the Subject: is about. if you do that, you

[RFC PATCH] emacs: require minimum version to be 24.1 c-b c-b c-b c-b c-t c-e

2017-09-20 Thread Tomi Ollila
The rfc part is how this should be done -- this change touches the earliest entry point when one loads/requires 'notmuch. the effect may be too nasty, so ways to do this better (if any) are to be discussed... --- emacs/notmuch.el | 7 +++ 1 file changed, 7 insertions(+) diff --git

Re: Guess 'From:' when doing notmuch reply

2017-09-20 Thread Damien Cassou
David Bremner writes: > One option would be for you to customize the output of notmuch-reply in > Emacs. I think the package message-templ (unfortunately only in > marmalade, debian, and my git repo) I'm using borg, so that's not a problem for me. > provides tools that

Re: Guess 'From:' when doing notmuch reply

2017-09-20 Thread David Bremner
Damien Cassou writes: > I'm not sure my company would let me send you even headers of the > email. What I can tell is that my email address does not appear in the > headers (I had a look at > notmuch-reply.c/guess_from_in_received_headers()). I can't really blame > notmuch for

Re: [PATCH v2] emacs: cl-letf enriched-decode-display-prop for text/encriched display

2017-09-20 Thread David Bremner
Tomi Ollila writes: > Dynamically bind enriched-decode-display-prop when inserting > text/enriched part. This complements commit 9b0582383833 for > emacs versions before 24.4 which do not have advice-add functionality. > > Note the (require 'enriched). Without that if