From: Ryan Bloom <rbb@apache.org>
Date: Sun, 21 Jan 2001 17:22:18 +0000 (+0000)
Subject: If --enable-suexec is specified, then we need to build suexec
X-Git-Tag: APACHE_2_0_BETA_CANDIDATE_1~139
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=8349a6f08558b0a4e8070e23126cd74c588c21d3;p=apache

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 3f8fa3e9f9..5ef6342f0e 100644
--- 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 1da5a0fbec..76fa470af3 100644
--- 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 3a12a9d661..1b94134086 100644
--- 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 2a1ed8ff27..5e9f45ec13 100644
--- 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"