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

Re: [PATCH/POLL] git-format-patch: the default suffix is now .patch, not .txt

From
Alex Riesen <raa.lkml@gmail.com>
Date
Jan 18, 2007, 14:41 UTC
Message-ID
<81b0412b0701180641v55987657t331d6a1868dabee0@mail.gmail.com>
In-Reply-To
<45AF7FE8.5060003@op5.se>
On 1/18/07, Andreas Ericsson <ae@op5.se> wrote:
Show 15 quoted lines
> Alex Riesen wrote:
> > On 1/18/07, Josh Boyer <jwboyer@gmail.com> wrote:
> >> > As much as like Unix and everything related, I think it is
> >> > not reasonable to try to change the majority. Not unless
> >> > we have something earth-shattering. Well, git is, but
> >> > 0001-fix....patch in email attachment probably not.
> >>
> >> I would venture to say that the _majority_ of git users are not using
> >> Windows.
> >
> > The _real_ majority of the programmers desperately need a better
> > VCS than CVS, SVN, Perforce, SourceSafe, ClearCase, etc.
>
> They're free to use git ofcourse, provided they install cygwin or help
> migrate it to run natively on windows. I don't think anyone would cry if
My example had cygwin installed for him. Is it too unusual?
> a competent cross-platform programmer stepped up and started submitting
> patches to get git working on windows without having to resort to the
> cygwin emulation layer.

don't think so. I _would_ cry seeing how fork(2) gets ported to Windows, and you will, probably... after seeing how it is done in cygwin.

> The thing is, no-one's getting paid for it, so until someone *does* step
> up, it won't happen, as 95% of the *current* git users are still running
> what we on this list will indefinitely refer to as "a sane OS".

95% of the current users probably is not even 1% of all programmers who would gladly use it and maybe less than a fraction of percent of the ones who need it.

Show 5 quoted lines
> > People often understand "funny ways" the others may have. They
> > don't like been told they are wrong or stupid (especially when they
> > actually are stupid).
>
> I still don't see the problem. When he understands (and uses) git, the

he does not use git. He knows what the patch is (now I have explained it to him), I suppose he would still be mildly surprised seeing a file.patch, but he'll probably recover.

Show 5 quoted lines
> name and look of the patch will become blindingly clear to him, and then
> it doesn't matter if it's called .txt or .patch. He might even have some
> tool by then that displays patches color-coded and what-not (there are a
> plethora of such tools for Windows already, most of which register
> .patch and .diff as file-types they handle).

That wont happen until windows registers it for them. And that, again, wont ever happen. Every user will register .patch to notepad, visual studio or ultraedit or something else for himself.

> Otoh, *until* he uses git, the change doesn't affect him,

actually, patch works for git patches, so format-patch produces files useful output for anyone.

BTW, Junio, how about making the _default_ settable at compile time? It'd be reasonable to allow local installations choose to default to what they find the most paranoid?

> so why bother catering for his needs?
I don't know. It's kind of good style lately.
Previous: Johannes SchindelinNext: Johannes Schindelin
Message 35 of 59 in “[RFC] Add a suffix option to git-format-patch”
  1. Josh BoyerJan 17, 2007
  2. Johannes SchindelinJan 17, 2007
  3. Josh BoyerJan 17, 2007
  4. Horst H. von BrandJan 17, 2007
  5. Introduce 'git-format-patch --suffix=patch'Junio C Hamano, Jan 17, 2007
  6. Andy WhitcroftJan 17, 2007
  7. Junio C HamanoJan 17, 2007
  8. Brian GernhardtJan 17, 2007
  9. Junio C HamanoJan 17, 2007
  10. Brian GernhardtJan 17, 2007
  11. Make format-patch --suffix="" not add any suffixBrian Gernhardt, Jan 17, 2007
  12. Johannes SchindelinJan 18, 2007
  13. David KågedalJan 17, 2007
  14. Andreas EricssonJan 17, 2007
  15. Johannes SchindelinJan 17, 2007
  16. Junio C HamanoJan 17, 2007
  17. David KågedalJan 17, 2007
  18. Josh BoyerJan 17, 2007
  19. Josh BoyerJan 17, 2007
  20. git-format-patch: the default suffix is now .patch, not .txtJunio C Hamano, Jan 18, 2007
  21. Johannes SchindelinJan 18, 2007
  22. Alex RiesenJan 18, 2007
  23. Shawn O. PearceJan 18, 2007
  24. Alex RiesenJan 18, 2007
  25. Junio C HamanoJan 18, 2007
  26. Alex RiesenJan 18, 2007
  27. Junio C HamanoJan 18, 2007
  28. Alex RiesenJan 18, 2007
  29. Josh BoyerJan 18, 2007
  30. Johannes SchindelinJan 18, 2007
  31. Alex RiesenJan 18, 2007
  32. Alex RiesenJan 18, 2007
  33. Andreas EricssonJan 18, 2007
  34. Johannes SchindelinJan 18, 2007
  35. Alex RiesenJan 18, 2007
  36. Johannes SchindelinJan 18, 2007
  37. Alex RiesenJan 18, 2007
  38. Johannes SchindelinJan 18, 2007
  39. Alex RiesenJan 18, 2007
  40. Josh BoyerJan 18, 2007
  41. Johannes SchindelinJan 18, 2007
  42. Josh BoyerJan 18, 2007
  43. Shawn O. PearceJan 18, 2007
  44. Alex RiesenJan 18, 2007
  45. Steven GrimmJan 18, 2007
  46. Johannes SchindelinJan 18, 2007
  47. Johannes SixtJan 18, 2007
  48. Jakub NarebskiJan 19, 2007
  49. Shawn O. PearceJan 18, 2007
  50. Alex RiesenJan 18, 2007
  51. Andreas EricssonJan 18, 2007
  52. Shawn O. PearceJan 18, 2007
  53. Andreas EricssonJan 18, 2007
  54. Martin LanghoffJan 18, 2007
  55. Martin LanghoffJan 18, 2007
  56. Andreas EricssonJan 18, 2007
  57. Lukas SandströmJan 18, 2007
  58. Brian GernhardtJan 18, 2007
  59. Alexandre JulliardJan 18, 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.