Since make is using /bin/sh to execute shell code, avoid the newish shell construct $() so older (ancient) shells can execute the shell code in the Makefile. Signed-off-by: Brandon Casey <casey@xxxxxxxxxxxxxxx> --- I know that $() is preferred in the main scripts, but the Makefile is using /bin/sh to execute shell code and there are already a few places in the Makefile using back-ticks, so it doesn't seem like going against the flow too much. Otherwise, should we set the SHELL variable to the configured SHELL_PATH at some point in the Makefile? -brandon Makefile | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index 608a185..421af24 100644 --- a/Makefile +++ b/Makefile @@ -1366,8 +1366,8 @@ endif ifneq (,$X) $(foreach p,$(patsubst %$X,%,$(filter %$X,$(ALL_PROGRAMS) $(BUILT_INS) git$X)), $(RM) '$(DESTDIR_SQ)$(gitexec_instdir_SQ)/$p';) endif - bindir=$$(cd '$(DESTDIR_SQ)$(bindir_SQ)' && pwd) && \ - execdir=$$(cd '$(DESTDIR_SQ)$(gitexec_instdir_SQ)' && pwd) && \ + bindir=`cd '$(DESTDIR_SQ)$(bindir_SQ)' && pwd` && \ + execdir=`cd '$(DESTDIR_SQ)$(gitexec_instdir_SQ)' && pwd` && \ if test "z$$bindir" != "z$$execdir"; \ then \ ln -f "$$bindir/git$X" "$$execdir/git$X" || \ -- 1.6.0.rc1.87.g56c9f.dirty -- 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
