git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Perry Wagle <wagle@cs.indiana.edu>
To: Teemu Likonen <tlikonen@iki.fi>
Cc: 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>, Jeff King <peff@peff.net>,
	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 13:23:50 -0700	[thread overview]
Message-ID: <4B9831F7-3CB8-49CB-A1DB-111481A271FE@cs.indiana.edu> (raw)
In-Reply-To: <20080828195211.GA3545@mithlond.arda.local>


On Aug 28, 2008, at 12:52 PM, Teemu Likonen wrote:

> Perry Wagle wrote (2008-08-28 12:24 -0700):
>
>> Is is really only that I
>>
>>    sed s/git-/git<SPACE>/g
>>
>> on the scripts?  I'm doubting it, given the quality of reasoning and
>> lack of respect for upward compatibility on this thread.
>
> I have come to understand that "git " has quite long time been more
> robust and portable way of writing scripts. They work in both
> configurations so I'd definitely suggest doing "s/git-/git /g" for  
> every
> script. Of course in an interactive shell everyone can use whatever  
> they
> prefer and works at the moment.

Sure.  Its an extra fork in git command intensive scripts (and git is  
racey still maybe), but *shrug*.

When I started with git in Fall 2007, the sample scripts and gitweb  
both used git<DASH> and git<SPACE> willy-nilly.  I liked git<DASH>  
better since it was what I used at the command line for the <TAB>  
completion for the commands and for the man pages, and I like being  
meticulously consistent so I can greatly reduce mistakes.

Even as of March 2008 (our last sync with git before the git scripting  
was completed and we got on to other things), the sample scripts and  
gitweb still used the git<DASH> form.  If this has been brewing for  
two years, there shouldn't have been a git<DASH> form in the scripts  
in the standard source *anywhere* for those two years.

There was.  Therefore, you don't get to claim that this was decided  
two years ago, finally done now, and what's Perry's problem anyway?

But, my problem is not git<DASH> vs git<SPACE>, but the slap-dash way  
upward compatibility was broken and the "water over the dam" slippery  
slope rationalizations that refuse to consider reverting.  "You" will  
do it again in the future since you are declaring success here.  And  
"you" have likely done it in the past 6 months.

-- Perry

  reply	other threads:[~2008-08-28 20:27 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 [this message]
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
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=4B9831F7-3CB8-49CB-A1DB-111481A271FE@cs.indiana.edu \
    --to=wagle@cs.indiana.edu \
    --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=peff@peff.net \
    --cc=rostedt@goodmis.org \
    --cc=tlikonen@iki.fi \
    --cc=torvalds@linux-foundation.org \
    --cc=ulrich.windl@rz.uni-regensburg.DE \
    /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).