Re: [PATCH] Fix configure CXX check for absolute paths

[Fyodor <fyodor () insecure org>]

On Sun, Feb 12, 2006 at 05:18:30PM +0100, Daniel Roethlisberger wrote:
> The configure script will bail out if CXX is specified with an absolute
> path (such as /usr/local/libexec/ccache/c++ for ccache users).  The
> attached patch adds "/" to the locations searched for CXX, which makes
> it work for absolute paths too.  A bit of a hack, but seems to work with
> no evil side effects.

Looks good to me.  I have applied it for the next version.  I changed
it slightly to put the / at the end of the path rather than the
beginning:

--- configure.ac        (revision 3120)
+++ configure.ac        (working copy)
@@ -42,7 +42,7 @@
  if test -n "$GXX"; then
       CXXFLAGS="$CXXFLAGS -Wall "
  fi
-AC_CHECK_PROG(CXXPROG, "$CXX", "AVAILABLE", "MISSING")
+AC_CHECK_PROG(CXXPROG, "$CXX", "AVAILABLE", "MISSING", "$PATH":/)
 if test $CXXPROG = "MISSING"; then
   AC_MSG_ERROR([Could not locate a C++ compiler. If it exists, add it to your PATH or give configure the 
CXX=path_to_compiler argument.  Otherwise, install a C++ compiler such as g++ or install a binary package of Nmap (see 
http://www.insecure.org/nmap/nmap_download.html ))])


I hope that still works.

Cheers,
Fyodor


_______________________________________________
Sent through the nmap-dev mailing list
http://cgi.insecure.org/mailman/listinfo/nmap-dev