Hi, On Tue, 29 Jul 2008, Steffen Prohaska wrote: > On Jul 29, 2008, at 7:24 AM, Shawn O. Pearce wrote: > > > Steffen Prohaska <prohaska@xxxxxx> wrote: > > > We prefer running the dashless form, so we should use it in MinGW's > > > start_command(), too. > > ... > > > - We have non-builtins that are implemented in C, e.g. fast-import.c. > > > These non-builtins will still compute wrong paths. > > > > This feels wrong to me. fast-import probably won't be adversly > > impacted by not being able to read /etc/gitconfig, unless the user has > > set something like core.deltaBaseCacheLimit and is doing an > > incremental import. But other non-builtins may be impacted. > > > > It feels like we're fixing this in the wrong place. If the issue is > > we don't find our installation directory correctly, we should find our > > installation directory correctly, not work around it by calling > > builtins through the git wrapper. > > For builtins it is not a work around but a real solution. No, it is not a real solution. At least not for the goal you stated: getting the correct "toplevel" directory. It is a workaround, because it avoids the builtins being called as proper programs (which they should be perfectly capable of, and which scripts still _are allowed to do_). However, it is a solution. To a totally other problem, namely "I would like to be able _not_ to install the builtins into the exec-path". Sure, you break a few scripts that rely on the builtins being callable in the dash-form, but at least the scripts we ship in git.git are safe. Ciao, Dscho -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
