]> granicus.if.org Git - postgresql/commitdiff
Make unknown-option-warning code more portable. echo -n is not portable,
authorPeter Eisentraut <peter_e@gmx.net>
Fri, 13 Oct 2006 20:23:07 +0000 (20:23 +0000)
committerPeter Eisentraut <peter_e@gmx.net>
Fri, 13 Oct 2006 20:23:07 +0000 (20:23 +0000)
and neither is "|" or "\|" in basic regular expressions.

config/general.m4
configure

index 62d97e120e551bbe9bdbc550f0e2e7f94a2df05c..8139f0ae3f66f7087b12da30c6e9713db4c51242 100644 (file)
@@ -1,4 +1,4 @@
-# $PostgreSQL: pgsql/config/general.m4,v 1.5 2006/08/11 18:51:50 alvherre Exp $
+# $PostgreSQL: pgsql/config/general.m4,v 1.6 2006/10/13 20:23:07 petere Exp $
 
 # This file defines new macros to process configure command line
 # arguments, to replace the brain-dead AC_ARG_WITH and AC_ARG_ENABLE.
@@ -75,20 +75,20 @@ AC_ARG_WITH([$2], [$3], [
 # PGAC_CHECK_ARGS()
 # -----------------
 # Checks if the user passed any --with/without/enable/disable arguments that
-# we don't recognise. Just prints out a warning message, so this should be
+# we don't recognize. Just prints out a warning message, so this should be
 # called near the end, so the user will see it.
 
 AC_DEFUN([PGAC_CHECK_ARGS],
 [
-  for var in `set |sed -ne '/^\(with_\|enable\_\)/ s/=.*//p'` ; do
-     case $var in
-       m4_undivert([PGAC_ARGS])
-       with_gnu_ld) ;;
-     *)
-       echo -n "*** Option ignored: "
-       echo $var | sed -e 's/\([^=]*\)/--\1/;s/_/-/g'
-       ;;
-     esac
+  for pgac_var in `set | sed 's/=.*//' | $EGREP 'with_|enable_'`; do
+    case $pgac_var in
+      m4_undivert([PGAC_ARGS])
+      with_gnu_ld) ;;
+    *)
+      pgac_txt=`echo $pgac_var | tr '_' '-'`
+      AC_MSG_WARN([option ignored: --$pgac_txt])
+      ;;
+    esac
   done
 ])# PGAC_CHECK_ARGS
 
index f01b03b15c81fe3085e9e7d2e148c109620f7466..3a826db3649ddecce7c703aabcf51be95c89fdae 100755 (executable)
--- a/configure
+++ b/configure
@@ -22980,9 +22980,9 @@ fi
 #
 
 
-  for var in `set |sed -ne '/^\(with_\|enable\_\)/ s/=.*//p'` ; do
-     case $var in
-       with_template) ;;
+  for pgac_var in `set | sed 's/=.*//' | $EGREP 'with_|enable_'`; do
+    case $pgac_var in
+      with_template) ;;
 with_docdir) ;;
 with_includes) ;;
 with_libraries) ;;
@@ -23014,12 +23014,13 @@ with_readline) ;;
 with_libedit_preferred) ;;
 with_zlib) ;;
 
-       with_gnu_ld) ;;
-     *)
-       echo -n "*** Option ignored: "
-       echo $var | sed -e 's/\(^=*\)/--\1/;s/_/-/g'
-       ;;
-     esac
+      with_gnu_ld) ;;
+    *)
+      pgac_txt=`echo $pgac_var | tr '_' '-'`
+      { echo "$as_me:$LINENO: WARNING: option ignored: --$pgac_txt" >&5
+echo "$as_me: WARNING: option ignored: --$pgac_txt" >&2;}
+      ;;
+    esac
   done