git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Nicolas Pitre <nico@cam.org>
Cc: Nguyen Thai Ngoc Duy <pclouds@gmail.com>, Jan Hudec <bulb@ucw.cz>,
	Johannes Schindelin <Johannes.Schindelin@gmx.de>,
	git@vger.kernel.org
Subject: Re: [PATCH] Move all dashed form git commands to libexecdir
Date: Thu, 29 Nov 2007 14:36:15 -0800	[thread overview]
Message-ID: <7vd4tsvfvk.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <alpine.LFD.0.99999.0711290905510.9605@xanadu.home> (Nicolas Pitre's message of "Thu, 29 Nov 2007 09:09:42 -0500 (EST)")

Nicolas Pitre <nico@cam.org> writes:

> On Thu, 29 Nov 2007, Nguyen Thai Ngoc Duy wrote:
>
>> There won't be a stage when only porcelain git-foos are in $(bindir)?
>> I could stop working on the relevant patch then.
>
> Well, I personally found your effort really nice.  I think Junio is 
> overly cautious in this case, and I would prefer to see the number of 
> git commands in the default path drop rather sooner than later.

I agree with the first sentence.  And yes I am playing it safe, and at
the same time I do not think the "default" really matters as much as
people think.

If people are really serious about reducing the number of commands in
the path, I would expect fixes and bugreports saying "I am setting
gitexecdir different from bindir in _my_ installation when I build git,
and here are the things that does not work if I do so".  Within the span
of more than 20 months (77cb17e9 introduced gitexecdir in Jan 2006), I
do not think there was a single such report or patch, other than the
message from Nguyen that started this thread.

Which means one of two things (1) we got everything right and there is
nothing to fix, other than changing the default like Nguyen's patch
does, or (2) nobody is interested in moving git-foo out of their PATH
for _his_ own use, but pushing changes that would affect _other_ people
without testing.

I am of course hoping that (1) is the case.  And it could be that in
open-source settings often the silent majority is content with what's
already there, and that many people who are indeed interested in moving
git-foo out of their PATH are doing so happily without telling the
others of their success.

But it still worries me.

And people's scripts, especially old/unmaintained ones that google still
knows about, are worrysome too.  Didn't we just see a message that says
"git-update-cache in a script I picked up from google does not work" on
the list?

  reply	other threads:[~2007-11-29 22:37 UTC|newest]

Thread overview: 92+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-27 15:02 [PATCH RFC] Move all dashed form git commands to libexecdir Nguyễn Thái Ngọc Duy
2007-11-27 15:12 ` Johannes Schindelin
2007-11-27 15:25 ` Nicolas Pitre
2007-11-27 16:04 ` [PATCH] " Nguyễn Thái Ngoc Duy
2007-11-27 16:18   ` Johannes Schindelin
2007-11-28  0:07     ` Jan Hudec
2007-11-28  1:13       ` Junio C Hamano
2007-11-28  8:18         ` Jan Hudec
2007-11-28  8:36         ` Nguyen Thai Ngoc Duy
2007-11-28 23:14           ` Junio C Hamano
2007-11-28 23:40             ` Johannes Schindelin
2007-11-28 23:48               ` Junio C Hamano
2007-11-29  0:01                 ` Johannes Schindelin
2007-11-29  0:59             ` A Large Angry SCM
2007-11-29  1:02               ` Junio C Hamano
2007-11-29  3:17             ` Nguyen Thai Ngoc Duy
2007-11-29 14:09               ` Nicolas Pitre
2007-11-29 22:36                 ` Junio C Hamano [this message]
2007-11-30  7:32                   ` Wincent Colaiuta
2007-11-30 11:28                   ` Eyvind Bernhardsen
2007-11-30 12:08                     ` [PATCH] transport.c: call dash-less form of receive-pack and upload-pack on remote Johannes Schindelin
2007-12-01  2:36                       ` Junio C Hamano
2007-12-01 10:17                         ` Johannes Schindelin
2007-12-01 19:30                           ` Junio C Hamano
2007-12-01 23:03                             ` Johannes Schindelin
2007-12-01 23:15                               ` Johannes Schindelin
2007-12-02  1:57                                 ` Junio C Hamano
2007-12-02  2:52                                 ` [PATCH 0/3] Call builtin functions directly, was " Johannes Schindelin
2007-12-02  2:54                                   ` [PATCH 1/3] Introduce release_all_objects() Johannes Schindelin
2007-12-02  2:54                                   ` [PATCH 2/3] Include the objects needed for the builtin functions into libgit.a Johannes Schindelin
2007-12-02  2:55                                   ` [PATCH 3/3] Introduce execv_git_builtin() and use it Johannes Schindelin
2007-12-02  3:04                                     ` Johannes Schindelin
2007-12-02  3:16                                       ` [REPLACEMENT PATCH " Johannes Schindelin
2007-12-02  5:19                                   ` [PATCH 0/3] Call builtin functions directly, was Re: [PATCH] transport.c: call dash-less form of receive-pack and upload-pack on remote Junio C Hamano
2007-12-02 11:35                                     ` Johannes Schindelin
2007-11-30 12:19                     ` [PATCH] Move all dashed form git commands to libexecdir Nguyen Thai Ngoc Duy
2007-11-30 13:35                       ` Johannes Schindelin
2007-11-29 15:08             ` Jeff King
2007-11-29 20:05               ` Nguyen Thai Ngoc Duy
2007-11-29 21:14                 ` Jeff King
2007-11-29 22:19                   ` Johannes Schindelin
2007-11-29 23:14                     ` Jeff King
2007-11-29 23:30                       ` Linus Torvalds
2007-11-30  0:13                         ` Junio C Hamano
2007-11-30  0:35                           ` Jeff King
2007-11-30  0:49                             ` Junio C Hamano
2007-11-30  0:58                               ` Jeff King
2007-11-30  1:13                                 ` Nicolas Pitre
2007-11-30  1:17                                   ` Jeff King
2007-11-30  5:42                                     ` Steffen Prohaska
2007-11-30  7:18                                     ` Andreas Ericsson
2007-11-30 15:09                                       ` Jeff King
2007-11-30 20:01                                         ` Junio C Hamano
2007-11-30 21:25                                           ` Jeff King
2007-11-30 23:10                                             ` Johannes Schindelin
2007-12-02 15:02                                               ` Wincent Colaiuta
2007-12-02 16:39                                                 ` Johannes Schindelin
2007-12-02 16:56                                                   ` Pascal Obry
2007-12-02 17:23                                                     ` Johannes Schindelin
2007-12-01  2:37                                             ` Junio C Hamano
2007-12-01  4:17                                               ` Jeff King
2007-11-30  2:29                                 ` Linus Torvalds
2007-11-30  2:55                                   ` Nicolas Pitre
2007-11-30  5:51                                   ` Steffen Prohaska
2007-11-30 15:12                                     ` Jeff King
2007-11-30 15:28                                       ` Santi Béjar
2007-11-30 15:29                                         ` Jeff King
2007-11-30 15:50                                           ` Linus Torvalds
2007-11-30 16:22                                             ` Jeff King
2007-11-30 18:28                                               ` Johannes Schindelin
2007-11-30 18:37                                                 ` Jeff King
2007-11-30 23:05                                                   ` Johannes Schindelin
2007-11-30 23:21                                                     ` Jeff King
2007-11-30 23:38                                                       ` Johannes Schindelin
     [not found]                                             ` <fcaeb9bf0711302234l32460a1fqbf9825fc8055f99d@mail.gmail.com>
2007-12-01 19:32                                               ` Junio C Hamano
2007-12-01 21:26                                                 ` Jeff King
2007-12-02  5:50                                                 ` Nguyen Thai Ngoc Duy
2007-11-30  0:52                             ` Nicolas Pitre
2007-11-30  1:00                               ` Jeff King
2007-11-30  1:19                                 ` Nicolas Pitre
2007-11-30  1:25                                   ` Jeff King
2007-11-30  1:33                                     ` Nicolas Pitre
2007-11-30  1:53                                       ` Jeff King
2007-11-30  2:23                                         ` A Large Angry SCM
2007-11-30  0:40                           ` Nguyen Thai Ngoc Duy
2007-11-30  0:51                           ` A Large Angry SCM
2007-11-30  0:54                             ` Johannes Schindelin
2007-11-30  2:03                               ` A Large Angry SCM
2007-11-30  1:01                             ` Nicolas Pitre
2007-11-30  2:17                               ` A Large Angry SCM
2007-11-30  2:27                                 ` Nicolas Pitre
2007-11-29  0:14           ` Jakub Narebski

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=7vd4tsvfvk.fsf@gitster.siamese.dyndns.org \
    --to=gitster@pobox.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=bulb@ucw.cz \
    --cc=git@vger.kernel.org \
    --cc=nico@cam.org \
    --cc=pclouds@gmail.com \
    /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).