Junio C Hamano <gitster@xxxxxxxxx> writes: > Chris Webb <chris@xxxxxxxxxxxx> writes: > > > In exec_cmd.c, git hard-codes a default path of > > /usr/local/bin:/usr/bin:/bin. Get an appropriate value for the system > > from <paths.h> if possible instead. We only try to include <paths.h> on > > Linux, FreeBSD, NetBSD, OpenBSD and GNU where it is known to exist. > > > > Signed-off-by: Chris Webb <chris@xxxxxxxxxxxx> > > --- > > Makefile | 10 ++++++++++ > > exec_cmd.c | 2 +- > > git-compat-util.h | 6 ++++++ > > 3 files changed, 17 insertions(+), 1 deletions(-) > > > > diff --git a/Makefile b/Makefile > > index 910f471..f4fe941 100644 > > --- a/Makefile > > +++ b/Makefile > > @@ -735,10 +735,12 @@ EXTLIBS = > > ifeq ($(uname_S),Linux) > > NO_STRLCPY = YesPlease > > NO_MKSTEMPS = YesPlease > > + HAVE_PATHS_H = YesPlease > > endif > > Ok. Somebody else may want to add an autoconf support on top of this, but > this is good as-is, I think. Something like that? -- >8 -- Subject: [PATCH] autoconf: Check if <paths.h> exists Signed-off-by: Jakub Narebski <jnareb@xxxxxxxxx> --- config.mak.in | 1 + configure.ac | 6 ++++++ 2 files changed, 7 insertions(+), 0 deletions(-) diff --git a/config.mak.in b/config.mak.in index e8d96e8..f0aeb8e 100644 --- a/config.mak.in +++ b/config.mak.in @@ -31,6 +31,7 @@ NO_OPENSSL=@NO_OPENSSL@ NO_CURL=@NO_CURL@ NO_EXPAT=@NO_EXPAT@ NO_LIBGEN_H=@NO_LIBGEN_H@ +HAVE_PATHS_H=@HAVE_PATHS_H@ NEEDS_LIBICONV=@NEEDS_LIBICONV@ NEEDS_SOCKET=@NEEDS_SOCKET@ NO_SYS_SELECT_H=@NO_SYS_SELECT_H@ diff --git a/configure.ac b/configure.ac index 108a97f..9dc0320 100644 --- a/configure.ac +++ b/configure.ac @@ -633,6 +633,12 @@ AC_CHECK_HEADER([libgen.h], [NO_LIBGEN_H=YesPlease]) AC_SUBST(NO_LIBGEN_H) # +# Define HAVE_PATHS_H if you have paths.h. +AC_CHECK_HEADER([paths.h], +[HAVE_PATHS_H=YesPlease], +[HAVE_PATHS_H=]) +AC_SUBST(HAVE_PATHS_H) +# # Define NO_STRCASESTR if you don't have strcasestr. GIT_CHECK_FUNC(strcasestr, [NO_STRCASESTR=], -- 1.7.0.1 -- 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