Re: [PATCH v2 0/2] Allow forwarding of all open messages in the current view.

Subject: Re: [PATCH v2 0/2] Allow forwarding of all open messages in the current view.

Date: Sun, 07 Feb 2016 23:57:18 +0000

To: David Edmondson, notmuch@notmuchmail.org

Cc:

From: Mark Walters


On Sat, 06 Feb 2016, David Edmondson <dme@dme.org> wrote:
> Allow forwarding of all open messages in the current view.
>
> In response to id:"87fvyhb40x.fsf@alice.fifthhorseman.net".
>
> v2:
> - Use multiple message forwarding to implement single message
>   forwarding.
>

This basically looks good to me. At the moment there is some typo in the
first patch introducing two extra parenthesis at the end of the 
notmuch-mua-new-forward-messages function.

I checked with both message-forward-as-mime true (default giving rfc822
parts) and set to nil (including the messages as plain text) and both
looked as I would expect.

Finally, with this in place it would be very easy to introduce thread
forwarding from search mode. But that can be left until after this.

Best wishes

Mark


>
> David Edmondson (2):
>   emacs: Add `notmuch-show-forward-open-messages'.
>   emacs: `notmuch-show-forward-message' can use
>     `notmuch-mua-new-forward-messages'
>
>  emacs/notmuch-mua.el  | 62 +++++++++++++++++++++++++++++++++++++--------------
>  emacs/notmuch-show.el | 15 +++++++++++--
>  2 files changed, 58 insertions(+), 19 deletions(-)
>
> -- 
> 2.1.4
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch

Thread: