unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH v2] emacs: Improve notmuch-message-mode initialization
@ 2015-12-21  9:41 Michal Sojka
  2015-12-21  9:41 ` Michal Sojka
  0 siblings, 1 reply; 7+ messages in thread
From: Michal Sojka @ 2015-12-21  9:41 UTC (permalink / raw)
  To: notmuch

Hi,

this is v2 of id:1446573161-28068-1-git-send-email-sojkam1@fel.cvut.cz.
As suggested by David, instead of using advices, it reimplements the
needed pieces of message-mail in notmuch-specific way.

Michal Sojka (1):
  emacs: Improve notmuch-message-mode initialization

 emacs/notmuch-mua.el | 49 +++++++++++++++++++++++++++++++++++++++++++------
 1 file changed, 43 insertions(+), 6 deletions(-)

-- 
2.6.4

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

* [PATCH v2] emacs: Improve notmuch-message-mode initialization
  2015-12-21  9:41 [PATCH v2] emacs: Improve notmuch-message-mode initialization Michal Sojka
@ 2015-12-21  9:41 ` Michal Sojka
  2015-12-30 15:22   ` David Bremner
  2016-01-01 15:02   ` David Bremner
  0 siblings, 2 replies; 7+ messages in thread
From: Michal Sojka @ 2015-12-21  9:41 UTC (permalink / raw)
  To: notmuch

Recent addition of notmuch-message-mode introduced several problems:

1. When message-setup-hook is used to set buffer local variables,
   these settings are not effective, because all buffer local
   variables are immediately erased by notmuch-message-mode
   initialization.

2. message-mode-hook gets invoked twice - first when message-mail
   invokes message-mode and second when notmuch-mua-mail invokes
   notmuch-message-mode.

This commit fixes these problems by replacing a call to message-mail
with notmuch-specific code that is (hopefully) equivalent to
message-mail functionality before introduction of
notmuch-message-mode.

We first initialize notmuch-message-mode with
notmuch-mua-pop-to-buffer, which is a modified version of
message-pop-to-buffer and then call message-setup-1, which is the only
functionality of message-mail that is needed for notmuch.
---
 emacs/notmuch-mua.el | 49 +++++++++++++++++++++++++++++++++++++++++++------
 1 file changed, 43 insertions(+), 6 deletions(-)

diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el
index 466edd7..a66a306 100644
--- a/emacs/notmuch-mua.el
+++ b/emacs/notmuch-mua.el
@@ -278,10 +278,36 @@ Note that these functions use `mail-citation-hook' if that is non-nil."
 (define-key notmuch-message-mode-map (kbd "C-c C-c") #'notmuch-mua-send-and-exit)
 (define-key notmuch-message-mode-map (kbd "C-c C-s") #'notmuch-mua-send)
 
-(defun notmuch-mua-mail (&optional to subject other-headers &rest other-args)
-  "Invoke the notmuch mail composition window.
-
-OTHER-ARGS are passed through to `message-mail'."
+(defun notmuch-mua-pop-to-buffer (name)
+  "Pop to buffer NAME, and warn if it already exists and is
+modified. This function is notmuch addaptation of
+`message-pop-to-buffer'."
+  (let ((buffer (get-buffer name)))
+    (if (and buffer
+	     (buffer-name buffer))
+	(let ((window (get-buffer-window buffer 0)))
+	  (if window
+	      ;; Raise the frame already displaying the message buffer.
+	      (progn
+		(gnus-select-frame-set-input-focus (window-frame window))
+		(select-window window))
+	    (funcall (notmuch-mua-get-switch-function) buffer)
+	    (set-buffer buffer))
+	  (when (and (buffer-modified-p)
+		     (not (prog1
+			      (y-or-n-p
+			       "Message already being composed; erase? ")
+			    (message nil))))
+	    (error "Message being composed")))
+      (funcall (notmuch-mua-get-switch-function) name)
+      (set-buffer name))
+    (erase-buffer)
+    (notmuch-message-mode)))
+
+(defun notmuch-mua-mail (&optional to subject other-headers continue
+				   switch-function yank-action send-actions
+				   return-action &rest ignored)
+  "Invoke the notmuch mail composition window."
   (interactive)
 
   (when notmuch-mua-user-agent-function
@@ -293,8 +319,19 @@ OTHER-ARGS are passed through to `message-mail'."
     (push (cons 'From (concat
 		       (notmuch-user-name) " <" (notmuch-user-primary-email) ">")) other-headers))
 
-  (apply #'message-mail to subject other-headers other-args)
-  (notmuch-message-mode)
+  (notmuch-mua-pop-to-buffer (message-buffer-name "mail" to))
+  (message-setup-1
+   ;; The following sexp is copied from `message-mail'
+   (nconc
+    `((To . ,(or to "")) (Subject . ,(or subject "")))
+    ;; C-h f compose-mail says that headers should be specified as
+    ;; (string . value); however all the rest of message expects
+    ;; headers to be symbols, not strings (eg message-header-format-alist).
+    ;; http://lists.gnu.org/archive/html/emacs-devel/2011-01/msg00337.html
+    ;; We need to convert any string input, eg from rmail-start-mail.
+    (dolist (h other-headers other-headers)
+      (if (stringp (car h)) (setcar h (intern (capitalize (car h)))))))
+   yank-action send-actions return-action)
   (notmuch-fcc-header-setup)
   (message-sort-headers)
   (message-hide-headers)
-- 
2.6.4

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

