unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH] emacs: make use of `message-make-from'
@ 2016-04-08 21:43 aeuii
  2016-04-09 23:10 ` David Bremner
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: aeuii @ 2016-04-08 21:43 UTC (permalink / raw)
  To: notmuch

[-- Attachment #1: Type: text/plain, Size: 53 bytes --]

Please put my address in CC when replying.  Thanks!


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-emacs-make-use-of-message-make-from.patch --]
[-- Type: text/x-diff, Size: 1432 bytes --]

From 4b9ab261a0ea8a31065e310c5150f522be86d37b Mon Sep 17 00:00:00 2001
From: stefan <aeuii@posteo.de>
Date: Fri, 8 Apr 2016 22:47:06 +0200
Subject: [PATCH] emacs: make use of `message-make-from'

Will respect `mail-from-style'.
---
 emacs/notmuch-mua.el | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el
index cfdac0e..935cbd3 100644
--- a/emacs/notmuch-mua.el
+++ b/emacs/notmuch-mua.el
@@ -318,8 +318,8 @@ modified. This function is notmuch addaptation of
 	(push (cons 'User-Agent user-agent) other-headers))))
 
   (unless (assq 'From other-headers)
-    (push (cons 'From (concat
-		       (notmuch-user-name) " <" (notmuch-user-primary-email) ">")) other-headers))
+    (push (cons 'From (message-make-from
+		       (notmuch-user-name) (notmuch-user-primary-email))) other-headers))
 
   (notmuch-mua-pop-to-buffer (message-buffer-name "mail" to)
 			     (or switch-function (notmuch-mua-get-switch-function)))
@@ -393,7 +393,7 @@ the From: header is already filled in by notmuch."
 	    (ido-completing-read (concat "Sender address for " name ": ") addrs
 				 nil nil nil 'notmuch-mua-sender-history
 				 (car addrs))))
-      (concat name " <" address ">"))))
+      (message-make-from name address))))
 
 (put 'notmuch-mua-new-mail 'notmuch-prefix-doc "... and prompt for sender")
 (defun notmuch-mua-new-mail (&optional prompt-for-sender)
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] emacs: make use of `message-make-from'
  2016-04-08 21:43 [PATCH] emacs: make use of `message-make-from' aeuii
@ 2016-04-09 23:10 ` David Bremner
  2016-04-10  7:24 ` Tomi Ollila
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 7+ messages in thread
From: David Bremner @ 2016-04-09 23:10 UTC (permalink / raw)
  To: aeuii, notmuch

aeuii@posteo.de writes:

> -    (push (cons 'From (concat
> -		       (notmuch-user-name) " <" (notmuch-user-primary-email) ">")) other-headers))
> +    (push (cons 'From (message-make-from
> +		       (notmuch-user-name) (notmuch-user-primary-email))) other-headers))

Looks OK to me. Any comments from our emacs front end experts?

d

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] emacs: make use of `message-make-from'
  2016-04-08 21:43 [PATCH] emacs: make use of `message-make-from' aeuii
  2016-04-09 23:10 ` David Bremner
@ 2016-04-10  7:24 ` Tomi Ollila
  2016-04-10 11:26 ` David Edmondson
  2016-04-10 23:57 ` David Bremner
  3 siblings, 0 replies; 7+ messages in thread
From: Tomi Ollila @ 2016-04-10  7:24 UTC (permalink / raw)
  To: aeuii, notmuch

On Sat, Apr 09 2016, aeuii@posteo.de wrote:

> Please put my address in CC when replying.  Thanks!
>
> From 4b9ab261a0ea8a31065e310c5150f522be86d37b Mon Sep 17 00:00:00 2001
> From: stefan <aeuii@posteo.de>
> Date: Fri, 8 Apr 2016 22:47:06 +0200
> Subject: [PATCH] emacs: make use of `message-make-from'
>
> Will respect `mail-from-style'.

LTGM

Tomi

