git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff King <peff@peff.net>
To: Perry Wagle <wagle@cs.indiana.edu>
Cc: Teemu Likonen <tlikonen@iki.fi>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Petr Baudis <pasky@suse.cz>, Kristian H??gsberg <krh@redhat.com>,
	Matthias Kestenholz <mk@spinlock.ch>,
	Steven Rostedt <rostedt@goodmis.org>,
	Ulrich Windl <ulrich.windl@rz.uni-regensburg.DE>,
	Andi Kleen <andi@firstfloor.org>,
	Johannes Schindelin <Johannes.Schindelin@gmx.de>,
	Junio C Hamano <gitster@pobox.com>,
	David Woodhouse <dwmw2@infradead.org>,
	git@vger.kernel.org
Subject: Re: [kernel.org users] [RFD] On deprecating "git-foo" for builtins
Date: Thu, 28 Aug 2008 19:04:01 -0400	[thread overview]
Message-ID: <20080828230401.GC29609@coredump.intra.peff.net> (raw)
In-Reply-To: <3DE083DB-ADFF-45E7-B3EB-A76985941271@cs.indiana.edu>

On Thu, Aug 28, 2008 at 03:33:26PM -0700, Perry Wagle wrote:

> 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?

No, it really is the first time.

> 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.

Yes, and some of the test scripts still have git-* in them. I think in
that respect, the git community has been very bad about eating our own
dog food.

>>    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.

I think the bug is not in the change, but in the deprecation process and
communication. But I think the definition of "bug" is vague enough to
simply be in the eye of the beholder.

> 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.

Ultimately you must be the judge of what and how much to test on your
systems. But if you are asking if there are other similar compatibility
bugs in 1.6.0, then my opinion, as somebody who follows the git list
quite closely and contributes some code, is that no, there are not.

>>  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.

Induction in this manner is not a very rigorous argument (we're being
engineers, right?). I gave several reasons already why the probability
is low that another such bug exists, mostly related to the lack of
indicators.

> 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.

Sure, I wouldn't want to go back and read all of the messages either.
But this was mentioned in the release notes, too. Why don't you start
with them?

> 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.

That isn't quite what I meant. What I meant was that our idea of a major
version number is the middle number. That is, the time to introduce a
few minor incompatibilities is when that number jumps (but they should
be documented for a significant time period ahead of time, which this
was).  I don't expect to jump to "git 2.0" basically ever. Just as I
don't really expect Linux 3.0 anytime soon. But I, of course, am not in
charge of such things, so you may take that with a grain of salt.

> 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.

I think the reasons have been mentioned a few times. Maybe you just
didn't think they were good.

-Peff

  reply	other threads:[~2008-08-28 23:05 UTC|newest]

