From: Jeff Trawick <trawick@apache.org>
Date: Sat, 12 May 2001 13:05:27 +0000 (+0000)
Subject: Only turn on AP_DEBUG if we're using gcc.
X-Git-Tag: 2.0.18~32
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5c52ea73e120f1a02fe498404fe64b976e978936;p=thirdparty%2Fapache%2Fhttpd.git

Only turn on AP_DEBUG if we're using gcc.

The AP_DEBUG flavor of the cmd_func declarations is not portable.
Last night Roy removed the check for gcc before turning on AP_DEBUG.
Interestingly (or not :) ) the AP_DEBUG flavor of cmd_func compiles
cleanly on gcc but the non-AP_DEBUG flavor doesn't. The non-AP_DEBUG
flavor compiles cleanly with Tru64 cc and AIX xlc but the AP_DEBUG
flavor doesn't. (On Tru64 we get a bunch of warnings with the
AP_DEBUG flavor but at least it compiles; the same cannot be said of
AIX xlc.)

Perhaps we should use a symbol other than AP_DEBUG to distinguish
between the cmd_func flavors.  Then AP_DEBUG can be turned on
regardless of compiler.  The new flag can be turned on for gcc+
--enable-maintainer-mode, or perhaps for gcc always since the non-
AP_DEBUG flavor won't compile cleanly with gcc.  (what a mess; this
probably wouldn't work right for different levels of gcc anyway)


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@89096 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/configure.in b/configure.in
index 3f08b1ad84c..9e6b9081bf3 100644
--- a/configure.in
+++ b/configure.in
@@ -208,7 +208,7 @@ AC_ARG_WITH(port,[  --with-port=PORT        Port on which to listen (default is
 	[PORT=80])
 
 AC_ARG_ENABLE(maintainer-mode,[  --enable-maintainer-mode  Turn on debugging and compile time warnings],
-  [APR_ADDTO(CPPFLAGS,-DAP_DEBUG)
+  [if test "$ac_cv_prog_gcc" = "yes"; then APR_ADDTO(CPPFLAGS,-DAP_DEBUG); fi
 ])dnl
 
 APACHE_ENABLE_LAYOUT