From 5c52ea73e120f1a02fe498404fe64b976e978936 Mon Sep 17 00:00:00 2001 From: Jeff Trawick Date: Sat, 12 May 2001 13:05:27 +0000 Subject: [PATCH] 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 --- configure.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.in b/configure.in index 3f08b1ad84..9e6b9081bf 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 -- 2.40.0