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

Re: [kernel.org users] [RFD] On deprecating "git-foo" for builtins

From
PWPerry Wagle <wagle@cs.indiana.edu>
Date
Aug 28, 2008, 22:33 UTC
Message-ID
<3DE083DB-ADFF-45E7-B3EB-A76985941271@cs.indiana.edu>
In-Reply-To
<20080828215907.GE27867@coredump.intra.peff.net>
On Aug 28, 2008, at 2:59 PM, Jeff King wrote:
Show 18 quoted lines
>> I now have to TEST to find those crazy backwards-incompatibility bugs
>> before I can upgrade us to 1.6.0.  To test, I have to try to  
>> imagine what
>> I and others were assuming about git.  And this episode means that  
>> I can't
>> make any assumptions about the sanity of any changes since March,  
>> which is
>> the version I'm thinking of upgrading.
>>
>> But note that THIS upward compatibility bug has been declared to  
>> not be a
>> bug.  Will any others receive the same stamp?
>>
>> So please put on your engineer hat, and stop talking about "specious
>> claims" and hurting feelings.
>
> My engineer hat _is_ on. Here is the logic that led to my use of the
> phrase "specious claims":
Cool.  Thanks!  (seriously)
>  - you are claiming that there are backwards incompatibility changes
>    lurking in git (or at least that is what I believe you to mean)

I'm saying that I don't know and will have to do complete exhaustive testing to be sure (my faith in git has been severely shaken). I already tested every step for months, and to do it "right", I have to do that all over again. I don't have the time, so I have to do severe approximations. One of the fixes is to see if I can get people to stop making frivolous changes: "ooo! we have to rename everything in the API because lists, I mean hashtables, with 143 entries in them are offensive!". If there was more reasoning than that, it was not displayed in this thread.

>  - there has been _one_ such problem, and the person responsible for
>    vetting such changes has solicited suggestions for doing better in
>    the future. I don't think that is indicative of a pattern of such
>    changes.

Ok. My suggestion is that it shouldn't have been done in the first place, and we should now revert. But others are saying over and over "its done! live with it!". I came in late. What did I miss in the last 6 months. Sounds like people have lots of practice with these water-over-the-dam's, surely this isn't the first time?

Show 7 quoted lines
>  - But let's say you have lost some faith in the git development
>    process due to _this_ bug. But let's look at the history of this
>    bug. It has been discussed several times for the past 2 years,  
> along
>    with a mention in the release notes several versions ago. It was  
> not
>    a surprise to anybody who has been developing git.

In March 2008, the sample git-hooks and git-web used git<DASH> commands. That was the last I looked at git until Tuesday of this week.

Show 5 quoted lines
>    So yes, maybe there _are_ other bugs just like it lurking. But
>    wouldn't it stand to reason that those bugs have _also_ been
>    discussed and mentioned in the release notes, or that the  
> developers
>    would know about them?

This is declared to not be a bug, even though it breaks existing scripts, even those published in the main branch of git itself.

Show 6 quoted lines
>    In other words, I can see you losing enough faith to say "wow, the
>    git developers don't communicate very well and I need to vet their
>    changes and notes more carefully". I don't think it is reasonable  
> to
>    say that there are likely to be other, totally unknown backwards
>    incompatible changes.

I'm going by the reasoning shown in this thread. Why not? I'm looking for a way not to have to do exhaustive testing on those scripts, so would love to hear it.

> So
>
>  1. I find your claim that such bugs exist to have little evidence to
>     back it up.

Induction. If it happened once, it probably happened more than once. This wasn't a show stopper problem. It wasn't broke, but it was "fixed" anyway.

Show 5 quoted lines
>  2. As an engineer, I have seen evidence of one problem (insufficient
>     communication) but not of another (introduction of
>     incompatibilities without on-list discussion). So I would choose  
> to
>     focus my resources on fixing the problem I have seen.

