Moving APXS-specific checks into respective config.m4. The placement of

`%APXS -q..` caused error messages, if APXS was not used.

configure.in

@@ -798,15 +798,13 @@ if true; then
   else
       rm -f internal_functions.c.old
   fi
-  
-  if test -n "$APXS" ; then
-    if test "a`$APXS -q LD_SHLIB`" = "a" || test "`$APXS -q LIBEXECDIR`" = "modules"; then
-        echo "+--------------------------------------------------------------------+"
-        echo "| WARNING: Your $APXS script is most likely broken."
-        echo "|                                                                    |"
-        echo "| Please go read http://www.php.net/FAQ.php3#4.11 and make the       |"
-        echo "| changes described there and try again.                             |"
-    fi
+
+  if test -n "$PHP_APXS_BROKEN"; then
+    echo "+--------------------------------------------------------------------+"
+    echo "| WARNING: Your $APXS script is most likely broken."
+    echo "|                                                                    |"
+    echo "| Please go read http://www.php.net/FAQ.php3#4.11 and make the       |"
+    echo "| changes described there and try again.                             |"
   fi
 
   # Warn about CGI version with no extra security options.

sapi/apache/config.m4

@@ -23,6 +23,9 @@ AC_ARG_WITH(apxs,
 	PHP_SAPI=apache
 	APACHE_INSTALL="$APXS -i -a -n php4 $SAPI_SHARED"
 	PHP_BUILD_SHARED
+	if test -z "`$APXS -q LD_SHLIB`" || test "`$APXS -q LIBEXECDIR`" = "modules"; then
+		PHP_APXS_BROKEN=yes
+	fi
 	STRONGHOLD=
 	AC_DEFINE(APACHE)
 	AC_DEFINE(HAVE_AP_CONFIG_H)