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
Junio C Hamano <gitster@pobox.com>
Date
Feb 13, 2009, 04:19 UTC
Message-ID
<7viqnfezo5.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<499446D0.90602@codeweavers.com>

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.

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?

Previous: Jeremy WhiteNext: Michael J Gruber
Message 2 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.