jim 99/01/15 12:06:59
Modified: . Makefile.tmpl configure
Log:
Some configure cleanups and the first attempts at some sort of
universal style.
First of all, we remove the hard-wire of TAR=tar and check to see if
the tar we wind up with supports '-h'.
Secondly, when we do shell string tests, [ ".$var" != .yes ] looks better
when checking for the string "yes", I think it's better if, when checking
for/against a NULL string, we use 'x', eg:
[ "x$var" != x ]
I'll start adjusting the other shell scripts as well.
Revision Changes Path
1.63 +4 -3 apache-1.3/Makefile.tmpl
Index: Makefile.tmpl
===================================================================
RCS file: /export/home/cvs/apache-1.3/Makefile.tmpl,v
retrieving revision 1.62
retrieving revision 1.63
diff -u -r1.62 -r1.63
--- Makefile.tmpl 1999/01/07 11:50:45 1.62
+++ Makefile.tmpl 1999/01/15 20:06:56 1.63
@@ -79,7 +79,6 @@
CP = cp
LN = ln
RM = rm -f
-TAR = tar
MKDIR = $(TOP)/$(AUX)/mkdir.sh
INSTALL = $(TOP)/$(AUX)/install.sh -c
INSTALL_PROGRAM = $(INSTALL) -s -m 755
@@ -87,6 +86,8 @@
INSTALL_SCRIPT = $(INSTALL) -m 755
INSTALL_DATA = $(INSTALL) -m 644
PERL = @PERL@
+TAR = @TAR@
+TAROPT = @TAROPT@
# installation name of Apache webserver
TARGET = @TARGET@
@@ -367,7 +368,7 @@
echo "[PRESERVING EXISTING DATA SUBDIR:
$(root)$(datadir)/htdocs/]"; \
else \
echo "Copying tree $(TOP)/htdocs/ ->
$(root)$(datadir)/htdocs/"; \
- (cd $(TOP)/htdocs/ && $(TAR) -hcf - *) |\
+ (cd $(TOP)/htdocs/ && $(TAR) $(TAROPT) - *) |\
(cd $(root)$(datadir)/htdocs/ && $(TAR) -xf -); \
find $(root)$(datadir)/htdocs/ -type d -exec chmod a+rx {} \; ;
\
find $(root)$(datadir)/htdocs/ -type f -exec chmod a+r {} \; ; \
@@ -384,7 +385,7 @@
done; \
fi
@echo "Copying tree $(TOP)/icons/ -> $(root)$(datadir)/icons/"; \
- (cd $(TOP)/icons/ && $(TAR) -hcf - *) |\
+ (cd $(TOP)/icons/ && $(TAR) $(TAROPT) - *) |\
(cd $(root)$(datadir)/icons/ && $(TAR) -xf -); \
find $(root)$(datadir)/icons/ -type d -exec chmod a+rx {} \; ;\
find $(root)$(datadir)/icons/ -type f -exec chmod a+r {} \;
1.68 +43 -20 apache-1.3/configure
Index: configure
===================================================================
RCS file: /export/home/cvs/apache-1.3/configure,v
retrieving revision 1.67
retrieving revision 1.68
diff -u -r1.67 -r1.68
--- configure 1999/01/02 13:19:41 1.67
+++ configure 1999/01/15 20:06:57 1.68
@@ -119,7 +119,7 @@
##
PERL=no-perl-on-this-system
perlpath="`$aux/PrintPath perl5 perl miniperl`"
-if [ ".$perlpath" != . ]; then
+if [ "x$perlpath" != x ]; then
PERL="$perlpath"
fi
@@ -152,11 +152,32 @@
##
AWK=awk
awkpath="`$aux/PrintPath nawk gawk awk`"
-if [ ".$awkpath" != . ]; then
+if [ "x$awkpath" != x ]; then
AWK="$awkpath"
fi
##
+## Look for a good tar. If we don't find 'GNU tar' then make
+## sure ours can handle the '-h' (don't copy symlink, copy
+## the actual data) option.
+##
+TAR=tar
+tarpath="`$aux/PrintPath gtar gnutar tar`"
+if [ "x$awkpath" != x ]; then
+ TAR="$tarpath"
+fi
+case "`$TAR --version 2>/dev/null`" in
+ *GNU*) TAROPT="-hcf" ;;
+ *) if $TAR -hcf - Makefile.tmpl > /dev/null 2>&1
+ then
+ TAROPT="-hcf"
+ else
+ TAROPT="-cf"
+ fi
+ ;;
+esac
+
+##
## determine default parameters
##
@@ -237,7 +258,7 @@
OIFS="$IFS" IFS="$DIFS"
for var in CFLAGS LDFLAGS LIBS INCLUDES DEPS; do
eval "val=\$EXTRA_$var"
- if [ ".$val" != . ]; then
+ if [ "x$val" != x ]; then
eval "$var=\$val"
eval "EXTRA_$var=\"\"; export EXTRA_$var"
echo " + Hint: please use $var instead of EXTRA_$var next time"
@@ -252,7 +273,7 @@
*--with-layout=* )
;;
* )
- if [ ".$*" = . ]; then
+ if [ "x$*" = x ]; then
set -- '--with-layout=Apache'
else
set -- '--with-layout=Apache' "$@"
@@ -272,7 +293,7 @@
for apc_option
do
# if previous option needs an argument, assign it.
- if [ ".$apc_prev" != . ]; then
+ if [ "x$apc_prev" != x ]; then
eval "$apc_prev=\$apc_option"
apc_prev=""
continue
@@ -412,7 +433,7 @@
for apc_option
do
# if previous option needs an argument, assign it.
- if [ ".$apc_prev" != . ]; then
+ if [ "x$apc_prev" != x ]; then
eval "$apc_prev=\$apc_option"
apc_prev=""
continue
@@ -465,7 +486,7 @@
echo "## Apache Makefile (shadow wrapper)" >> Makefile
echo "##" >> Makefile
echo "" >> Makefile
- if [ ".$shadow" != . ]; then
+ if [ "x$shadow" != x ]; then
echo "SHADOW=$shadow" >> Makefile
else
echo "SHADOW=." >> Makefile
@@ -635,7 +656,7 @@
apc_optarg=`echo "$apc_optarg" | tr "A-Z" "a-z"`
apc_optarg_real=`echo "$apc_optarg" | tr "a-z" "A-Z"`
eval "exists=\$rule_${apc_optarg}"
- if [ ".$exists" = . ]; then
+ if [ "x$exists" = x ]; then
echo "configure:Error: No such rule named
'${apc_optarg_real}'" 1>&2
exit 1
fi
@@ -665,7 +686,7 @@
;;
* )
eval "exists=\$module_${apc_optarg}"
- if [ ".$exists" = . ]; then
+ if [ "x$exists" = x ]; then
echo "configure:Error: No such module named
'${apc_optarg}'" 1>&2
exit 1
fi
@@ -697,7 +718,7 @@
;;
* )
eval "exists=\$module_${apc_optarg}"
- if [ ".$exists" = . ]; then
+ if [ "x$exists" = x ]; then
echo "configure:Error: No such module named
'${apc_optarg}'" 1>&2
exit 1
fi
@@ -722,7 +743,7 @@
apc_optarg=`echo "$apc_optarg" | tr "A-Z" "a-z"`
apc_optarg_real=`echo "$apc_optarg" | tr "a-z" "A-Z"`
eval "exists=\$rule_${apc_optarg}"
- if [ ".$exists" = . ]; then
+ if [ "x$exists" = x ]; then
echo "configure:Error: No such rule named
'${apc_optarg_real}'" 1>&2
exit 1
fi
@@ -739,7 +760,7 @@
;;
* )
eval "exists=\$module_${apc_optarg}"
- if [ ".$exists" = . ]; then
+ if [ "x$exists" = x ]; then
echo "configure:Error: No such module named
'${apc_optarg}'" 1>&2
exit 1
fi
@@ -758,7 +779,7 @@
;;
* )
eval "exists=\$module_${apc_optarg}"
- if [ ".$exists" = . ]; then
+ if [ "x$exists" = x ]; then
echo "configure:Error: No such module named
'${apc_optarg}'" 1>&2
exit 1
fi
@@ -776,7 +797,7 @@
BEGIN|END)
;;
*) eval "exists=\$module_${mod}"
- if [ ".$exists" = . ]; then
+ if [ "x$exists" = x ]; then
echo "configure:Error: No such module named
'${mod}'" 1>&2
exit 1
fi
@@ -835,7 +856,7 @@
esac
done
IFS="$OIFS1"
-if [ ".$apc_prev" != . ]; then
+if [ "x$apc_prev" != x ]; then
echo "configure:Error: missing argument to --`echo $apc_prev | sed
's/_/-/g'`" 1>&2
exit 1
fi
@@ -860,7 +881,7 @@
for var in CC CPP OPTIM CFLAGS CFLAGS_SHLIB LDFLAGS LD_SHLIB LDFLAGS_SHLIB \
LDFLAGS_SHLIB_EXPORT LIBS INCLUDES RANLIB DEPS TARGET; do
eval "val=\"\$$var\""
- if [ ".$val" != . ]; then
+ if [ "x$val" != x ]; then
echo "$var=$val" |\
sed -e 's:\(["$\\]\):\\\1:g' \
-e 's:\([A-Z]*=\):\1":' \
@@ -906,7 +927,7 @@
##
## target name
##
-if [ ".$TARGET" != . ]; then
+if [ "x$TARGET" != x ]; then
thetarget=$TARGET
else
thetarget=httpd
@@ -1042,6 +1063,8 @@
fi
sed <Makefile.tmpl >$mkf \
-e "[EMAIL PROTECTED]@%$PERL%g" \
+-e "[EMAIL PROTECTED]@%$TAR%g" \
+-e "[EMAIL PROTECTED]@%$TAROPT%g" \
-e "[EMAIL PROTECTED]@%$src%g" \
-e "[EMAIL PROTECTED]@%$mkf%g" \
-e "[EMAIL PROTECTED]@%$aux%g" \
@@ -1114,7 +1137,7 @@
for var in CC CPP OPTIM CFLAGS CFLAGS_SHLIB LDFLAGS LD_SHLIB LDFLAGS_SHLIB \
LDFLAGS_SHLIB_EXPORT LIBS INCLUDES RANLIB DEPS TARGET; do
eval "val=\"\$$var\"";
- if [ ".$val" != . ]; then
+ if [ "x$val" != x ]; then
case $var in
CFLAGS|LDFLAGS|LIBS|INCLUDES|DEPS)
echo $SEO "s%^#*\\(EXTRA_$var=\\).*%\\1$val%g" >>$sedsubst
@@ -1165,7 +1188,7 @@
fi
# module permutation support
-if [ ".$permute" != . ]; then
+if [ "x$permute" != x ]; then
sed -e '/## mod_mmap_static/,$d' <src/Configuration.tmpl >$tplconf
OIFS="$IFS" IFS='
'
@@ -1308,7 +1331,7 @@
## final hints
##
if [ .$quiet = .no ]; then
- if [ ".$shadow" != . ]; then
+ if [ "x$shadow" != x ]; then
echo "Hint: You now have to build inside $shadow."
echo "This can be done either by running the canonical commands"
echo " \$ cd $shadow"