Thread overview: 193+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-24  3:33 [RFD] On deprecating "git-foo" for builtins Junio C Hamano
2008-08-24  4:23 ` [kernel.org users] " Linus Torvalds
2008-08-24  4:54   ` Imran M Yousuf
2008-08-24  8:40 ` Stefan Richter
2008-08-25 11:49 ` David Woodhouse
2008-08-25 12:17   ` Geert Uytterhoeven
2008-08-25 12:43   ` Andi Kleen
2008-08-26  3:00     ` A Large Angry SCM
2008-08-25 13:22   ` Ben Collins
2008-08-25 14:38     ` Felipe Contreras
2008-08-25 18:19   ` Johannes Schindelin
2008-08-25 23:41     ` Junio C Hamano
2008-08-26 12:27       ` David Woodhouse
2008-08-26 14:57       ` Jeff King
2008-08-26 15:34         ` Kristian Høgsberg
2008-08-26 15:59           ` David Woodhouse
2008-08-26 16:04             ` Matthias Kestenholz
2008-08-26 16:25               ` Petr Baudis
2008-08-26 16:45                 ` Andi Kleen
2008-08-26 17:04                   ` Jeff King
2008-08-26 17:05                   ` Linus Torvalds
2008-08-26 17:27                     ` Andi Kleen
2008-08-27  6:04                       ` Ulrich Windl
2008-08-26 17:10                   ` Petr Baudis
2008-08-26 17:11                     ` [PATCH] bash completion: Hide more plumbing commands Petr Baudis
2008-08-26 17:24                       ` Shawn O. Pearce
2008-08-26 17:43                         ` Jakub Narebski
2008-08-26 18:25                         ` Junio C Hamano
2008-08-26 18:27                           ` Shawn O. Pearce
2008-08-26 19:04                           ` Daniel Barkalow
2008-08-26 19:07                             ` Shawn O. Pearce
2008-08-26 19:23                               ` Daniel Barkalow
2008-09-03 22:23                           ` Petr Baudis
2008-09-03 22:31                             ` Petr Baudis
2008-08-26 17:38                       ` Jakub Narebski
2008-09-03 17:56                         ` Petr Baudis
2008-09-04  4:57                           ` Junio C Hamano
2008-08-26 21:53                       ` Matthieu Moy
2008-08-27  9:09                         ` Karl Hasselström
2008-08-26 17:12                     ` [kernel.org users] [RFD] On deprecating "git-foo" for builtins Shawn O. Pearce
2008-08-26 17:16                       ` Jeff King
2008-08-26 17:47                         ` Nguyen Thai Ngoc Duy
2008-08-26 21:06                         ` Willy Tarreau
2008-08-27 20:24                           ` Jeff Garzik
2008-08-27 20:27                             ` Jeff King
2008-08-27 20:38                               ` Jeff Garzik
2008-08-27 20:53                                 ` Jeff King
2008-08-27 21:05                                 ` Matthew Wilcox
2008-08-27 21:13                                   ` Adrian Bunk
2008-08-27 21:22                                     ` Jeff King
2008-08-27 22:29                                       ` Adrian Bunk
2008-08-27 21:23                                 ` Linus Torvalds
2008-08-27 23:49                                   ` Jeff Garzik
2008-08-28  6:51                                     ` Ingo Molnar
2008-08-28  7:46                                       ` git-show vs git-log (or: git show vs git log) Dominik Brodowski
2008-08-28 14:44                                       ` [kernel.org users] [RFD] On deprecating "git-foo" for builtins Alex Riesen
2008-08-28  5:43                                 ` Mike Hommey
2008-08-27 20:50                             ` Linus Torvalds
2008-08-27  6:00                   ` Ulrich Windl
2008-08-27  7:59                     ` H. Peter Anvin
2008-08-27 19:14                     ` Matthew Wilcox
2008-08-27 19:43                     ` Perry Wagle
2008-08-27 19:50                       ` Jeff King
2008-08-27 19:54                         ` Perry Wagle
2008-08-27 20:43                           ` H. Peter Anvin
2008-08-27 21:19                             ` Steven Rostedt
2008-08-27 23:27                               ` Junio C Hamano
2008-08-27 23:53                                 ` Perry Wagle
2008-08-28  0:05                                   ` Perry Wagle
2008-08-28  9:04                                     ` Petr Baudis
2008-08-28 10:33                                       ` Perry Wagle
2008-08-28 10:42                                         ` David Woodhouse
2008-08-28 19:56                                           ` Perry Wagle
2008-08-28 10:47                                         ` Petr Baudis
2008-08-28 16:35                                         ` Linus Torvalds
2008-08-28 19:24                                           ` Perry Wagle
2008-08-28 19:52                                             ` Teemu Likonen
2008-08-28 20:23                                               ` Perry Wagle
2008-08-28 20:44                                                 ` Petr Baudis
2008-08-28 20:57                                                   ` Perry Wagle
2008-08-28 21:23                                                 ` Jeff King
2008-08-28 21:41                                                   ` Perry Wagle
2008-08-28 21:53                                                     ` Petr Baudis
2008-08-28 21:59                                                     ` Jeff King
2008-08-28 22:33                                                       ` Perry Wagle
2008-08-28 23:04                                                         ` Jeff King [this message]
2008-08-28 23:22                                                           ` Perry Wagle
2008-08-28 23:36                                                             ` Jeff King
2008-08-28 23:24                                                           ` Junio C Hamano
2008-08-28 23:28                                                             ` Perry Wagle
2008-08-28 23:30                                                             ` Petr Baudis
2008-08-28 23:41                                                             ` git-* in test scripts (was On deprecating "git-foo" for builtins) Jeff King
2008-08-29  0:04                                                               ` git-* in test scripts Junio C Hamano
2008-08-29  0:10                                                                 ` Jeff King
2008-08-29  7:51                                                           ` [kernel.org users] [RFD] On deprecating "git-foo" for builtins Andreas Ericsson
2008-08-29  8:05                                                             ` Matthieu Moy
2008-08-29  8:23                                                               ` Andreas Ericsson
2008-08-29  8:27                                                               ` Matthias Kestenholz
2008-08-29  9:02                                                                 ` Matthieu Moy
2008-08-28 23:12                                                         ` Perry Wagle
2008-08-29 15:24                                                           ` Aidan Van Dyk
2008-08-29 16:11                                                             ` Felipe Contreras
2008-08-29 16:24                                                               ` Aidan Van Dyk
2008-08-29 16:28                                                                 ` Felipe Contreras
2008-08-29 16:41                                                                   ` Aidan Van Dyk
2008-08-30  8:13                                                                   ` Andreas Ericsson
2008-08-28 23:31                                                         ` Jakub Narebski
2008-08-29  9:33                                                   ` Wincent Colaiuta
2008-08-30 13:24                                                     ` Steven Rostedt
2008-08-30 13:50                                                       ` Teemu Likonen
2008-08-30 14:08                                                         ` Steven Rostedt
2008-08-26 17:07                 ` Jeff King
2008-08-26 17:42                   ` Teemu Likonen
2008-08-26 16:17             ` Kristian Høgsberg
2008-08-26 16:23             ` Jean Delvare
2008-08-26 16:50               ` Takashi Iwai
2008-08-26 17:20                 ` Jean Delvare
2008-08-27  7:42                   ` Andreas Ericsson
2008-08-27  8:21                     ` Jean Delvare
2008-08-27 15:14                       ` Geert Uytterhoeven
2008-08-26 20:14               ` H. Peter Anvin
2008-08-27  7:42                 ` Jean Delvare
2008-08-27  8:32                 ` [kernel.org users] " Andreas Ericsson
2008-08-26 17:03             ` Linus Torvalds
2008-08-26 17:29               ` Bruce Stephens
2008-08-26 17:35                 ` Petr Baudis
2008-08-26 17:38                   ` Bruce Stephens
2008-08-28 15:21                   ` Johannes Schindelin
2008-08-26 17:34               ` Takashi Iwai
2008-08-26 18:09               ` Dominik Brodowski
2008-08-26 18:19                 ` Linus Torvalds
2008-08-26 18:55                   ` Al Viro
2008-08-26 19:04                     ` Linus Torvalds
2008-08-26 19:22                       ` Al Viro
2008-08-26 19:11                     ` Teemu Likonen
2008-08-28 15:24                     ` Johannes Schindelin
2008-08-26 21:08                   ` Dominik Brodowski
2008-08-26 21:28                   ` Junio C Hamano
2008-08-26 21:38                     ` Linus Torvalds
2008-08-26 23:21                   ` Perry Wagle
2008-08-27 15:27                     ` Steven Rostedt
2008-08-27 23:09                       ` Russell King
2008-08-27 23:53                         ` Stefan Richter
2008-08-28 19:19                           ` Russell King
2008-08-28 20:10                             ` Junio C Hamano
2008-08-28 20:30                               ` Matthew Wilcox
2008-08-28 20:36                               ` Petr Baudis
2008-08-28 20:57                             ` Stefan Richter
2008-08-28 21:05                               ` Perry Wagle
2008-08-28  1:25                         ` Steven Rostedt
2008-08-27  0:34               ` A Large Angry SCM
2008-08-27 12:23           ` Krzysztof Halasa
2008-08-26 20:39         ` Junio C Hamano
2008-08-26 21:03           ` Jeff King
2008-08-27  2:24             ` Jay Soffian
2008-08-27 14:49               ` H. Peter Anvin
2008-08-26 23:36           ` Perry Wagle
2008-08-26 23:45           ` Nicolas Pitre
2008-08-28 15:32             ` Johannes Schindelin
2008-08-27  0:17           ` Matthew Wilcox
2008-08-27 22:52             ` Russell King
2008-08-28 15:34               ` Johannes Schindelin
2008-08-28 16:10                 ` Matthew Wilcox
2008-08-28 19:18                   ` Johannes Schindelin
2008-08-28 19:27                     ` Matthew Wilcox
2008-08-27 23:38             ` Junio C Hamano
2008-08-28  0:09               ` Felipe Contreras
2008-08-28  0:44                 ` Jeff Garzik
2008-08-28  7:14               ` David Woodhouse
2008-08-28  8:17                 ` Junio C Hamano
2008-08-28  8:32                   ` David Woodhouse
2008-08-28  8:57                     ` Felipe Contreras
2008-08-28 11:54                       ` Al Viro
2008-08-28 13:15                         ` Felipe Contreras
2008-08-28 13:34                           ` Felipe Contreras
2008-08-28 13:45                             ` Paolo Ciarrocchi
2008-08-28 16:37                             ` Linus Torvalds
2008-08-28 20:42                               ` Perry Wagle
2008-08-28 23:03                                 ` Jakub Narebski
2008-08-28 23:14                                   ` Perry Wagle
2008-08-28 23:45                                     ` Jeff King
2008-08-28 23:55                                       ` Perry Wagle
2008-08-29 14:12                               ` Felipe Contreras
2008-08-28 14:06                     ` Nicolas Pitre
2008-08-28 14:13                       ` Nicolas Pitre
2008-08-28 16:17                     ` Linus Torvalds
2008-08-26  2:58   ` A Large Angry SCM
2008-08-26  7:17     ` Jean Delvare
2008-08-26 11:12       ` A Large Angry SCM
2008-08-26 11:56         ` Stefan Richter
2008-08-26 21:00           ` Steven Rostedt
2008-08-26 14:28         ` Shawn O. Pearce
2008-08-26 14:46       ` Jeff King

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080828230401.GC29609@coredump.intra.peff.net \
    --to=peff@peff.net \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=andi@firstfloor.org \
    --cc=dwmw2@infradead.org \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=krh@redhat.com \
    --cc=mk@spinlock.ch \
    --cc=pasky@suse.cz \
    --cc=rostedt@goodmis.org \
    --cc=tlikonen@iki.fi \
    --cc=torvalds@linux-foundation.org \
    --cc=ulrich.windl@rz.uni-regensburg.DE \
    --cc=wagle@cs.indiana.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).