More packaging fixups from maintainer.

PR:		ports/17139
Submitted by:	maintainer

4 files changed, 91 insertions, 56 deletions

diff --git a/www/apache13-fp/Makefile b/www/apache13-fp/Makefile
index fb3493f01e47..3a53c927fb69 100644
--- a/www/apache13-fp/Makefile
+++ b/www/apache13-fp/Makefile
@@ -54,7 +54,6 @@ AP_SHARE=	${PREFIX}/share/doc/apache
 MOD_FPDOCDIR=	${AP_SHARE}/manual/mod/mod_frontpage
 MOD_FP=		${FILESDIR}/mod_frontpage.c
 IMAGES_DIR=	${AP_SHARE}/manual/images
-IMAGES_VTI=	${PREFIX}/www/data/images/_vti_cnf
 
 INSTALL_FILE=	${INSTALL} -c -m 555 -o bin -g bin
 
@@ -166,13 +165,7 @@ post-install:
 	@${INSTALL} -c -m 644 ${DISTDIR}/powerlogo.gif ${IMAGES_DIR}
 	@${INSTALL} -c -m 644 ${DISTDIR}/fplogo.gif ${IMAGES_DIR}
 	@${INSTALL} -c -m 644 ${AP_SHARE}/apache_pb.gif ${IMAGES_DIR}
-	@mkdir -p ${PREFIX}/www/data/images
 	@( cd ${PREFIX}/share/doc/apache ; if [ -h images ] ; then ${RM} -f images ; fi ; ln -fs manual/images images)
-.for file in ${IMAGES}
-	@if [ ! -f ${PREFIX}/www/data/images/${file} ]; then \
-		${CP} -rp ${IMAGES_DIR}/${file} ${PREFIX}/www/data/images ; \
-	fi
-.endfor
 	# Untar frontpage extentions
 	@(cd ${PREFIX} && ${EXTRACT_CMD} ${EXTRACT_BEFORE_ARGS} ${DISTDIR}/${FRONTPAGE} ${EXTRACT_AFTER_ARGS})
 	@${RM} ${FPHTTPD}/httpd ${FPHTTPD}/httpd.Compat
diff --git a/www/apache13-fp/files/patch-ab b/www/apache13-fp/files/patch-ab
index c45e9fe7b6e6..270b7073b66e 100644
--- a/www/apache13-fp/files/patch-ab
+++ b/www/apache13-fp/files/patch-ab
@@ -1,5 +1,5 @@
 --- Makefile.tmpl.orig	Tue Jan 11 13:47:41 2000
-+++ Makefile.tmpl	Thu Feb 24 20:58:51 2000
++++ Makefile.tmpl	Thu Mar  2 23:00:19 2000
 @@ -123,6 +123,7 @@
  runtimedir      = @runtimedir@
  logfiledir      = @logfiledir@
@@ -68,7 +68,7 @@
  		< $(TOP)/$(SRC)/support/apachectl > $(TOP)/$(SRC)/.apaci.install.tmp && \
  		$(INSTALL_SCRIPT) $(TOP)/$(SRC)/.apaci.install.tmp $(root)$(sbindir)/$${apachectl}; \
  	echo "$(INSTALL_DATA) $(TOP)/$(SRC)/support/apachectl.8 $(root)$(mandir)/man8/$${apachectl}.8"; \
-@@ -452,25 +469,31 @@
+@@ -452,26 +469,30 @@
  #   icons and distributed CGI scripts.
  install-data:
  	@echo "===> [data: Installing initial data files]"
@@ -108,12 +108,12 @@
 +			echo "$(INSTALL_DATA) $(TOP)/conf/$${script}[*] $(root)$(cgidir).default/$${script}"; \
 +			$(INSTALL_DATA) $(TOP)/$(SRC)/.apaci.install.tmp $(root)$(cgidir).default/$${script}; \
  		done; \
-+	if [ ! -d $(root)$(cgidir) ]; then \
-+		$(CP) -rp $(root)$(cgidir).default $(root)$(cgidir); \
- 	fi
+-	fi
++#	fi
  	@echo "Copying tree $(TOP)/icons/ -> $(root)$(iconsdir)/"; \
  	(cd $(TOP)/icons/ && $(TAR) $(TAROPT) - *) |\
-@@ -504,28 +527,33 @@
+ 	(cd $(root)$(iconsdir)/ && $(TAR) -xf -); \
+@@ -504,42 +525,47 @@
  			-e 's;@@ServerRoot@@/icons;$(iconsdir);' \
  			-e 's;@@ServerRoot@@/cgi-bin;$(cgidir);' \
  			-e 's;@@ServerRoot@@/proxy;$(proxycachedir);' \
