Ralf Wildenhues <Ralf.Wildenhues <at> gmx.de> writes: > > if test -n "`echo $ac_package| sed 's/[-_a-zA-Z0-9]//g'`"; then > Which Autoconf version was the configure script created with?
This string does not occur in current autoconf.git sources, so you are definitely running a package that used an older autoconf. So the packager's claim in http://bugs.php.net/bug.php?id=48098 that it is an autoconf bug and that they can't do anything about it is somewhat false; they could update to a newer autoconf. > > Solution: use [:alpha:] instead a-zA-z in various regular expressions. > > No, sorry; these classes are unfortunately not portable enough yet. However, m4sh DOES provide as_cr_alnum which is longhand for abcdefg...XYZ0123456789, such that this could be written: as_tmp=`echo $ac_package | sed "s/[-_$as_cr_alnum]//g"` if test -n "$as_tmp"; then without tripping over yet other portability bugs. > The > configure script already sets LANGUAGE and LC_ALL to sane values. If > that is not sufficient, then I suppose we need to set LANG, too. What > system does this happen on, which sed version, and why does it not honor > LC_ALL=C? That's important to learn before we try eradicating any other uses of [a-z] ranges passed to sed (although your particular example does not appear in autoconf.git, others still do). -- Eric Blake
