[BUG/PATCH] emacs: Fix header problem in reply for emacs 23.2
Jani Nikula
jani at nikula.org
Thu Mar 29 07:13:05 PDT 2012
On Wed, 28 Mar 2012 08:02:18 -0600, Adam Wolfe Gordon <awg+notmuch at xvx.ca> wrote:
> The new reply code used strings instead of symbols for header names,
> which message-mail is OK with on emacs 23.3, but not 23.2. The symptom
> is that on 23.2 (and presumably on earlier versions) the reply message
> would end up with two of some headers.
>
> This fixes the problem by converting the header names to symbols of
> the type message.el usually expects before passing the headers to
> message-mail.
The patch depends on some set of the other reply fixes (*), so I applied
this on top of them. All tests pass with emacs 23.2.
Unfortunately, the emacs UI fails with "Symbol's function definition is
void: remove-if" when trying to reply. After a manual (require 'cl) the
UI seems to otherwise work as expected. This is probably unrelated to
this particular patch, but why is it a (eval-when-compile (require 'cl))
and not (require 'cl) in notmuch-mua.el?
BR,
Jani.
(*)
id:"1332941635-21019-2-git-send-email-awg+notmuch at xvx.ca"
id:"1332941635-21019-3-git-send-email-awg+notmuch at xvx.ca"
id:"1332996818-15700-1-git-send-email-awg+notmuch at xvx.ca"
> ---
> emacs/notmuch-lib.el | 5 ++++-
> emacs/notmuch-mua.el | 10 +++++-----
> 2 files changed, 9 insertions(+), 6 deletions(-)
>
> diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el
> index c146748..af46611 100644
> --- a/emacs/notmuch-lib.el
> +++ b/emacs/notmuch-lib.el
> @@ -232,9 +232,12 @@ the given type."
> (or (plist-get part :content)
> (notmuch-get-bodypart-internal (concat "id:" (plist-get msg :id)) nth process-crypto)))
>
> +;; Converts a plist of headers to an alist of headers. The input plist should
> +;; have symbols of the form :Header as keys, and the resulting alist will have
> +;; symbols of the form 'Header as keys.
> (defun notmuch-plist-to-alist (plist)
> (loop for (key value . rest) on plist by #'cddr
> - collect (cons (substring (symbol-name key) 1) value)))
> + collect (cons (intern (substring (symbol-name key) 1)) value)))
>
> ;; Compatibility functions for versions of emacs before emacs 23.
> ;;
> diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el
> index 9805d79..24918d3 100644
> --- a/emacs/notmuch-mua.el
> +++ b/emacs/notmuch-mua.el
> @@ -185,11 +185,11 @@ OTHER-ARGS are passed through to `message-mail'."
> (when notmuch-mua-user-agent-function
> (let ((user-agent (funcall notmuch-mua-user-agent-function)))
> (when (not (string= "" user-agent))
> - (push (cons "User-Agent" user-agent) other-headers))))
> + (push (cons 'User-Agent user-agent) other-headers))))
>
> - (unless (assoc "From" other-headers)
> - (push (cons "From" (concat
> - (notmuch-user-name) " <" (notmuch-user-primary-email) ">")) other-headers))
> + (unless (assq 'From other-headers)
> + (push (cons 'From (concat
> + (notmuch-user-name) " <" (notmuch-user-primary-email) ">")) other-headers))
>
> (apply #'message-mail to subject other-headers other-args)
> (message-sort-headers)
> @@ -250,7 +250,7 @@ the From: address first."
> (interactive "P")
> (let ((other-headers
> (when (or prompt-for-sender notmuch-always-prompt-for-sender)
> - (list (cons "From" (notmuch-mua-prompt-for-sender))))))
> + (list (cons 'From (notmuch-mua-prompt-for-sender))))))
> (notmuch-mua-mail nil nil other-headers)))
>
> (defun notmuch-mua-new-forward-message (&optional prompt-for-sender)
> --
> 1.7.5.4
>
More information about the notmuch
mailing list