On Sat, Sep 22, 2012 at 02:37:38AM +0530, Ramkumar Ramachandra wrote: > > I don't think that is the right thing to do. The point of SHELL is to > > point at a bourne-compatible shell. On some systems, the main reason to > > set it is that /bin/sh is _broken_, and we are trying to avoid it. > > But you're only avoiding it in the --tee/ --va* codepath. In the > normal codepath, you're stuck with /bin/sh anyway.
No, the #!-header is only information. When you run "make test" we actually invoke the shell ourselves using $SHELL_PATH. > > A bigger question is: why are you setting SHELL=zsh in the first place? > > I use ZSH as my primary shell, so SHELL is set to zsh when I run > tests. How can we trust $SHELL to be a bourne-compatible shell? Ah, my fault. I was thinking we overrode $SHELL along with $SHELL_PATH, but we do not. The correct patch is to stop using $SHELL, but not to switch to a manual /bin/sh. It should use $SHELL_PATH instead, which is how you tell git your path to a sane bourne shell. -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html