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

Re: FEATURE REQUEST: git-format-path: Add option to encode patch content

From
Linus Torvalds <torvalds@linux-foundation.org>
Date
Apr 27, 2007, 23:41 UTC
Message-ID
<alpine.LFD.0.98.0704271636130.9964@woody.linux-foundation.org>
In-Reply-To
<87wszxo2b5.wl%cworth@cworth.org>
On Fri, 27 Apr 2007, Carl Worth wrote:
> 
> PS. A separate reason I like attachments is that I like the subject
> and message body to be the actual discussion of the patch, (and the
> commit message and patch content to come later).

And the reason I _hate_ attachements is that you cannot say "no, that one is wrong", and just skip/edit it.

Because once you start doing attachments, you're not going have just one patch per email. You're going to have people send you ten attachments in one email for patches 1-10.

And when patch 5 is crap, you're basically screwed. There is almost no support for replying to that _individually_ and saying "That one sucks". You press "reply", and you get all ten.

So the fundamental rule *must* be: one patch, one email. That's the starting point. Anything else is simply broken.

And once you accept that fundamental rule (and quite frankly, I don't care if others accept it or not - I'm personally 100% convinved that anythign else is a major mistake), suddenly attachments have no upsides at all, only downsides. They mean that when you send the patch top a mailing list, everybody sees it differently, and a *lot* of people will be unable to reply to it with proper quoting.

		Linus
Previous: Carl WorthNext: Linus Torvalds
Message 12 of 20 in “FEATURE REQUEST: git-format-path: Add option to encode patch content”
  1. Jari AaltoApr 27, 2007
  2. Johannes SchindelinApr 27, 2007
  3. Jari AaltoApr 27, 2007
  4. Alex RiesenApr 27, 2007
  5. Johannes SchindelinApr 28, 2007
  6. Junio C HamanoApr 27, 2007
  7. Robin H. JohnsonApr 27, 2007
  8. Carl WorthApr 27, 2007
  9. Junio C HamanoApr 27, 2007
  10. Linus TorvaldsApr 27, 2007
  11. Carl WorthApr 27, 2007
  12. Linus TorvaldsApr 27, 2007
  13. Linus TorvaldsApr 27, 2007
  14. Carl WorthApr 27, 2007
  15. Linus TorvaldsApr 28, 2007
  16. Junio C HamanoApr 28, 2007
  17. Carl WorthApr 30, 2007
  18. Junio C HamanoApr 28, 2007
  19. Carl WorthApr 30, 2007
  20. Junio C HamanoApr 30, 2007

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.