[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: Argument quoting problem when processing config_subdirs
From: |
Tim Van Holder |
Subject: |
RE: Argument quoting problem when processing config_subdirs |
Date: |
Sat, 6 Jan 2001 22:38:28 +0100 |
It's not my best day, it would seem. Here is yet another revision
of my diff.
a) I was using ac_cach_file instead of ac_sub_cache_file
b) The single quotes in the case needed escaping
c) No space was guaranteed between the --srcdir and --cache-file options
--- acgeneral.m4 Sat Jan 6 22:36:12 2001
+++ ../autoconf-2.49b.gnu/acgeneral.m4 Wed Dec 6 14:44:12 2000
@@ -4681,10 +4597,8 @@
if test "$no_recursion" != yes; then
# Remove --cache-file and --srcdir arguments so they do not pile up.
-dnl Don't forget we might have quoted these!
ac_sub_configure_args=
ac_prev=
- ac_sep=
for ac_arg in $ac_configure_args; do
if test -n "$ac_prev"; then
ac_prev=
@@ -4698,21 +4612,13 @@
| --cache-f=* | --cache-=* | --cache=* | --cach=* | --cac=* | --ca=* \
| --c=*)
;;
- \'-cache-file=*\' | \'--cache-file=*\' | \'--cache-fil=*\' \
- | \'--cache-fi=*\' | \'--cache-f=*\' | \'--cache-=*\' | \'--cache=*\' \
- | \'--cach=*\' | \'--cac=*\' | \'--ca=*\' | \'--c=*\')
- ;;
--config-cache | -C)
;;
-srcdir | --srcdir | --srcdi | --srcd | --src | --sr)
ac_prev=srcdir ;;
-srcdir=* | --srcdir=* | --srcdi=* | --srcd=* | --src=* | --sr=*)
;;
- \'-srcdir=*\' | \'--srcdir=*\' | \'--srcdi=*\' | \'--srcd=*\' \
- | \'--src=*\' | \'--sr=*\')
- ;;
- *) ac_sub_configure_args="$ac_sub_configure_args$ac_sep$ac_arg"
- ac_sep=" ";;
+ *) ac_sub_configure_args="$ac_sub_configure_args $ac_arg" ;;
esac
done
@@ -4755,9 +4661,9 @@
# Check for guested configure; otherwise get Cygnus style configure.
if test -f $ac_sub_srcdir/configure.gnu; then
- ac_sub_configure="$SHELL '$ac_sub_srcdir/configure.gnu'"
+ ac_sub_configure="$SHELL $ac_sub_srcdir/configure.gnu"
elif test -f $ac_sub_srcdir/configure; then
- ac_sub_configure="$SHELL '$ac_sub_srcdir/configure'"
+ ac_sub_configure="$SHELL $ac_sub_srcdir/configure"
elif test -f $ac_sub_srcdir/configure.in; then
ac_sub_configure=$ac_configure
else
@@ -4780,28 +4686,9 @@
esac
])dnl
- # Since we're fabricating the --srcdir option, quote it like
- # like we normally would.
- case $ac_sub_srcdir in
-[ *" "*|*" "*|*[\[\]\~\#\$\^\&\*\(\)\{\}\\\|\;\<\>\?\"\']*)]
- ac_sub_srcdir=`echo "$ac_sub_srcdir" | sed "s/'/'\\\\\\\\''/g"`
- ac_sub_srcdir_opt="'--srcdir=$ac_sub_srcdir'" ;;
- *) ac_sub_srcdir_opt="--srcdir=$ac_sub_srcdir" ;;
- esac
- ac_sub_configure_args="$ac_sub_configure_args$ac_sep$ac_sub_srcdir_opt"
-
- # Same goes for --cache-file
- case $ac_sub_cache_file in
-[ *" "*|*" "*|*[\[\]\~\#\$\^\&\*\(\)\{\}\\\|\;\<\>\?\"\']*)]
- ac_sub_cache_file=`echo "$ac_sub_cache_file" | sed "s/'/'\\\\\\\\''/g"`
- ac_sub_cache_opt="'--cache-file=$ac_sub_cache_file'" ;;
- *) ac_sub_cache_opt="--cache-file=$ac_sub_cache_file" ;;
- esac
- ac_sub_configure_args="$ac_sub_configure_args $ac_sub_cache_opt"
-
- AC_MSG_NOTICE([running $ac_sub_configure $ac_sub_configure_args])
+ AC_MSG_NOTICE([running $ac_sub_configure $ac_sub_configure_args
--cache-file=$ac_sub_cache_file --srcdir=$ac_sub_srcdir])
# The eval makes quoting arguments work.
- if eval $ac_sub_configure $ac_sub_configure_args
+ if eval $ac_sub_configure $ac_sub_configure_args
--cache-file=$ac_sub_cache_file --srcdir=$ac_sub_srcdir
then :
else
AC_MSG_ERROR([$ac_sub_configure failed for $ac_subdir])
- Argument quoting problem when processing config_subdirs, Tim Van Holder, 2001/01/06
- RE: Argument quoting problem when processing config_subdirs, Tim Van Holder, 2001/01/06
- RE: Argument quoting problem when processing config_subdirs,
Tim Van Holder <=
- RE: Argument quoting problem when processing config_subdirs, Tim Van Holder, 2001/01/06
- Re: Argument quoting problem when processing config_subdirs, Akim Demaille, 2001/01/11
- Re: Argument quoting problem when processing config_subdirs, Pavel Roskin, 2001/01/11
- RE: Argument quoting problem when processing config_subdirs, Tim Van Holder, 2001/01/11
- Re: Argument quoting problem when processing config_subdirs, Akim Demaille, 2001/01/15
- Re: Argument quoting problem when processing config_subdirs, Akim Demaille, 2001/01/15