@@ -143,11 +143,37 @@
  			> $(TOP)/$(SRC)/.apaci.install.tmp && \
  		echo "$(INSTALL_DATA) $(TOP)/conf/$${conf}-dist[*] $(root)$(sysconfdir)/$${target_conf}.default"; \
  		$(INSTALL_DATA) $(TOP)/$(SRC)/.apaci.install.tmp $(root)$(sysconfdir)/$${target_conf}.default; \
- 		if [ ! -f "$(root)$(sysconfdir)/$${target_conf}" ]; then \
-+		    if [ .$${target_conf} != .httpd.conf ] ; then \
- 			echo "$(INSTALL_DATA) $(TOP)/conf/$${conf}-dist[*] $(root)$(sysconfdir)/$${target_conf}"; \
- 			$(INSTALL_DATA) $(TOP)/$(SRC)/.apaci.install.tmp $(root)$(sysconfdir)/$${target_conf}; \
-+		    fi; \
- 		else \
- 			echo "[PRESERVING EXISTING CONFIG FILE: $(root)$(sysconfdir)/$${target_conf}]"; \
- 		fi; \
+-		if [ ! -f "$(root)$(sysconfdir)/$${target_conf}" ]; then \
+-			echo "$(INSTALL_DATA) $(TOP)/conf/$${conf}-dist[*] $(root)$(sysconfdir)/$${target_conf}"; \
+-			$(INSTALL_DATA) $(TOP)/$(SRC)/.apaci.install.tmp $(root)$(sysconfdir)/$${target_conf}; \
+-		else \
+-			echo "[PRESERVING EXISTING CONFIG FILE: $(root)$(sysconfdir)/$${target_conf}]"; \
+-		fi; \
+ 	done
++#		if [ ! -f "$(root)$(sysconfdir)/$${target_conf}" ]; then \
++#			echo "$(INSTALL_DATA) $(TOP)/conf/$${conf}-dist[*] $(root)$(sysconfdir)/$${target_conf}"; \
++#			$(INSTALL_DATA) $(TOP)/$(SRC)/.apaci.install.tmp $(root)$(sysconfdir)/$${target_conf}; \
++#		else \
++#			echo "[PRESERVING EXISTING CONFIG FILE: $(root)$(sysconfdir)/$${target_conf}]"; \
++#		fi; \
++#	done
+ 	-@for conf in mime.types magic; do \
+ 		echo "$(INSTALL_DATA) $(TOP)/conf/$${conf} $(root)$(sysconfdir)/$${conf}.default"; \
+ 		$(INSTALL_DATA) $(TOP)/conf/$${conf} $(root)$(sysconfdir)/$${conf}.default; \
+-		if [ ! -f "$(root)$(sysconfdir)/$${conf}" ]; then \
+-			echo "$(INSTALL_DATA) $(TOP)/conf/$${conf} $(root)$(sysconfdir)/$${conf}"; \
+-			$(INSTALL_DATA) $(TOP)/conf/$${conf} $(root)$(sysconfdir)/$${conf}; \
+-		else \
+-			echo "[PRESERVING EXISTING CONFIG FILE: $(root)$(sysconfdir)/$${conf}]"; \
+-		fi; \
+ 	done
++#		if [ ! -f "$(root)$(sysconfdir)/$${conf}" ]; then \
++#			echo "$(INSTALL_DATA) $(TOP)/conf/$${conf} $(root)$(sysconfdir)/$${conf}"; \
++#			$(INSTALL_DATA) $(TOP)/conf/$${conf} $(root)$(sysconfdir)/$${conf}; \
++#		else \
++#			echo "[PRESERVING EXISTING CONFIG FILE: $(root)$(sysconfdir)/$${conf}]"; \
++#		fi; \
++#	done
+ 	@echo "<=== [config]"
+ 
+ 
diff --git a/www/apache13-fp/pkg-install b/www/apache13-fp/pkg-install
index d2b43b878d20..943c41d4863f 100644
--- a/www/apache13-fp/pkg-install
+++ b/www/apache13-fp/pkg-install
@@ -7,35 +7,50 @@
 PKG_BATCH=${BATCH:=NO}
 
 PKG_PREFIX=${PKG_PREFIX}
-AP_SHARE=${PKG_PREFIX}/share/doc/apache
 HOST_NAME=`/bin/hostname`
+
+AP_CGI=${PKG_PREFIX}/www/cgi-bin
+AP_CONF=${PKG_PREFIX}/etc/apache
 AP_DATA=${PKG_PREFIX}/www/data
+AP_SHARE=${PKG_PREFIX}/share/doc/apache
+
 FPINSTALL=${PKG_PREFIX}/frontpage/version4.0/fp_install.sh
 CHANGESERVER=${PKG_PREFIX}/frontpage/version4.0/change_server.sh
 FPDOCDIR=${AP_SHARE}/manual/mod/mod_frontpage
+IMAGES_DIR=${AP_SHARE}/manual/images
 IMAGES_VTI=${PKG_PREFIX}/www/data/images/_vti_cnf
-HTTPD_CONF=${PKG_PREFIX}/etc/apache/httpd.conf
 
 create_apache_lang_doc ()
 {
-    if [ ! -d ${AP_DATA} ]; then
-	/bin/mkdir -p ${AP_DATA}
-    fi
-
     for lang in ca cz de dk ee en es fr it ja.jis lu nl po.iso-pl pt pt-br se
     {
 	/bin/cat ${AP_SHARE}/index.html.${lang}-dist | \
-	    /usr/bin/sed -e 's;@@HOSTNAME@@;'${HOST_NAME}';' \
-	    > ${AP_SHARE}/index.html.${lang}
-	if [ ! -f ${AP_DATA}/index.html ] ; then
-	    if [ ! -f ${AP_DATA}/index.html.${lang} ]; then
-		/bin/cp -rp ${AP_SHARE}/index.html.${lang} \
-		    ${AP_DATA}/index.html.${lang}
-	    fi
-	fi
+	   /usr/bin/sed -e 's;@@HOSTNAME@@;'${HOST_NAME}';' \
+	   > ${AP_SHARE}/index.html.${lang}
     }
 }
 
+create_apache_doc_root ()
+{
+    if [ ! -d ${AP_CGI} ]; then
+                /bin/cp -rp ${AP_CGI}.default ${AP_CGI}
+    fi
+
+    if [ ! -d ${AP_DATA} ]; then
+	/bin/mkdir -p ${AP_DATA}/images
+	for file in apache_pb.gif fplogo.gif powerlogo.gif
+	{
+	    /bin/cp -rp ${IMAGES_DIR}/${file} ${AP_DATA}/images
+	}
+
+	for lang in ca cz de dk ee en es fr it ja.jis lu nl po.iso-pl pt pt-br se
+	{
+	    /bin/cp -rp ${AP_SHARE}/index.html.${lang} \
+		${AP_DATA}/index.html.${lang}
+	}
+    fi
+}
+
 fix_frontpage_scripts ()
 {
     /bin/cat ${FPINSTALL}-dist | \
@@ -51,24 +66,35 @@ fix_frontpage_scripts ()
 
 fix_httpd_conf ()
 {
-    if [ ! -f ${HTTPD_CONF} ] ; then
-	/bin/cat ${HTTPD_CONF}.default | \
+    if [ ! -f ${AP_CONF}/httpd.conf} ] ; then
+	/bin/cat ${AP_CONF}/httpd.conf.default | \
 	    /usr/bin/sed -e 's;@@HOSTNAME@@;'${HOST_NAME}';' \
-	    > ${HTTPD_CONF}
+	    > ${AP_CONF}/httpd.conf
     fi
+
+    for file in mime.types magic srm.conf access.conf
+    {
+	if [ ! -f ${AP_CONF}/${file} ]; then
+	    cp -rp ${AP_CONF}/${file}.default ${AP_CONF}/${file}
+	fi
+    }
 }
+
 #Add the appropriate comment to the images/_vti_cnf file.
 comment_files ()
 {
-    if [ -d ${IMAGES_VTI} ]; then \
-	if [ ! "`grep description ${IMAGES_VTI}/apache_pb.gif`" ] ; then \
-	    /bin/echo "vti_description:SW|Apache Webserver" >> ${IMAGES_VTI}/apache_pb.gif ;\
+    if [ -d ${IMAGES_VTI} ]; then 
+	if [ -f ${IMAGES_VTI}/apache_pb.gif ] && \
+	   [ ! "`grep description ${IMAGES_VTI}/apache_pb.gif`" ] ; then
+	    /bin/echo "vti_description:SW|Apache Webserver" >> ${IMAGES_VTI}/apache_pb.gif 
 	fi
-	if [ ! "`grep description ${IMAGES_VTI}/fplogo.gif`" ] ; then \
-	    /bin/echo "vti_description:SW|Created with Microsoft FrontPage 2000" >> ${IMAGES_VTI}/fplogo.gif ; \
+	if [ -f ${IMAGES_VTI}/fplogo.gif ] && \
+	   [ ! "`grep description ${IMAGES_VTI}/fplogo.gif`" ] ; then
+	    /bin/echo "vti_description:SW|Created with Microsoft FrontPage 2000" >> ${IMAGES_VTI}/fplogo.gif 
 	fi
-	if [ ! "`grep description ${IMAGES_VTI}/powerlogo.gif`" ] ; then \
-	    /bin/echo "vti_description:SW|Powered by FreeBSD" >> ${IMAGES_VTI}/powerlogo.gif ; \
+	if [ -f ${IMAGES_VTI}/powerlogo.gif ] && \
+	   [ ! "`grep description ${IMAGES_VTI}/powerlogo.gif`" ] ; then
+	    /bin/echo "vti_description:SW|Powered by FreeBSD" >> ${IMAGES_VTI}/powerlogo.gif
 	fi
     fi
 }