> ---
>  emacs/notmuch-mua.el | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el
> index cfdac0e..935cbd3 100644
> --- a/emacs/notmuch-mua.el
> +++ b/emacs/notmuch-mua.el
> @@ -318,8 +318,8 @@ modified. This function is notmuch addaptation of
>  	(push (cons 'User-Agent user-agent) other-headers))))
>  
>    (unless (assq 'From other-headers)
> -    (push (cons 'From (concat
> -		       (notmuch-user-name) " <" (notmuch-user-primary-email) ">")) other-headers))
> +    (push (cons 'From (message-make-from
> +		       (notmuch-user-name) (notmuch-user-primary-email))) other-headers))
>  
>    (notmuch-mua-pop-to-buffer (message-buffer-name "mail" to)
>  			     (or switch-function (notmuch-mua-get-switch-function)))
> @@ -393,7 +393,7 @@ the From: header is already filled in by notmuch."
>  	    (ido-completing-read (concat "Sender address for " name ": ") addrs
>  				 nil nil nil 'notmuch-mua-sender-history
>  				 (car addrs))))
> -      (concat name " <" address ">"))))
> +      (message-make-from name address))))
>  
>  (put 'notmuch-mua-new-mail 'notmuch-prefix-doc "... and prompt for sender")
>  (defun notmuch-mua-new-mail (&optional prompt-for-sender)
> -- 
> 2.7.4
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] emacs: make use of `message-make-from'
  2016-04-10 11:26 ` David Edmondson
@ 2016-04-10 10:05   ` Mark Walters
  0 siblings, 0 replies; 7+ messages in thread
From: Mark Walters @ 2016-04-10 10:05 UTC (permalink / raw)
  To: David Edmondson, aeuii, notmuch




On Sun, 10 Apr 2016, David Edmondson <dme@dme.org> wrote:
> On Fri, Apr 08 2016, aeuii@posteo.de wrote:
>> Please put my address in CC when replying.  Thanks!
>
> Looks good.

I agree.

Best wishes

Mark

>
>> From 4b9ab261a0ea8a31065e310c5150f522be86d37b Mon Sep 17 00:00:00 2001
>> From: stefan <aeuii@posteo.de>
>> Date: Fri, 8 Apr 2016 22:47:06 +0200
>> Subject: [PATCH] emacs: make use of `message-make-from'
>>
>> Will respect `mail-from-style'.
>> ---
>>  emacs/notmuch-mua.el | 6 +++---
>>  1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el
>> index cfdac0e..935cbd3 100644
>> --- a/emacs/notmuch-mua.el
>> +++ b/emacs/notmuch-mua.el
>> @@ -318,8 +318,8 @@ modified. This function is notmuch addaptation of
>>  	(push (cons 'User-Agent user-agent) other-headers))))
>>  
>>    (unless (assq 'From other-headers)
>> -    (push (cons 'From (concat
>> -		       (notmuch-user-name) " <" (notmuch-user-primary-email) ">")) other-headers))
>> +    (push (cons 'From (message-make-from
>> +		       (notmuch-user-name) (notmuch-user-primary-email))) other-headers))
>>  
>>    (notmuch-mua-pop-to-buffer (message-buffer-name "mail" to)
>>  			     (or switch-function (notmuch-mua-get-switch-function)))
>> @@ -393,7 +393,7 @@ the From: header is already filled in by notmuch."
>>  	    (ido-completing-read (concat "Sender address for " name ": ") addrs
>>  				 nil nil nil 'notmuch-mua-sender-history
>>  				 (car addrs))))
>> -      (concat name " <" address ">"))))
>> +      (message-make-from name address))))
>>  
>>  (put 'notmuch-mua-new-mail 'notmuch-prefix-doc "... and prompt for sender")
>>  (defun notmuch-mua-new-mail (&optional prompt-for-sender)
>> -- 
>> 2.7.4
>>
>> _______________________________________________
>> notmuch mailing list
>> notmuch@notmuchmail.org
>> https://notmuchmail.org/mailman/listinfo/notmuch
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] emacs: make use of `message-make-from'
  2016-04-08 21:43 [PATCH] emacs: make use of `message-make-from' aeuii
  2016-04-09 23:10 ` David Bremner
  2016-04-10  7:24 ` Tomi Ollila
@ 2016-04-10 11:26 ` David Edmondson
  2016-04-10 10:05   ` Mark Walters
  2016-04-10 23:57 ` David Bremner
  3 siblings, 1 reply; 7+ messages in thread
From: David Edmondson @ 2016-04-10 11:26 UTC (permalink / raw)
  To: aeuii, notmuch

