From: Myles English <mylesenglish@gmail.com>
To: Glenn Morris <rgm@gnu.org>
Cc: 18173@debbugs.gnu.org
Subject: bug#18173: 24.3; message-sendmail-extra-arguments not applied
Date: Sat, 04 Oct 2014 22:25:12 +0100 [thread overview]
Message-ID: <87y4suae8p.fsf@gmail.com> (raw)
In-Reply-To: <go61fzios5.fsf@fencepost.gnu.org>
Glenn Morris writes:
> Sorry, I'm being dense, because I cannot see a bug here, or how your
> patch makes any difference to anything. Can you explain?
I can't reproduce this bug anymore (you can stop reading here). I'll
explain what I thought the bug was for posterity.
With sendmail-program set to /usr/bin/msmtp. Without the patch below,
msmtp finds its config file in the default location ~/.msmtprc . It is
possible specify a different config file on the command line using the
-C option. However, using this:
(setq message-sendmail-extra-arguments '("-C"
"/home/myles/.config/msmtp/msmtprc"))
did not result in msmtp finding its config file. The doc for
message-sendmail-extra-arguments says "Additional arguments to
`sendmail-program'". The 'bug' was that the extra argument should be
applied but it is not.
After applying the patch below, the config file is found in the
non-default location. The 'bug' is fixed because the extra argument is
applied.
I don't know why the patch works or much about elisp (I copied from
elsewhere in the source).
While debugging, message-sendmail-extra-arguments seemed to be something
other than a string, the patch below seamed to cause it to be string
consistently.
Myles
>> --- message.el
>> +++ message.el
>> @@ -4723,9 +4723,7 @@
>> (erase-buffer))))
>> (let* ((default-directory "/")
>> (coding-system-for-write message-send-coding-system)
>> - (cpr (apply
>> - 'call-process-region
>> - (append
>> + (args (append
>> (list (point-min) (point-max) sendmail-program
>> nil errbuf nil "-oi")
>> message-sendmail-extra-arguments
>> @@ -4745,7 +4743,8 @@
>> ;; For a resend, include the specific addresses.
>> (if resend-to-addresses
>> (list resend-to-addresses)
>> - '("-t"))))))
>> + '("-t"))))
>> + (cpr (apply 'call-process-region args)))
>> (unless (or (null cpr) (and (numberp cpr) (zerop cpr)))
>> (if errbuf (pop-to-buffer errbuf))
>> (error "Sending...failed with exit value %d" cpr)))
prev parent reply other threads:[~2014-10-04 21:25 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-02 14:13 bug#18173: 24.3; message-sendmail-extra-arguments not applied Myles English
2014-10-04 20:05 ` Glenn Morris
2014-10-04 21:25 ` Myles English [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87y4suae8p.fsf@gmail.com \
--to=mylesenglish@gmail.com \
--cc=18173@debbugs.gnu.org \
--cc=rgm@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.