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
Shawn O. Pearce <spearce@spearce.org>
Date
Jan 18, 2007, 15:42 UTC
Message-ID
<20070118154257.GC15428@spearce.org>
In-Reply-To
<81b0412b0701180540x15d20453s3dbc0c061fd06d50@mail.gmail.com>
Alex Riesen <raa.lkml@gmail.com> wrote:
> The _real_ majority of the programmers desperately need a better
> VCS than CVS, SVN, Perforce, SourceSafe, ClearCase, etc.
Yes.  But...

Yesterday I had a conversation with the software configuration management guy at my day-time-pays-the-bills organization. They are seriously looking at Perforce and ClearCase, as these are lightyears ahead of what we have already (PVCS Version Manager). They also have 1-800-my-vendor telephone numbers which you can call and scream at someone when the tool corrupts its internal database[*1*], or when you cannot figure out what the "Checkout" action in the context menu does[*2*].

However my fellow developers and I use Git. We export our changes out to PVCS Version Manager via an *ugly* Perl script that I would never actually wish on anyone (which is one reason why its not contributed as git-pvcsexport). Configuration management guy won't even look at Git's real strengths as it lacks the all-important 1-800-git-help[*3*] phone number.

Yesterday we spent 4 half-man-days (4 developers working together all morning) trying to get a configuration of random files from PVCS Version Manager which compiled. In Git this would have been as simple as merging the two topics that management wanted moved to the next level of testing. But since our organization doesn't actually use Git and multiple topics affected the same files, well, yea, it was a mess. Unfortunately calling 1-800-my-vendor yields a "don't do that" from our vendor and nothing more in support. I'm very glad we pay them for the privilege of having a 1-800-my-vendor phone number in our rolodex. Yesterday it cost us over $1600 in labor.

Show 8 quoted lines
> Yes. For me and you. One of my coworkers knows nothing about patches,
> but wants (and perfectly able to) review my code. He has usable brains
> and is able to figure out what "+" and "-" is (he has, by now). He hasn't
> even realized that it was an automatically generated information, as
> I sent a patch to him first time, thought it was just a funny way to
> document changes (and was surprised when I told him a patch can be
> applied automatically, even if the original file is not exactly the same).
> But he is a typical windows-trained programmer.

I work with a few people who would rather copy and paste their changed parts into an email, then color them with pretty colors like red, green, blue, black, orange in Outlook (and all in the same email too). These then get sent to me for code review.

Prior to us using Git it may make some sense, as we did not have a diff tool available. Now that everyone is using Git and working in isolated topic branches (and doing very well with that concept) I still get these emails. People seem to have an easy time grasping the idea that Git is tracking their changes and when combined with my git-pvcsexport (see above) it just Does The Right Thing(tm) later on. But they have a hard time grasping the idea that Git can export these as a diff, or that they could just push their topic branch to me so I can pop it open in gitk. *sigh*

[*1*] This has happened to me with Perforce more than once.  Not a
      happy thought.  Never with Git.
[*2*] Yes, really, some of our version control users have difficulty
      grasping a concept like "checkout".  They *definately* have
      an issue with Git's "checkout -b" concept.
[*3*] Already taken.  Oddly enough by a company that could almost
      be considered to be a competiter to my day-time-pays-the-bills
      organization.
 
-- 
Shawn.
Previous: Jakub NarebskiNext: Alex Riesen
Message 49 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.