Re: cvs import and internal plumbing commands
- From
Jeff King <peff@peff.net>
- Date
- Jul 20, 2009, 15:34 UTC
- Message-ID
- <20090720153400.GE5347@coredump.intra.peff.net>
- In-Reply-To
- <7vprbwvzu7.fsf@alter.siamese.dyndns.org>
On Sun, Jul 19, 2009 at 07:43:28PM -0700, Junio C Hamano wrote:
> We encourage script writers to just prepend "git --exec-path" at the > beginning of their PATH and give guarantee that this simple procedure will > keep their scripts working, so it might be a good idea to have an example > for that as well.
I had it in my head that such a strategy was supposed to be a temporary step in the process of getting rid of dashed forms. That is, you can do that now to avoid having your scripts break, but in the long run, you should be converting to the space form.
But I guess the plan was softened, and we have no deprecation plan for dashed forms in exec-path. So maybe my thinking was outdated.
Show 5 quoted lines
> So the preferred fix might be just the matter of adding one line
>
> $ENV{'PATH'} = `git --exec-path`. ":$ENV{'PATH'}";
>
> at the beginning of the script.I can see that for a totally third-party script which wanted to use git plumbing. But why do that for a git-* script? The "git" wrapper sets up the environment like that already.
-Peff