Re: Updated version of the patch-set for tagging forwarded messages

Subject: Re: Updated version of the patch-set for tagging forwarded messages

Date: Sat, 13 Apr 2019 13:19:21 +0300

To: Örjan Ekeberg, notmuch@notmuchmail.org

Cc:

From: Tomi Ollila


On Fri, Apr 12 2019, Örjan Ekeberg wrote:

> This is a new (final?) version of the patch set for adding reference
> links to forwarded messages, and tags to forwarded and replied messages.
>
> Thanks for all the comments and suggesions for improvements.  Apart
> from cleaning up the test code, the only change from the last posted
> version is that I have reversed the dependency so that notmuch-mua.el
> requires notmuch-message.el.  This change was needed to make the code
> emacs-compile cleanly since the code in notmuch-mua uses the variables
> defined in notmuch-message.

Series LGTM.

>
> Örjan Ekeberg (4):
>   emacs: Add References header to forwarded messages
>   emacs: Use a buffer-local variable to update tags when sending replies
>   emacs: Tag forwarded messages with +forwarded (customizable)
>   test: add test for checking forwarded messages
>
>  emacs/notmuch-message.el      | 38 ++++++++++++++++++++++++--------
>  emacs/notmuch-mua.el          | 27 ++++++++++++++++++++---
>  test/T730-emacs-forwarding.sh | 41 +++++++++++++++++++++++++++++++++++
>  3 files changed, 94 insertions(+), 12 deletions(-)
>  create mode 100755 test/T730-emacs-forwarding.sh
>
> -- 
> 2.20.1
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch
_______________________________________________
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch

Thread: