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

Re: Thunderbird and patches (was Re: [PATCH v2] Enable setting attach as the default in .gitconfig for git-format-patch.)

From
BBBen Bucksch <ben.bucksch.news@beonex.com>
Date
Feb 9, 2009, 19:51 UTC
Message-ID
<49908927.2030408@beonex.com>
In-Reply-To
<r57oMSnamWCJ1G1-r98G6mYQjX0YNpQz1OzEBHSrDifmBU_ir2P77A@cipher.nrlssc.navy.mil>
On 09.02.2009 20:30, Brandon Casey wrote:
> Did you try it with the message I sent titled
> '[PATCH] example patch corrupted by thunderbird'?

I tried now, and none of the editors/viewers I tried are displaying anything that would come close to readable to me, even if you count clearly marked hex character codes as readable. I tried less, e3 and kwrite. Therefore, a) I can't verify whether the result is correct or not b) I declare this an edge case.

> When I save with Thunderbird, part of what was on line 5 is now
> on another line and the control-M is missing.
> Nope, '.eml' extension.
>

That would be a bug. If you save as .eml, it should save exactly what's in your IMAP mailbox or what View as source | File | Save... saves. If it doesn't do that, it's a bug.

The View Source workaround may be inconvenient, but is a workaround for such a strong edgecase, until this bug is fixed. Don't hope for it, though, because TB is working on completely different things, like a message database.

You're welcome to file a bug, but please without political statements or broad generic demands. TB is geared towards comfortable writing and reading of human language text.

---

Apart from that, I can only recommend that you re-consider sending patches as inline attachments (Content-Disposition: inline, which is an official Internet Standard since many years), which is IMHO correctly reflecting reality, and fixing the software which can't deal with *that*, including inline display and quoting.

Ben
Previous: Brandon CaseyNext: Ben Bucksch
Message 23 of 44 in “Enable setting attach as the default in .gitconfig for git-format-patch.”
  1. Enable setting attach as the default in .gitconfig for git-format-patch.Jeremy White, Feb 8, 2009
  2. Boyd Stephen Smith Jr.Feb 8, 2009
  3. Jeremy WhiteFeb 8, 2009
  4. Johannes SchindelinFeb 8, 2009
  5. Michael J GruberFeb 9, 2009
  6. Ben BuckschFeb 9, 2009
  7. Johannes SchindelinFeb 9, 2009
  8. Michael J GruberFeb 9, 2009
  9. Boyd Stephen Smith Jr.Feb 9, 2009
  10. Ben BuckschFeb 9, 2009
  11. Johannes SchindelinFeb 9, 2009
  12. Thunderbird and patches (was Re: [PATCH v2] Enable setting attach as the default in .gitconfig for git-format-patch.)Jeremy White, Feb 9, 2009
  13. Ben BuckschFeb 9, 2009
  14. Jeremy WhiteFeb 9, 2009
  15. Michael J GruberFeb 9, 2009
  16. Ben BuckschFeb 9, 2009
  17. Jeremy WhiteFeb 9, 2009
  18. Brandon CaseyFeb 9, 2009
  19. Ben BuckschFeb 9, 2009
  20. Johannes SchindelinFeb 9, 2009
  21. Ben BuckschFeb 9, 2009
  22. Brandon CaseyFeb 9, 2009
  23. Ben BuckschFeb 9, 2009
  24. Ben BuckschFeb 9, 2009
  25. Brandon CaseyFeb 9, 2009
  26. Brandon CaseyFeb 9, 2009
  27. Ben BuckschFeb 9, 2009
  28. Brandon CaseyFeb 9, 2009
  29. Ben BuckschFeb 9, 2009
  30. Brandon CaseyFeb 9, 2009
  31. Ben BuckschFeb 9, 2009
  32. Brandon CaseyFeb 10, 2009
  33. Brandon CaseyFeb 9, 2009
  34. Johannes SchindelinFeb 9, 2009
  35. Michael J GruberFeb 10, 2009
  36. Miles BaderFeb 12, 2009
  37. Michael J GruberFeb 12, 2009
  38. Brandon CaseyFeb 9, 2009
  39. example patch corrupted by thunderbirdBrandon Casey, Feb 9, 2009
  40. Michael J GruberFeb 9, 2009
  41. Ben BuckschFeb 9, 2009
  42. Jeremy WhiteFeb 9, 2009
  43. Ben BuckschFeb 9, 2009
  44. Junio C HamanoFeb 10, 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.