I don't know what I missed, and am not sure how to search for in in ten thousand messages or so since March. My style is to anticipate problems.

But I'll figure it out. Part of that figure out process is posting to this thread.

Show 7 quoted lines
>> Heck, I even got Linus himself to ask if us people were on drugs, and
>> I didn't take it personally.  At least I'm saying something that can
>> be disputed, and not ad hominem like Linus.  8)
>
> Sorry, but I'm not impressed by your getting Linus to yell at you.  
> It's
> like shooting fish in a barrel. :)

Yeah, well, that was supposed to be both a joke and to indicate that I'm not sitting here with steam coming out my ears.

Linus has his solution, that doesn't work for me. He hasn't listened to my several attempts to say why, and he's mad because he thinks I'm not listening. But I expect he's a busy guy with 10,000 emails a day to respond to, so them's the breaks.

Show 14 quoted lines
>> How to better notify them is to do it on a major release, like Git  
>> 2.0.
>> THEN, they expect upward compatibility to break.
>
> Now that _is_ a reasonable suggestion. This change _did_ wait until  
> the
> jump to 1.6.0, which we thought of as a major version jump (just as  
> 1.4
> to 1.5 introduced a few minor but documented changes). I don't think
> there is a plan for "git 2.0" short of serious incompatibilities in  
> the
> repo format (i.e., you can't use 2.x tools on 1.x repos and vice  
> versa).
> So perhaps our numbering should be more emphatic.

I think I hear you. Since git is young, I should expect incompatibilities with minor version changes, and not demand that they be held off for major version changes. That seems very plausible.

But I think I'll still remain wary because 1.6 introduced a nearly complete renaming of the API for what, in this thread anyway, completely silly reasons. If there are good reasons, I haven't seen them.

