passing body to mutt mail client

[Edwin Grubbs]

Here is a patch for adding support for passing the email body to mutt.

--- mail_client.py 2009-06-05 19:12:55.000000000 -0500
+++ /tmp/mail_client.py 2009-06-05 19:12:44.000000000 -0500
@@ -253,12 +253,12 @@
                               help=Evolution.__doc__)

-class Mutt(ExternalMailClient):
+class Mutt(BodyExternalMailClient):
     """Mutt mail client."""

     _client_commands = ['mutt']

- def _get_compose_commandline(self, to, subject, attach_path):
+ def _get_compose_commandline(self, to, subject, attach_path, body=None):
         """See ExternalMailClient._get_compose_commandline"""
         message_options = []
         if subject is not None:
@@ -266,6 +266,14 @@
         if attach_path is not None:
             message_options.extend(['-a',
                 self._encode_path(attach_path, 'attachment')])
+ if body is not None:
+ fd, temp_file = tempfile.mkstemp(prefix="mutt-body",
+ suffix=".txt")
+ try:
+ os.write(fd, body)
+ finally:
+ os.close(fd)
+ message_options.extend(['-i', temp_file])
         if to is not None:
             message_options.extend(['--', self._encode_safe(to)])
         return message_options

[Martin Pool (mbp)]

Thanks for the patch; please make a merge proposal for it.

Please also use a NamedTemporaryFile object not os.write unless
there's a compelling reason.

--
Martin <http://launchpad.net/~mbp/>

[Edwin Grubbs (edwin-grubbs)]

I submitted a merge proposal. BTW, the following wiki page says to use bundle buggy.

http://bazaar-vcs.org/BzrGivingBack

[Martin Pool (mbp)]

I'll merge this