* Re: [PATCH v2] emacs: Improve notmuch-message-mode initialization
  2015-12-21  9:41 ` Michal Sojka
@ 2015-12-30 15:22   ` David Bremner
  2016-01-01 15:02   ` David Bremner
  1 sibling, 0 replies; 7+ messages in thread
From: David Bremner @ 2015-12-30 15:22 UTC (permalink / raw)
  To: Michal Sojka, notmuch

Michal Sojka <sojkam1@fel.cvut.cz> writes:

> Recent addition of notmuch-message-mode introduced several problems:
>
> 1. When message-setup-hook is used to set buffer local variables,
>    these settings are not effective, because all buffer local
>    variables are immediately erased by notmuch-message-mode
>    initialization.
>
> 2. message-mode-hook gets invoked twice - first when message-mail
>    invokes message-mode and second when notmuch-mua-mail invokes
>    notmuch-message-mode.
>
> This commit fixes these problems by replacing a call to message-mail
> with notmuch-specific code that is (hopefully) equivalent to
> message-mail functionality before introduction of
> notmuch-message-mode.
>
> We first initialize notmuch-message-mode with
> notmuch-mua-pop-to-buffer, which is a modified version of
> message-pop-to-buffer and then call message-setup-1, which is the only
> functionality of message-mail that is needed for notmuch.

This doesn't break anything for me, and looks fairly sane. I'm not sure
if the indirectrion of notmuch-mua-get-switch-function is still needed,
but that can wait for an optional followup patch. 

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

* Re: [PATCH v2] emacs: Improve notmuch-message-mode initialization
  2015-12-21  9:41 ` Michal Sojka
  2015-12-30 15:22   ` David Bremner
@ 2016-01-01 15:02   ` David Bremner
  2016-01-01 16:37     ` David Bremner
  1 sibling, 1 reply; 7+ messages in thread
From: David Bremner @ 2016-01-01 15:02 UTC (permalink / raw)
  To: Michal Sojka, notmuch

Michal Sojka <sojkam1@fel.cvut.cz> writes:

> This commit fixes these problems by replacing a call to message-mail
> with notmuch-specific code that is (hopefully) equivalent to
> message-mail functionality before introduction of
> notmuch-message-mode.

pushed to master

d

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

* Re: [PATCH v2] emacs: Improve notmuch-message-mode initialization
  2016-01-01 15:02   ` David Bremner
@ 2016-01-01 16:37     ` David Bremner
  2016-01-02 16:50       ` Michal Sojka
  2016-03-23 10:32       ` David Bremner
  0 siblings, 2 replies; 7+ messages in thread
From: David Bremner @ 2016-01-01 16:37 UTC (permalink / raw)
  To: Michal Sojka, notmuch

David Bremner <david@tethera.net> writes:

> Michal Sojka <sojkam1@fel.cvut.cz> writes:
>
>> This commit fixes these problems by replacing a call to message-mail
>> with notmuch-specific code that is (hopefully) equivalent to
>> message-mail functionality before introduction of
>> notmuch-message-mode.
>
> pushed to master

It seems to have broken something with emacs23

   https://travis-ci.org/notmuch/notmuch/builds/99715812

I just verified the same problems (more or less) exist in debian
oldstable.

d

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

* Re: [PATCH v2] emacs: Improve notmuch-message-mode initialization
  2016-01-01 16:37     ` David Bremner
@ 2016-01-02 16:50       ` Michal Sojka
  2016-03-23 10:32       ` David Bremner
  1 sibling, 0 replies; 7+ messages in thread
From: Michal Sojka @ 2016-01-02 16:50 UTC (permalink / raw)
  To: David Bremner, notmuch

On Fri, Jan 01 2016, David Bremner wrote:
> David Bremner <david@tethera.net> writes:
>
>> Michal Sojka <sojkam1@fel.cvut.cz> writes:
>>
>>> This commit fixes these problems by replacing a call to message-mail
>>> with notmuch-specific code that is (hopefully) equivalent to
>>> message-mail functionality before introduction of
>>> notmuch-message-mode.
>>
>> pushed to master
>
> It seems to have broken something with emacs23
>
>    https://travis-ci.org/notmuch/notmuch/builds/99715812
>
> I just verified the same problems (more or less) exist in debian
> oldstable.

A fixup patch was sent as
id:1451753265-26713-2-git-send-email-sojkam1@fel.cvut.cz.

-Michal

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

* Re: [PATCH v2] emacs: Improve notmuch-message-mode initialization
  2016-01-01 16:37     ` David Bremner
  2016-01-02 16:50       ` Michal Sojka
@ 2016-03-23 10:32       ` David Bremner
  1 sibling, 0 replies; 7+ messages in thread
From: David Bremner @ 2016-03-23 10:32 UTC (permalink / raw)
  To: Michal Sojka, notmuch

David Bremner <david@tethera.net> writes:

> David Bremner <david@tethera.net> writes:
>
>> Michal Sojka <sojkam1@fel.cvut.cz> writes:
>>
>>> This commit fixes these problems by replacing a call to message-mail
>>> with notmuch-specific code that is (hopefully) equivalent to
>>> message-mail functionality before introduction of
>>> notmuch-message-mode.
>>
>> pushed to master
>
> It seems to have broken something with emacs23
>
>    https://travis-ci.org/notmuch/notmuch/builds/99715812
>
> I just verified the same problems (more or less) exist in debian
> oldstable.

Marking this fixed, since Michal's patch was applied some time ago.

d

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

end of thread, other threads:[~2016-03-23 10:32 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-21  9:41 [PATCH v2] emacs: Improve notmuch-message-mode initialization Michal Sojka
2015-12-21  9:41 ` Michal Sojka
2015-12-30 15:22   ` David Bremner
2016-01-01 15:02   ` David Bremner
2016-01-01 16:37     ` David Bremner
2016-01-02 16:50       ` Michal Sojka
2016-03-23 10:32       ` David Bremner

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).