-- Perry
Previous: Jeff KingNext: Jeff King
Message 85 of 193 in “[RFD] On deprecating "git-foo" for builtins”
  1. Junio C HamanoAug 24, 2008
  2. Linus TorvaldsAug 24, 2008
  3. Imran M YousufAug 24, 2008
  4. Stefan RichterAug 24, 2008
  5. David WoodhouseAug 25, 2008
  6. Geert UytterhoevenAug 25, 2008
  7. Andi KleenAug 25, 2008
  8. A Large Angry SCMAug 26, 2008
  9. Ben CollinsAug 25, 2008
  10. Felipe ContrerasAug 25, 2008
  11. Johannes SchindelinAug 25, 2008
  12. Junio C HamanoAug 25, 2008
  13. David WoodhouseAug 26, 2008
  14. Jeff KingAug 26, 2008
  15. Kristian HøgsbergAug 26, 2008
  16. David WoodhouseAug 26, 2008
  17. Matthias KestenholzAug 26, 2008
  18. Petr BaudisAug 26, 2008
  19. Andi KleenAug 26, 2008
  20. Jeff KingAug 26, 2008
  21. Linus TorvaldsAug 26, 2008
  22. Andi KleenAug 26, 2008
  23. Ulrich WindlAug 27, 2008
  24. Petr BaudisAug 26, 2008
  25. bash completion: Hide more plumbing commandsPetr Baudis, Aug 26, 2008
  26. Shawn O. PearceAug 26, 2008
  27. Jakub NarebskiAug 26, 2008
  28. Junio C HamanoAug 26, 2008
  29. Shawn O. PearceAug 26, 2008
  30. Daniel BarkalowAug 26, 2008
  31. Shawn O. PearceAug 26, 2008
  32. Daniel BarkalowAug 26, 2008
  33. Petr BaudisSep 3, 2008
  34. Petr BaudisSep 3, 2008
  35. Jakub NarebskiAug 26, 2008
  36. Petr BaudisSep 3, 2008
  37. Junio C HamanoSep 4, 2008
  38. Matthieu MoyAug 26, 2008
  39. Karl HasselströmAug 27, 2008
  40. Shawn O. PearceAug 26, 2008
  41. Jeff KingAug 26, 2008
  42. Nguyen Thai Ngoc DuyAug 26, 2008
  43. Willy TarreauAug 26, 2008
  44. Jeff GarzikAug 27, 2008
  45. Jeff KingAug 27, 2008
  46. Jeff GarzikAug 27, 2008
  47. Jeff KingAug 27, 2008
  48. Matthew WilcoxAug 27, 2008
  49. Adrian BunkAug 27, 2008
  50. Jeff KingAug 27, 2008
  51. Adrian BunkAug 27, 2008
  52. Linus TorvaldsAug 27, 2008
  53. Jeff GarzikAug 27, 2008
  54. Ingo MolnarAug 28, 2008
  55. git-show vs git-log (or: git show vs git log)Dominik Brodowski, Aug 28, 2008
  56. Alex RiesenAug 28, 2008
  57. Mike HommeyAug 28, 2008
  58. Linus TorvaldsAug 27, 2008
  59. Ulrich WindlAug 27, 2008
  60. H. Peter AnvinAug 27, 2008
  61. Matthew WilcoxAug 27, 2008
  62. Perry WagleAug 27, 2008
  63. Jeff KingAug 27, 2008
  64. Perry WagleAug 27, 2008
  65. H. Peter AnvinAug 27, 2008
  66. Steven RostedtAug 27, 2008
  67. Junio C HamanoAug 27, 2008
  68. Perry WagleAug 27, 2008
  69. Perry WagleAug 28, 2008
  70. Petr BaudisAug 28, 2008
  71. Perry WagleAug 28, 2008
  72. David WoodhouseAug 28, 2008
  73. Perry WagleAug 28, 2008
  74. Petr BaudisAug 28, 2008
  75. Linus TorvaldsAug 28, 2008
  76. Perry WagleAug 28, 2008
  77. Teemu LikonenAug 28, 2008
  78. Perry WagleAug 28, 2008
  79. Petr BaudisAug 28, 2008
  80. Perry WagleAug 28, 2008
  81. Jeff KingAug 28, 2008
  82. Perry WagleAug 28, 2008
  83. Petr BaudisAug 28, 2008
  84. Jeff KingAug 28, 2008
  85. Perry WagleAug 28, 2008
  86. Jeff KingAug 28, 2008
  87. Perry WagleAug 28, 2008
  88. Jeff KingAug 28, 2008
  89. Junio C HamanoAug 28, 2008
  90. Perry WagleAug 28, 2008
  91. Petr BaudisAug 28, 2008
  92. git-* in test scripts (was On deprecating "git-foo" for builtins)Jeff King, Aug 28, 2008
  93. Junio C HamanoAug 29, 2008
  94. Jeff KingAug 29, 2008
  95. Andreas EricssonAug 29, 2008
  96. Matthieu MoyAug 29, 2008
  97. Andreas EricssonAug 29, 2008
  98. Matthias KestenholzAug 29, 2008
  99. Matthieu MoyAug 29, 2008
  100. Perry WagleAug 28, 2008
  101. Aidan Van DykAug 29, 2008
  102. Felipe ContrerasAug 29, 2008
  103. Aidan Van DykAug 29, 2008
  104. Felipe ContrerasAug 29, 2008
  105. Aidan Van DykAug 29, 2008
  106. Andreas EricssonAug 30, 2008
  107. Jakub NarebskiAug 28, 2008
  108. Wincent ColaiutaAug 29, 2008
  109. Steven RostedtAug 30, 2008
  110. Teemu LikonenAug 30, 2008
  111. Steven RostedtAug 30, 2008
  112. Jeff KingAug 26, 2008
  113. Teemu LikonenAug 26, 2008
  114. Kristian HøgsbergAug 26, 2008
  115. Jean DelvareAug 26, 2008
  116. Takashi IwaiAug 26, 2008
  117. Jean DelvareAug 26, 2008
  118. Andreas EricssonAug 27, 2008
  119. Jean DelvareAug 27, 2008
  120. Geert UytterhoevenAug 27, 2008
  121. H. Peter AnvinAug 26, 2008
  122. Jean DelvareAug 27, 2008
  123. Andreas EricssonAug 27, 2008
  124. Linus TorvaldsAug 26, 2008
  125. Bruce StephensAug 26, 2008
  126. Petr BaudisAug 26, 2008
  127. Bruce StephensAug 26, 2008
  128. Johannes SchindelinAug 28, 2008
  129. Takashi IwaiAug 26, 2008
  130. Dominik BrodowskiAug 26, 2008
  131. Linus TorvaldsAug 26, 2008
  132. Al ViroAug 26, 2008
  133. Linus TorvaldsAug 26, 2008
  134. Al ViroAug 26, 2008
  135. Teemu LikonenAug 26, 2008
  136. Johannes SchindelinAug 28, 2008
  137. Dominik BrodowskiAug 26, 2008
  138. Junio C HamanoAug 26, 2008
  139. Linus TorvaldsAug 26, 2008
  140. Perry WagleAug 26, 2008
  141. Steven RostedtAug 27, 2008
  142. Russell KingAug 27, 2008
  143. Stefan RichterAug 27, 2008
  144. Russell KingAug 28, 2008
  145. Junio C HamanoAug 28, 2008
  146. Matthew WilcoxAug 28, 2008
  147. Petr BaudisAug 28, 2008
  148. Stefan RichterAug 28, 2008
  149. Perry WagleAug 28, 2008
  150. Steven RostedtAug 28, 2008
  151. A Large Angry SCMAug 27, 2008
  152. Krzysztof HalasaAug 27, 2008
  153. Junio C HamanoAug 26, 2008
  154. Jeff KingAug 26, 2008
  155. Jay SoffianAug 27, 2008
  156. H. Peter AnvinAug 27, 2008
  157. Perry WagleAug 26, 2008
  158. Nicolas PitreAug 26, 2008
  159. Johannes SchindelinAug 28, 2008
  160. Matthew WilcoxAug 27, 2008
  161. Russell KingAug 27, 2008
  162. Johannes SchindelinAug 28, 2008
  163. Matthew WilcoxAug 28, 2008
  164. Johannes SchindelinAug 28, 2008
  165. Matthew WilcoxAug 28, 2008
  166. Junio C HamanoAug 27, 2008
  167. Felipe ContrerasAug 28, 2008
  168. Jeff GarzikAug 28, 2008
  169. David WoodhouseAug 28, 2008
  170. Junio C HamanoAug 28, 2008
  171. David WoodhouseAug 28, 2008
  172. Felipe ContrerasAug 28, 2008
  173. Al ViroAug 28, 2008
  174. Felipe ContrerasAug 28, 2008
  175. Felipe ContrerasAug 28, 2008
  176. Paolo CiarrocchiAug 28, 2008
  177. Linus TorvaldsAug 28, 2008
  178. Perry WagleAug 28, 2008
  179. Jakub NarebskiAug 28, 2008
  180. Perry WagleAug 28, 2008
  181. Jeff KingAug 28, 2008
  182. Perry WagleAug 28, 2008
  183. Felipe ContrerasAug 29, 2008
  184. Nicolas PitreAug 28, 2008
  185. Nicolas PitreAug 28, 2008
  186. Linus TorvaldsAug 28, 2008
  187. A Large Angry SCMAug 26, 2008
  188. Jean DelvareAug 26, 2008
  189. A Large Angry SCMAug 26, 2008
  190. Stefan RichterAug 26, 2008
  191. Steven RostedtAug 26, 2008
  192. Shawn O. PearceAug 26, 2008
  193. Jeff KingAug 26, 2008

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.