| all your script is saying is that if it finds a package with
| non-alphanumeric (or _) characters, to print an error. If you
| make it something like this (an if-statement this time ;-):
|
| changequote(,)
| if test -n "`echo .$ac_package |sed -e 's/[a-zA-Z0-9_]//g'`"
| changequote([,])
| then
| AC_MSG_ERROR([invalid package: $ac_package]);;
| fi
|
| that has the desired effect. (may need a temporary variable to appease
| very old sh's)
Yep, but that's too hairy:
| if echo "$ac_package" | grep '[[^a-zA-Z0-9_]]' >/dev/null 2>&1; then
| AC_MSG_ERROR([invalid package: $ac_package]);;
| fi
is better, but that's where we come from. The idea was to have the
shell handle this, not a separate command. That's why I'd like to
hear about `expr' here. Otherwise, we will just back up to this one.
