Re: RFC re Thunderbird + imap-send

From: Michael J Gruber <git@drmicha.warpmail.net>
Date: Feb 10, 2009, 11:06 UTC
Message-ID: <49915FB6.8010803@drmicha.warpmail.net>
In-Reply-To: <4990A4FF.6020404@codeweavers.com>

Jeremy White venit, vidit, dixit 09.02.2009 22:49:

Show 18 quoted lines

> So, MajorDomo knows that what I'm trying is sick and twisted; I seem to
> have run afoul of the taboos:
>   http://vger.kernel.org/majordomo-taboos.txt
> 
> (ixnay on the ention may of ontentcay ypetay tmlhay :-/).
> 
> So here is a pointer to a proposed patch to enable git-imap-send to work
> with Thunderbird, based on Ben's suggestion:
> 
> http://www.codeweavers.com/~jwhite/0001-Add-an-option-to-wrap-a-patch-in-pre-in-git-imap-s.patch
> 
> Essentially, with this patch, I add an html option to the imap-send
> server configuration block.  If turned on, we structure each patch as
> html, inside of pre blocks.  And, in fact, Thunderbird does
> automatically convert such an email into a plain/text, format=fixed
> email, that seems to diff cleanly with a regular git-format-patch result.
> 
> Thoughts?  Tar and feathers?

Looks okay here. I leave the tarring and feathering to Dscho ;)

Trying this out showed that indeed git-imap-send needs much more love. The man page doesn't say much, doesn't describe all options. Most options can be specified in config only. You can't pass file arguments, only stdin. You can't even pass a recipient!

To be useful, imap-send should really obey the same config options and parameters as much as possible. In fact they should share some. imap-send and send-email should be usable interchangeably, without fiddling with format-patch options (which has to: and cc: where it doesn't really belong). Uhm, I'm getting off-topic.

Back on topic: TB does the automatic conversion to plain text only when the send options are appropriate, or if you set the recipient to "text only".

Also: For a typical text mail in Drafts I see lines like

X-Mozilla-Draft-Info: internal/draft; vcard=0; receipt=0; uuencode=0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: quoted-printable

Any chance we could use that rather than invoke the HTML editor? We would need to shut off q-p and f-f *and* avoid automatic word wrap (or make sure patches don't have longer lines then TBs editor allows), the latter I don't know how to do.

Michael