You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by sf...@apache.org on 2012/10/07 10:56:45 UTC

svn commit: r1395252 - in /httpd/httpd/trunk: acinclude.m4 server/mpm/event/config3.m4 server/mpm/mpmt_os2/config5.m4 server/mpm/prefork/config3.m4 server/mpm/simple/config3.m4 server/mpm/winnt/config3.m4 server/mpm/worker/config3.m4

Author: sf
Date: Sun Oct  7 08:56:45 2012
New Revision: 1395252

URL: http://svn.apache.org/viewvc?rev=1395252&view=rev
Log:
Fix usage of MOD_* variables for MPM modules

r1385216 broke building of mpm event with serf support. To fix,

* introduce APACHE_MPMPATH_INIT/FINISH similar to
  APACHE_MODPATH_INIT/FINISH to create modules.mk and add
  MOD_* vars to it.
* change APACHE_MPM_MODULE to only append to modules.mk, not
  overwrite
* add APACHE_MPMPATH_INIT/FINISH to all MPMs

Modified:
    httpd/httpd/trunk/acinclude.m4
    httpd/httpd/trunk/server/mpm/event/config3.m4
    httpd/httpd/trunk/server/mpm/mpmt_os2/config5.m4
    httpd/httpd/trunk/server/mpm/prefork/config3.m4
    httpd/httpd/trunk/server/mpm/simple/config3.m4
    httpd/httpd/trunk/server/mpm/winnt/config3.m4
    httpd/httpd/trunk/server/mpm/worker/config3.m4

Modified: httpd/httpd/trunk/acinclude.m4
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/acinclude.m4?rev=1395252&r1=1395251&r2=1395252&view=diff
==============================================================================
--- httpd/httpd/trunk/acinclude.m4 (original)
+++ httpd/httpd/trunk/acinclude.m4 Sun Oct  7 08:56:45 2012
@@ -217,6 +217,27 @@ EOF
     fi
   fi
 ])dnl
+dnl Same as APACHE_MODPATH_INIT/FINISH but for MPMs
+dnl APACHE_MPMPATH_INIT(mpmpath)
+AC_DEFUN(APACHE_MPMPATH_INIT,[
+  current_dir=$1
+  modpath_current=server/mpm/$1
+  modpath_static=
+  modpath_shared=
+  for var in mod_buildvars; do
+    eval MOD_$var=
+  done
+  test -d $1 || $srcdir/build/mkdir.sh $modpath_current
+  > $modpath_current/modules.mk
+])dnl
+dnl
+AC_DEFUN(APACHE_MPMPATH_FINISH,[
+  for var in mod_buildvars; do
+    if eval val=\"\$MOD_$var\"; test -n "$val"; then
+      echo "MOD_$var = $val" >> $modpath_current/modules.mk
+    fi
+  done
+])dnl
 
 dnl
 dnl APACHE_MPM_MODULE(name[, shared[, objects[, config[, path[, libs]]]]])