@@ -80,6 +106,7 @@ case $2 in
 	# If we are not in batch mode then run the FP install script.
         if [ "${PKG_BATCH}" = "NO" ]; then
 	    create_apache_lang_doc
+	    create_apache_doc_root
 	    fix_frontpage_scripts
 	    fix_httpd_conf
 	    ${FPINSTALL}
diff --git a/www/apache13-fp/pkg-plist b/www/apache13-fp/pkg-plist
index 75083e6746f3..a4db5334e257 100644
--- a/www/apache13-fp/pkg-plist
+++ b/www/apache13-fp/pkg-plist
@@ -2,14 +2,10 @@ bin/dbmmanage
 bin/htdigest
 bin/htpasswd
 etc/apache/access.conf.default
-@exec if [ ! -f %B/access.conf ]; then cp %B/%f %B/access.conf ; fi
 etc/apache/httpd.conf.default
 etc/apache/magic.default
-@exec if [ ! -f %B/magic ]; then cp %B/%f %B/magic ; fi
 etc/apache/mime.types.default
-@exec if [ ! -f %B/mime.types ]; then cp %B/%f %B/mime.types ; fi
 etc/apache/srm.conf.default
-@exec if [ ! -f %B/srm.conf ]; then cp %B/%f %B/srm.conf ; fi
 etc/rc.d/apache.sh
 include/apache/alloc.h
 include/apache/ap.h
@@ -96,7 +92,6 @@ sbin/logresolve
 sbin/rotatelogs
 sbin/suexec
 share/doc/apache/apache_pb.gif
-@exec if [ ! -d %D/www/data/images ] ; then mkdir -p %D/www/data/images ; fi
 @exec if [ ! -h %B/images ]; then (cd %B ; ln -s manual/images images); fi
 @unexec if [ -h %B/images ]; then rm -f %B/images; fi
 share/doc/apache/index.html.ca-dist
@@ -130,10 +125,8 @@ share/doc/apache/manual/footer.html
 share/doc/apache/manual/handler.html
 share/doc/apache/manual/header.html
 share/doc/apache/manual/images/apache_pb.gif
-@exec if [ ! -f %D/www/data/images/%f ]; then cp %D/%F %D/www/data/images/%f; fi
 share/doc/apache/manual/images/custom_errordocs.gif
 share/doc/apache/manual/images/fplogo.gif
-@exec if [ ! -f %D/www/data/images/%f ]; then cp %D/%F %D/www/data/images/%f; fi
 share/doc/apache/manual/images/home.gif
 share/doc/apache/manual/images/index.gif
 share/doc/apache/manual/images/mod_rewrite_fig1.fig
@@ -141,7 +134,6 @@ share/doc/apache/manual/images/mod_rewrite_fig1.gif
 share/doc/apache/manual/images/mod_rewrite_fig2.fig
 share/doc/apache/manual/images/mod_rewrite_fig2.gif
 share/doc/apache/manual/images/powerlogo.gif
-@exec if [ ! -f %D/www/data/images/%f ]; then cp %D/%F %D/www/data/images/%f; fi
 share/doc/apache/manual/images/sub.gif
 share/doc/apache/manual/index.html
 share/doc/apache/manual/install-tpf.html
@@ -256,11 +248,8 @@ share/doc/apache/manual/vhosts/name-based.html
 share/doc/apache/manual/vhosts/vhosts-in-depth.html
 share/doc/apache/manual/vhosts/virtual-host.html
 share/doc/apache/manual/windows.html
-@exec if [ ! -d %D/www/cgi-bin ]; then mkdir -p %D/www/cgi-bin; fi
 www/cgi-bin.default/printenv
-@exec if [ ! -f %D/www/cgi-bin/%f ]; then cp %D/%F %D/www/cgi-bin/%f; fi
 www/cgi-bin.default/test-cgi
-@exec if [ ! -f %D/www/cgi-bin/%f ]; then cp %D/%F %D/www/cgi-bin/%f; fi
 www/icons/README
 www/icons/a.gif
 www/icons/alert.black.gif