On Fri, Apr 08 2016, aeuii@posteo.de wrote:
> Please put my address in CC when replying.  Thanks!

Looks good.

> From 4b9ab261a0ea8a31065e310c5150f522be86d37b Mon Sep 17 00:00:00 2001
> From: stefan <aeuii@posteo.de>
> Date: Fri, 8 Apr 2016 22:47:06 +0200
> Subject: [PATCH] emacs: make use of `message-make-from'
>
> Will respect `mail-from-style'.
> ---
>  emacs/notmuch-mua.el | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el
> index cfdac0e..935cbd3 100644
> --- a/emacs/notmuch-mua.el
> +++ b/emacs/notmuch-mua.el
> @@ -318,8 +318,8 @@ modified. This function is notmuch addaptation of
>  	(push (cons 'User-Agent user-agent) other-headers))))
>  
>    (unless (assq 'From other-headers)
> -    (push (cons 'From (concat
> -		       (notmuch-user-name) " <" (notmuch-user-primary-email) ">")) other-headers))
> +    (push (cons 'From (message-make-from
> +		       (notmuch-user-name) (notmuch-user-primary-email))) other-headers))
>  
>    (notmuch-mua-pop-to-buffer (message-buffer-name "mail" to)
>  			     (or switch-function (notmuch-mua-get-switch-function)))
> @@ -393,7 +393,7 @@ the From: header is already filled in by notmuch."
>  	    (ido-completing-read (concat "Sender address for " name ": ") addrs
>  				 nil nil nil 'notmuch-mua-sender-history
>  				 (car addrs))))
> -      (concat name " <" address ">"))))
> +      (message-make-from name address))))
>  
>  (put 'notmuch-mua-new-mail 'notmuch-prefix-doc "... and prompt for sender")
>  (defun notmuch-mua-new-mail (&optional prompt-for-sender)
> -- 
> 2.7.4
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] emacs: make use of `message-make-from'
  2016-04-08 21:43 [PATCH] emacs: make use of `message-make-from' aeuii
                   ` (2 preceding siblings ...)
  2016-04-10 11:26 ` David Edmondson
@ 2016-04-10 23:57 ` David Bremner
  2016-04-11  6:29   ` aeuii
  3 siblings, 1 reply; 7+ messages in thread
From: David Bremner @ 2016-04-10 23:57 UTC (permalink / raw)
  To: aeuii, notmuch

aeuii@posteo.de writes:

> Please put my address in CC when replying.  Thanks!
>
> From 4b9ab261a0ea8a31065e310c5150f522be86d37b Mon Sep 17 00:00:00 2001
> From: stefan <aeuii@posteo.de>
> Date: Fri, 8 Apr 2016 22:47:06 +0200
> Subject: [PATCH] emacs: make use of `message-make-from'
>
> Will respect `mail-from-style'.

pushed to master, thanks for your contribution to notmuch

d

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] emacs: make use of `message-make-from'
  2016-04-10 23:57 ` David Bremner
@ 2016-04-11  6:29   ` aeuii
  0 siblings, 0 replies; 7+ messages in thread
From: aeuii @ 2016-04-11  6:29 UTC (permalink / raw)
  To: David Bremner, notmuch

2016-04-11 01:57 +0200, David Bremner <david@tethera.net>:

> aeuii@posteo.de writes:
>
>> Please put my address in CC when replying.  Thanks!
>>
>> From 4b9ab261a0ea8a31065e310c5150f522be86d37b Mon Sep 17 00:00:00 2001
>> From: stefan <aeuii@posteo.de>
>> Date: Fri, 8 Apr 2016 22:47:06 +0200
>> Subject: [PATCH] emacs: make use of `message-make-from'
>>
>> Will respect `mail-from-style'.
>
> pushed to master, thanks for your contribution to notmuch
>
> d

Thank you!

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2016-04-11  6:29 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-08 21:43 [PATCH] emacs: make use of `message-make-from' aeuii
2016-04-09 23:10 ` David Bremner
2016-04-10  7:24 ` Tomi Ollila
2016-04-10 11:26 ` David Edmondson
2016-04-10 10:05   ` Mark Walters
2016-04-10 23:57 ` David Bremner
2016-04-11  6:29   ` aeuii

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).