This patch changes many AC_DEFINE's to use the 3-argument form, and removes the corresponding #undef from acconfig.h, since acconfig.h is deprecated in autoconf 2.5x (and presumably the support for it may eventually disappear).
Fixing the rest would be easiest using AH_VERBATIM and friends, but I believe making autoconf 2.5x a requirement was undesirable? This patch makes only one functional change (verify this by diffing before-and-after output of "grep ^# apr_private.h.in | sort"): the #undef HAVE_isascii is removed - AFAICT configure never defines this and it's never used. Index: configure.in =================================================================== RCS file: /home/cvspublic/apr/configure.in,v retrieving revision 1.481 diff -u -r1.481 configure.in --- configure.in 10 Sep 2002 09:11:20 -0000 1.481 +++ configure.in 13 Sep 2002 08:23:56 -0000 @@ -363,7 +363,7 @@ APR_ADDTO(CPPFLAGS,-DBEOS) enable_threads="system_threads" native_mmap_emul="1" - APR_CHECK_DEFINE(BONE_VERSION, sys/socket.h) + APR_CHECK_DEFINE(BONE_VERSION, sys/socket.h, [Define if BONE_VERSION is defined]) eolstr="\\n" osver=`uname -r` proc_mutex_is_global=1 @@ -506,7 +506,7 @@ threads="1" pthreadh="1" pthreadser="1" - AC_DEFINE(USE_THREADS) ], [ + AC_DEFINE(USE_THREADS, 1, [Define if threads are supported]) ], [ threads="0" pthreadh="0" pthreadser="0" @@ -567,18 +567,21 @@ ac_cv_define_GETHOSTBYADDR_IS_THREAD_SAFE=no if test "$threads" = "1"; then echo "APR will use threads" - AC_CHECK_LIB(c_r, readdir, AC_DEFINE(READDIR_IS_THREAD_SAFE)) + AC_CHECK_LIB(c_r, readdir, + AC_DEFINE([READDIR_IS_THREAD_SAFE], 1, [Define if readdir is thread-safe])) if test "x$apr_gethostbyname_is_thread_safe" = "x"; then AC_CHECK_LIB(c_r, gethostbyname, apr_gethostbyname_is_thread_safe=yes) fi if test "$apr_gethostbyname_is_thread_safe" = "yes"; then - AC_DEFINE(GETHOSTBYNAME_IS_THREAD_SAFE) + AC_DEFINE([GETHOSTBYNAME_IS_THREAD_SAFE], 1, + [Define if gethostbyname is thread-safe]) fi if test "x$apr_gethostbyaddr_is_thread_safe" = "x"; then AC_CHECK_LIB(c_r, gethostbyaddr, apr_gethostbyaddr_is_thread_safe=yes) fi if test "$apr_gethostbyaddr_is_thread_safe" = "yes"; then - AC_DEFINE(GETHOSTBYADDR_IS_THREAD_SAFE) + AC_DEFINE([GETHOSTBYADDR_IS_THREAD_SAFE], 1, + [Define if gethostbyaddr is thread-safe]) fi AC_CHECK_FUNCS(gethostbyname_r gethostbyaddr_r) @@ -625,7 +628,7 @@ esac AC_CHECK_HEADERS(sys/mman.h) -APR_CHECK_DEFINE(MAP_ANON, sys/mman.h) +APR_CHECK_DEFINE(MAP_ANON, sys/mman.h, [Define if mmap supports MAP_ANON]) AC_CHECK_FUNCS(mmap munmap shm_open shm_unlink) APR_CHECK_FILE(/dev/zero) AC_CHECK_HEADERS(sys/ipc.h sys/shm.h sys/file.h) @@ -1108,7 +1111,7 @@ ], [ ac_cv_define_INT64_C=no AC_MSG_RESULT(no) - APR_CHECK_DEFINE(INT64_C, stdint.h) + APR_CHECK_DEFINE(INT64_C, stdint.h, [Define if INT64_C is available]) if test "$ac_cv_define_INT64_C" = "yes"; then stdint=1 fi @@ -1290,9 +1293,13 @@ aprdso="0" else case "$tempdso" in - dlfcn) AC_DEFINE(DSO_USE_DLFCN);; - shl) AC_DEFINE(DSO_USE_SHL);; - dyld) AC_DEFINE(DSO_USE_DYLD);; + dlfcn) AC_DEFINE(DSO_USE_DLFCN, 1, + [Define if dlopen should be used for DSO support]);; + shl) AC_DEFINE(DSO_USE_SHL, 1, + [Define if shl_load should be used for DSO support]);; + dyld) AC_DEFINE(DSO_USE_DYLD, 1, + [Define if NSLinkModule should be used for DSO support]) + ;; esac aprdso="1" apr_modules="$apr_modules dso" @@ -1323,7 +1330,7 @@ ], vla_msg=yes, vla_msg=no ) AC_MSG_RESULT([$vla_msg]) if test "$vla_msg" = "yes"; then - AC_DEFINE(HAVE_VLA) + AC_DEFINE(HAVE_VLA, 1, [Define if variable length arrays are supported]) fi AC_CACHE_CHECK(struct rlimit,ac_cv_struct_rlimit,[ @@ -1391,16 +1398,17 @@ AC_SUBST(have_union_semun) dnl Checks for libraries. -APR_CHECK_DEFINE(LOCK_EX, sys/file.h) -APR_CHECK_DEFINE(F_SETLK, fcntl.h) -APR_CHECK_DEFINE(SEM_UNDO, sys/sem.h) +APR_CHECK_DEFINE(LOCK_EX, sys/file.h, [Define if LOCK_EX is supported]) +APR_CHECK_DEFINE(F_SETLK, fcntl.h, [Define if F_SETLK is supported]) +APR_CHECK_DEFINE(SEM_UNDO, sys/sem.h, [Define if SEM_UNDO is supported]) # We are assuming that if the platform doesn't have POLLIN, it doesn't have # any POLL definitions. APR_CHECK_DEFINE_FILES(POLLIN, poll.h sys/poll.h) if test "$threads" = "1"; then - APR_CHECK_DEFINE(PTHREAD_PROCESS_SHARED, pthread.h) + APR_CHECK_DEFINE(PTHREAD_PROCESS_SHARED, pthread.h, + [Define if PTHREAD_PROCESS_SHARED is supported]) AC_CHECK_FUNCS(pthread_mutexattr_setpshared) dnl Some systems have setpshared and define PROCESS_SHARED, but don't dnl really support PROCESS_SHARED locks. So, we must validate that we @@ -1547,7 +1555,7 @@ AC_MSG_CHECKING(if Posix sems affect threads in the same process) if test "x$apr_posixsem_is_global" = "xyes"; then - AC_DEFINE(POSIXSEM_IS_GLOBAL) + AC_DEFINE([POSIXSEM_IS_GLOBAL], 1, [Define if POSIX semaphores are global]) AC_MSG_RESULT(yes) else AC_MSG_RESULT(no) @@ -1555,7 +1563,7 @@ AC_MSG_CHECKING(if SysV sems affect threads in the same process) if test "x$apr_sysvsem_is_global" = "xyes"; then - AC_DEFINE(SYSVSEM_IS_GLOBAL) + AC_DEFINE(SYSVSEM_IS_GLOBAL, 1, [Define if SysV semaphores are global]) AC_MSG_RESULT(yes) else AC_MSG_RESULT(no) @@ -1563,7 +1571,7 @@ AC_MSG_CHECKING(if fcntl locks affect threads in the same process) if test "x$apr_fcntl_is_global" = "xyes"; then - AC_DEFINE(FCNTL_IS_GLOBAL) + AC_DEFINE(FCNTL_IS_GLOBAL, 1, [Define if fcntl locks are global]) AC_MSG_RESULT(yes) else AC_MSG_RESULT(no) @@ -1571,7 +1579,7 @@ AC_MSG_CHECKING(if flock locks affect threads in the same process) if test "x$apr_flock_is_global" = "xyes"; then - AC_DEFINE(FLOCK_IS_GLOBAL) + AC_DEFINE(FLOCK_IS_GLOBAL, 1, [Define if flock locks are global]) AC_MSG_RESULT(yes) else AC_MSG_RESULT(no) @@ -1582,9 +1590,11 @@ AC_ARG_WITH(egd, [ --with-egd[[=<path>]] use egd-compatible socket], - [ AC_DEFINE(HAVE_EGD) + [ AC_DEFINE(HAVE_EGD, 1, [Define if EGD is available]) if test "$withval" = "yes"; then - AC_DEFINE_UNQUOTED(EGD_DEFAULT_SOCKET, ["/var/run/egd-pool","/dev/egd-pool","/etc/egd-pool","/etc/entropy"]) + AC_DEFINE_UNQUOTED(EGD_DEFAULT_SOCKET, + ["/var/run/egd-pool","/dev/egd-pool","/etc/egd-pool","/etc/entropy"], + [Define to be the list of paths to EGD sockets]) else AC_DEFINE_UNQUOTED(EGD_DEFAULT_SOCKET, ["$withval"]) fi @@ -1599,7 +1609,7 @@ if test "$apr_devrandom" = "yes"; then if test -r "/dev/random"; then - AC_DEFINE(DEV_RANDOM, [/dev/random]) + AC_DEFINE(DEV_RANDOM, [/dev/random], [Define to path of random device]) AC_MSG_RESULT(/dev/random) rand="1" elif test -r "/dev/arandom"; then @@ -1632,7 +1642,7 @@ *) if test "$rand" != "1"; then if test "$ac_cv_lib_truerand_main" = "yes"; then - AC_DEFINE(HAVE_TRUERAND) + AC_DEFINE(HAVE_TRUERAND, 1, [Define if truerand is available]) AC_MSG_RESULT(truerand) rand="1" else @@ -1653,7 +1663,7 @@ ac_cv_struct_tm_gmtoff=yes, ac_cv_struct_tm_gmtoff=no)]) if test "$ac_cv_struct_tm_gmtoff" = "yes"; then - AC_DEFINE(HAVE_GMTOFF) + AC_DEFINE([HAVE_GMTOFF], 1, [Define if struct tm has a tm_gmtoff field]) fi dnl #----------------------------- Checking for Networking Support @@ -1692,8 +1702,8 @@ APR_CHECK_O_NONBLOCK_INHERITED dnl # Look for a way of corking TCP... -APR_CHECK_DEFINE(TCP_CORK, netinet/tcp.h) -APR_CHECK_DEFINE(TCP_NOPUSH, netinet/tcp.h) +APR_CHECK_DEFINE(TCP_CORK, netinet/tcp.h, [Define if TCP_CORK is supported]) +APR_CHECK_DEFINE(TCP_NOPUSH, netinet/tcp.h, [Define if TCP_NOPUSH is supported]) apr_tcp_nopush_flag="0" have_corkable_tcp="0" if test "x$ac_cv_define_TCP_CORK" = "xyes"; then @@ -1721,7 +1731,7 @@ have_corkable_tcp="1" fi -APR_CHECK_DEFINE(SO_ACCEPTFILTER, sys/socket.h) +APR_CHECK_DEFINE(SO_ACCEPTFILTER, sys/socket.h, [Define if SO_ACCEPTFILTER is supported]) if test "x$ac_cv_define_SO_ACCEPTFILTER" = "xyes"; then acceptfilter="1" else Index: acconfig.h =================================================================== RCS file: /home/cvspublic/apr/acconfig.h,v retrieving revision 1.57 diff -u -r1.57 acconfig.h --- acconfig.h 2 Aug 2002 18:12:03 -0000 1.57 +++ acconfig.h 13 Sep 2002 08:23:56 -0000 @@ -4,18 +4,7 @@ @TOP@ /* Various #defines we need to know about */ -#undef HAVE_LOCK_EX -#undef HAVE_F_SETLK -#undef HAVE_SEM_UNDO -#undef HAVE_PTHREAD_PROCESS_SHARED -#undef DEV_RANDOM -#undef HAVE_EGD -#undef EGD_DEFAULT_SOCKET -#undef HAVE_TRUERAND #undef HAVE_POLLIN -#undef HAVE_isascii -#undef HAVE_SO_ACCEPTFILTER -#undef HAVE_MAP_ANON /* Cross process serialization techniques */ #undef USE_FLOCK_SERIALIZE @@ -23,39 +12,6 @@ #undef USE_FCNTL_SERIALIZE #undef USE_PROC_PTHREAD_SERIALIZE #undef USE_PTHREAD_SERIALIZE - -#undef POSIXSEM_IS_GLOBAL -#undef SYSVSEM_IS_GLOBAL -#undef FCNTL_IS_GLOBAL -#undef FLOCK_IS_GLOBAL - -#undef READDIR_IS_THREAD_SAFE -#undef GETHOSTBYNAME_IS_THREAD_SAFE -#undef GETHOSTBYADDR_IS_THREAD_SAFE -#undef STRERROR_R_RC_INT - -#undef HAVE_GMTOFF -#undef USE_THREADS - -#undef DSO_USE_DLFCN -#undef DSO_USE_SHL -#undef DSO_USE_DYLD - -#undef SIZEOF_SSIZE_T -#undef SIZEOF_SIZE_T -#undef SIZEOF_OFF_T -#undef SIZEOF_PID_T - -#undef HAVE_INT64_C - -#undef HAVE_VLA - -/* BeOS specific flag */ -#undef HAVE_BONE_VERSION - -/* Does this system have a corkable TCP? */ -#undef HAVE_TCP_CORK -#undef HAVE_TCP_NOPUSH @BOTTOM@