Ethereal-dev: [Ethereal-dev] [PATCH] acinclude.m4: --with-krb5

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Thomas Anders <thomas.anders@xxxxxxxxxxxxx>
Date: Tue, 18 May 2004 12:07:31 +0200
configure currently behaves badly for "--with-krb5" if either
- the path to krb5-config contains spaces (a bad idea, of course), or
- krb5-config is not in PATH (making $KRB5_CONFIG empty, but "test -x"
  succeeds)

The attached patch (for acinclude.m4) should fix this.


+Thomas

--
Thomas Anders (thomas.anders at blue-cable.de)
--- acinclude.m4.FCS	2004-05-14 12:33:00.000000000 +0200
+++ acinclude.m4	2004-05-18 11:54:52.000000000 +0200
@@ -886,12 +886,12 @@
 	  ac_krb5_version=`grep -i heimdal $krb5_dir/include/krb5.h | head -n 1 | sed 's/^.*heimdal.*$/HEIMDAL/i'` 
 	else
 	  AC_PATH_PROG(KRB5_CONFIG, krb5-config) 
-	  if test -x $KRB5_CONFIG
+	  if test -x "$KRB5_CONFIG"
 	  then
-	    KRB5_FLAGS=`$KRB5_CONFIG --cflags`
+	    KRB5_FLAGS=`"$KRB5_CONFIG" --cflags`
 	    CFLAGS="$CFLAGS $KRB5_FLAGS"
             CPPFLAGS="$CPPFLAGS $KRB5_FLAGS"
-	    KRB5_LIBS=`$KRB5_CONFIG --libs`
+	    KRB5_LIBS=`"$KRB5_CONFIG" --libs`
 	    # Looks like krb5-config is lacking -lresolv on some systems
 	    AC_MSG_CHECKING(whether library list looks OK)
 	    if echo "$KRB5_LIBS" | grep resolv >/dev/null
@@ -903,7 +903,7 @@
 	    fi
 
 	    #LIBS="$LIBS $KRB5_LIBS"
-	    ac_krb5_version=`$KRB5_CONFIG --version | head -n 1 | sed 's/^.*heimdal.*$/HEIMDAL/i'`
+	    ac_krb5_version=`"$KRB5_CONFIG" --version | head -n 1 | sed 's/^.*heimdal.*$/HEIMDAL/i'`
  	  fi
 	fi