autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: char-ranges.patch


From: Pavel Roskin
Subject: Re: char-ranges.patch
Date: Tue, 24 Oct 2000 14:59:33 -0400 (EDT)

Hello, Akim!

>From the original patch:

> @@ -2518,15 +2531,16 @@
>    (set) 2>&1 |
>      case `(ac_space=' '; set | grep ac_space) 2>&1` in
>      *ac_space=\ *)
> -      # `set' does not quote correctly, so add quotes (double-quote 
> substitution
> -      # turns \\\\ into \\, and sed turns \\ into \).
> +      # `set' does not quote correctly, so add quotes (double-quote
> +      # substitution turns \\\\ into \\, and sed turns \\ into \).
>        sed -n \
> -     -e "s/'/'\\\\''/g" \
> -     -e ["s/^\\([a-zA-Z0-9_]*_cv_[a-zA-Z0-9_]*\\)=\\(.*\\)/\\1='\\2'/p"]
> +        ["s/'/'\\\\''/g \
> +       s/^\\([_$ac_cr_alnum]*_cv_[_$ac_cr_alnum]*\\)=\\(.*\\)/\\1='\\2'/p"]

I missed an error here, so the Autoconf is badly broken now :-(
There should be a separator after "g".
config.status uses "g;" and newline - let's do the same to be on the safe
side.

__________________________
Index: ChangeLog
--- ChangeLog   Tue Oct 24 14:54:45 2000
+++ ChangeLog   Tue Oct 24 14:56:33 2000
@@ -1 +1,6 @@
+2000-10-24  Pavel Roskin  <address@hidden>
+
+       * acgeneral.m4 (_AC_CACHE_DUMP): Add missing separator for sed
+       commands.
+
 2000-10-24  Akim Demaille  <address@hidden>
Index: acgeneral.m4
--- acgeneral.m4        Tue Oct 24 14:57:29 2000
+++ acgeneral.m4        Tue Oct 24 14:58:21 2000
@@ -2547,7 +2547,7 @@
       # `set' does not quote correctly, so add quotes (double-quote
       # substitution turns \\\\ into \\, and sed turns \\ into \).
       sed -n \
-        ["s/'/'\\\\''/g \
+        ["s/'/'\\\\''/g;
          s/^\\([_$ac_cr_alnum]*_cv_[_$ac_cr_alnum]*\\)=\\(.*\\)/\\1='\\2'/p"]
       ;;
     *)
__________________________

Regards,
Pavel Roskin




reply via email to

[Prev in Thread] Current Thread [Next in Thread]