Ben Walton wrote: > --- a/Makefile > +++ b/Makefile > @@ -1913,6 +1913,8 @@ builtin/help.sp builtin/help.s builtin/help.o: EXTRA_CPPFLAGS = \ > '-DGIT_MAN_PATH="$(mandir_SQ)"' \ > '-DGIT_INFO_PATH="$(infodir_SQ)"' > > +run-command.o: EXTRA_CPPFLAGS = -DSHELL_PATH='"$(SHELL_PATH_SQ)"' Sorry I forgot to send this before. Quick tweaks: - let assembler listings (from "make run-command.s") and sparse checks reflect the SHELL_PATH setting, too - the entire -DFOO="bar" argument is surrounded with single quotes in other EXTRA_CPPFLAGS settings, so let this one follow that pattern to avoid standing out diff --git i/Makefile w/Makefile index dea1f157..a3791139 100644 --- i/Makefile +++ w/Makefile @@ -1913,7 +1913,8 @@ builtin/help.sp builtin/help.s builtin/help.o: EXTRA_CPPFLAGS = \ '-DGIT_MAN_PATH="$(mandir_SQ)"' \ '-DGIT_INFO_PATH="$(infodir_SQ)"' -run-command.o: EXTRA_CPPFLAGS = -DSHELL_PATH='"$(SHELL_PATH_SQ)"' +run-command.sp run-command.s run-command.o: EXTRA_CPPFLAGS = \ + '-DSHELL_PATH="$(SHELL_PATH_SQ)"' $(BUILT_INS): git$X $(QUIET_BUILT_IN)$(RM) $@ && \ -- 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