git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [Virtual PATCH] Add an option to wrap a patch in <pre> in git-imap-send which ironically results in a cleaner patch from Thunderbird.

From
Michael J Gruber <git@drmicha.warpmail.net>
Date
Feb 13, 2009, 11:24 UTC
Message-ID
<49955860.80504@drmicha.warpmail.net>
In-Reply-To
<7viqnfezo5.fsf@gitster.siamese.dyndns.org>
Junio C Hamano venit, vidit, dixit 13.02.2009 05:19:
Show 9 quoted lines
> I do not think of a reason, other than to trigger the workaround you
> mentioned in the documentation part of the patch, why any sane user would
> want to send a patch as HTML.  This configuration variable sounds more
> like "imap.forceThunderbirdToSendNonFlowedTextByExploitingItsBug" than
> "imap.html", in other words.
> 
> What worries me the most is if there is any guarantee that this bug you
> are exploiting to force it to send a patch in the common denominator
> format _will not be fixed_ in future versions of Thunderbird.
It's not a bug, it's a feature ;)

In fact it really is: preformatted text in HTML (<pre>) is by definition left alone. Now, when you are about to send an HTML mail TB asks you what to do (or takes a choice from preferences/addressbook): send as HTML, as text or both.

The fact that the current HTML->text converter respects preformated text without reflowing (and without f-f, which would allow reflowing on the receiving side) is a feature. TB trys to represent the HTML in text form as closely as possible.

> I see your patch deals only with ampersand, less-than, greater-than and
> dquot.  Do you know if this is enough, or would letters outside US-ASCII
> need to be expressed in ampersand-hash "character reference" notation?

According to Ben of Mozilla fame this is enough for special characters. I don't know about UTF-8, though. Usually, TB recognizes the proper encoding.

Michael J Gruber
Previous: Junio C HamanoNext: Jeremy White
Message 3 of 6 in “Add an option to wrap a patch in <pre> in git-imap-send which ironically results in a cleaner patch from Thunderbird.”
  1. Add an option to wrap a patch in <pre> in git-imap-send which ironically results in a cleaner patch from Thunderbird.Jeremy White, Feb 12, 2009
  2. Junio C HamanoFeb 13, 2009
  3. Michael J GruberFeb 13, 2009
  4. Jeremy WhiteFeb 13, 2009
  5. Ben BuckschFeb 13, 2009
  6. Junio C HamanoFeb 13, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.