If --enable-suexec is specified, then we need to build suexec

automatically.  This bit me multiple times last night as I was trying to
setup apache.org.  I would make a change to the configuration, run make;
make install, and it would fail.  The problem was that suexec wasn't
getting re-compiled ever, so my changes weren't getting picked up.  This
solves that problem, by making sure that suexec gets compiled if it was
specified on the configure line.

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

diff --git a/CHANGES b/CHANGES
index 3f8fa3e9f906a81734633e30e3d29e83d3ef33de..5ef6342f0e030ea43d6660395b789a85fe08c340 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,5 +1,10 @@
 Changes with Apache 2.0b1
 
+  *) When SuExec is specified, we need to add it to the list of
+     targets to be built.  If we don't, then any changes to the
+     configuration won't affect SuExec, unless 'make suexec' is
+     specifically run.  [Ryan Bloom]
+
   *) Cleaned out open_file from mod_file_cache, as apr now accepts
      the APR_XTHREAD argument to open a file for consumption by
      parallel threads on win32.  [William Rowe]

diff --git a/Makefile.in b/Makefile.in
index 1da5a0fbecad20c4add0c7107a22e045e21f29ff..76fa470af35d47e0e24e46d5b8388430964e55a2 100644 (file)
--- a/Makefile.in
+++ b/Makefile.in
@@ -12,7 +12,7 @@ PROGRAM_DEPENDENCIES = \
   $(AP_LIBS)
 
 PROGRAMS        = $(PROGRAM_NAME)
-targets         = $(PROGRAMS) 
+targets         = $(PROGRAMS) $(other_targets)
 phony_targets   = $(srcdir)/buildmark.c
 install_targets = install-conf install-htdocs install-icons install-other \
        install-cgi install-include install-support install-suexec

diff --git a/acinclude.m4 b/acinclude.m4
index 3a12a9d66166ebb7b54b0ef7d13ccedbf1941e13..1b9413408677c9d5f113e618579794f5e5f2a618 100644 (file)
--- a/acinclude.m4
+++ b/acinclude.m4
@@ -64,6 +64,7 @@ AC_DEFUN(APACHE_GEN_CONFIG_VARS,[
   APACHE_SUBST(includedir)
   APACHE_SUBST(iconsdir)
   APACHE_SUBST(sysconfdir)
+  APACHE_SUBST(other_targets)
   APACHE_SUBST(progname)
   APACHE_SUBST(prefix)
   APACHE_SUBST(AWK)

diff --git a/modules/generators/config5.m4 b/modules/generators/config5.m4
index 2a1ed8ff27879f29f50cdefe18dbc769cc4c9d04..5e9f45ec13f22a13a136f336bb325e4454e40684 100644 (file)
--- a/modules/generators/config5.m4
+++ b/modules/generators/config5.m4
@@ -8,7 +8,8 @@ APACHE_MODULE(status, process/thread monitoring, , , no)
 APACHE_MODULE(autoindex, directory listing, , , yes)
 APACHE_MODULE(asis, as-is filetypes, , , yes)
 APACHE_MODULE(info, server information, , , no)
-APACHE_MODULE(suexec, set uid and gid for spawned processes, , , no)
+APACHE_MODULE(suexec, set uid and gid for spawned processes, , , no, [
+              other_targets=suexec ] )
 
 LTFLAGS="$LTFLAGS -export-dynamic"