(Shell Substitutions): Fix typo in case statement.

Warn about newline stripping in `` and $().  Update Solaris
version to 9.
(Limitations of Builtins): Use expr "X...", not expr "x...", as
X insulates us from future changes to Posix.
(Limitations of Usual Tools): For AS_DIRNAME, warn about newline
stripping.

diff --git a/doc/autoconf.texi b/doc/autoconf.texi
index 4ee449c7c4370d03e0fdf2cd4fc272db8e709ee0..921d4a8e4f7cbbfd4ce1193fdb4fe3330bf3bffc 100644 (file)
--- a/doc/autoconf.texi
+++ b/doc/autoconf.texi
@@ -10002,7 +10002,7 @@ For instance, the following code:
 
 @example
 case "$given_srcdir" in
-.)  top_srcdir="`echo "$dots" | sed 's,/$,,'`"
+.)  top_srcdir="`echo "$dots" | sed 's,/$,,'`" ;;
 *)  top_srcdir="$dots$given_srcdir" ;;
 esac
 @end example
@@ -10012,7 +10012,7 @@ is more readable when written as:
 
 @example
 case $given_srcdir in
-.)  top_srcdir=`echo "$dots" | sed 's,/$,,'`
+.)  top_srcdir=`echo "$dots" | sed 's,/$,,'` ;;
 *)  top_srcdir=$dots$given_srcdir ;;
 esac
 @end example
@@ -10190,6 +10190,11 @@ test "$@{var+set@}" = set || var=@var{@{value@}}
 @item `@var{commands}`
 @cindex `@var{commands}`
 @cindex Command Substitution
+Posix requires shells to trim all trailing newlines from command
+output before substituting it, so assignments like
+@samp{dir=`AS_DIRNAME(["$file"])`} will not work as expected if the
+directory of @samp{$file} ends in a newline.
+
 While in general it makes no sense, do not substitute a single builtin
 with side effects, because Ash 0.2, trying to optimize, does not fork a
 subshell to perform the command.
@@ -10210,14 +10215,17 @@ The result of @samp{foo=`exit 1`} is left as an exercise to the reader.
 
 @item $(@var{commands})
 @cindex $(@var{commands})
-This construct is meant to replace @samp{`@var{commands}`}; they can be
+This construct is meant to replace @samp{`@var{commands}`},
+and it has most of the problems listed under @code{`@var{commands}`}.
+
+This construct can be
 nested while this is impossible to do portably with back quotes.
-Unfortunately it is not yet widely supported.  Most notably, even recent
+Unfortunately it is not yet universally supported.  Most notably, even recent
 releases of Solaris don't support it:
 
 @example
 $ @kbd{showrev -c /bin/sh | grep version}
-Command version: SunOS 5.8 Generic 109324-02 February 2001
+Command version: SunOS 5.9 Generic May 2002
 $ @kbd{echo $(echo blah)}
 syntax error: `(' unexpected
 @end example
@@ -11155,7 +11163,7 @@ expr "$ac_feature" : '.*[^-a-zA-Z0-9_]' >/dev/null &&
 or better yet
 
 @example
-expr "x$ac_feature" : '.*[^-a-zA-Z0-9_]' >/dev/null &&
+expr "X$ac_feature" : '.*[^-a-zA-Z0-9_]' >/dev/null &&
   @var{action}
 @end example
 
@@ -11481,6 +11489,10 @@ dir=`dirname "$file"`       # This is not portable.
 dir=`AS_DIRNAME(["$file"])` # This is more portable.
 @end example
 
+Unfortunately, neither of the above commands work if @code{$file}'s
+directory name ends in newline, since @samp{`@dots{}`} removes all
+trailing newlines.
+
 @noindent
 This handles a few subtleties in the standard way required by
 Posix.  For example, under UN*X, should @samp{dirname //1} give