@@ -253,7 +274,7 @@ AC_DEFUN(APACHE_MPM_MODULE,[
         if test -z "$2"; then
             APR_ADDTO(AP_LIBS, [$6])
             libname="lib$1.la"
-            cat >$mpmpath/modules.mk<<EOF
+            cat >>$mpmpath/modules.mk<<EOF
 $libname: $objects
 	\$(MOD_LINK) $objects
 DISTCLEAN_TARGETS = modules.mk
@@ -264,7 +285,7 @@ EOF
             apache_need_shared=yes
             libname="mod_mpm_$1.la"
             shobjects=`echo $objects | sed 's/\.lo/.slo/g'`
-            cat >$mpmpath/modules.mk<<EOF
+            cat >>$mpmpath/modules.mk<<EOF
 $libname: $shobjects
 	\$(SH_LINK) -rpath \$(libexecdir) -module -avoid-version $objects $6
 DISTCLEAN_TARGETS = modules.mk

Modified: httpd/httpd/trunk/server/mpm/event/config3.m4
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/mpm/event/config3.m4?rev=1395252&r1=1395251&r2=1395252&view=diff
==============================================================================
--- httpd/httpd/trunk/server/mpm/event/config3.m4 (original)
+++ httpd/httpd/trunk/server/mpm/event/config3.m4 Sun Oct  7 08:56:45 2012
@@ -1,3 +1,5 @@
+APACHE_MPMPATH_INIT(event)
+
 dnl ## XXX - Need a more thorough check of the proper flags to use
 
 APACHE_CHECK_SERF
@@ -9,3 +11,5 @@ APACHE_SUBST(MOD_MPM_EVENT_LDADD)
 APACHE_MPM_MODULE(event, $enable_mpm_event, event.lo fdqueue.lo pod.lo,[
     AC_CHECK_FUNCS(pthread_kill)
 ], , [\$(MOD_MPM_EVENT_LDADD)])
+
+APACHE_MPMPATH_FINISH

Modified: httpd/httpd/trunk/server/mpm/mpmt_os2/config5.m4
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/mpm/mpmt_os2/config5.m4?rev=1395252&r1=1395251&r2=1395252&view=diff
==============================================================================
--- httpd/httpd/trunk/server/mpm/mpmt_os2/config5.m4 (original)
+++ httpd/httpd/trunk/server/mpm/mpmt_os2/config5.m4 Sun Oct  7 08:56:45 2012
@@ -1,3 +1,7 @@
+APACHE_MPMPATH_INIT(mpmt_os2)
+
 APACHE_MPM_MODULE(mpmt_os2, $enable_mpm_mpmt_os2, mpmt_os2.lo mpmt_os2_child.lo,[
     APR_ADDTO(CFLAGS,-Zmt)
 ])
+
+APACHE_MPMPATH_FINISH

Modified: httpd/httpd/trunk/server/mpm/prefork/config3.m4
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/mpm/prefork/config3.m4?rev=1395252&r1=1395251&r2=1395252&view=diff
==============================================================================
--- httpd/httpd/trunk/server/mpm/prefork/config3.m4 (original)
+++ httpd/httpd/trunk/server/mpm/prefork/config3.m4 Sun Oct  7 08:56:45 2012
@@ -1 +1,5 @@
+APACHE_MPMPATH_INIT(prefork)
+
 APACHE_MPM_MODULE(prefork, $enable_mpm_prefork)
+
+APACHE_MPMPATH_FINISH

Modified: httpd/httpd/trunk/server/mpm/simple/config3.m4
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/mpm/simple/config3.m4?rev=1395252&r1=1395251&r2=1395252&view=diff
==============================================================================
--- httpd/httpd/trunk/server/mpm/simple/config3.m4 (original)
+++ httpd/httpd/trunk/server/mpm/simple/config3.m4 Sun Oct  7 08:56:45 2012
@@ -1,3 +1,7 @@
+APACHE_MPMPATH_INIT(simple)
+
 simple_objects="simple_api.lo simple_children.lo simple_core.lo \
 simple_event.lo simple_run.lo simple_io.lo"
 APACHE_MPM_MODULE(simple, $enable_mpm_simple, $simple_objects)
+
+APACHE_MPMPATH_FINISH

Modified: httpd/httpd/trunk/server/mpm/winnt/config3.m4
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/mpm/winnt/config3.m4?rev=1395252&r1=1395251&r2=1395252&view=diff
==============================================================================
--- httpd/httpd/trunk/server/mpm/winnt/config3.m4 (original)
+++ httpd/httpd/trunk/server/mpm/winnt/config3.m4 Sun Oct  7 08:56:45 2012
@@ -1,2 +1,6 @@
+APACHE_MPMPATH_INIT(winnt)
+
 winnt_objects="child.lo mpm_winnt.lo nt_eventlog.lo service.lo"
 APACHE_MPM_MODULE(winnt, $enable_mpm_winnt, $winnt_objects)
+
+APACHE_MPMPATH_FINISH

Modified: httpd/httpd/trunk/server/mpm/worker/config3.m4
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/mpm/worker/config3.m4?rev=1395252&r1=1395251&r2=1395252&view=diff
==============================================================================
--- httpd/httpd/trunk/server/mpm/worker/config3.m4 (original)
+++ httpd/httpd/trunk/server/mpm/worker/config3.m4 Sun Oct  7 08:56:45 2012
@@ -1,5 +1,8 @@
-dnl ## XXX - Need a more thorough check of the proper flags to use
+APACHE_MPMPATH_INIT(worker)
 
+dnl ## XXX - Need a more thorough check of the proper flags to use
 APACHE_MPM_MODULE(worker, $enable_mpm_worker, worker.lo fdqueue.lo pod.lo,[
     AC_CHECK_FUNCS(pthread_kill)
 ])
+
+APACHE_MPMPATH_FINISH