Re: FYI: Egrep and test suite (Was: FYI: AS_EXIT)

[Akim Demaille]

>>>>> "Daniele" == Daniele Arena <address@hidden> writes:

Daniele> Sorry, I really can't run it right now, my schedule is *too*
Daniele> tight. As soon as I have some spare minutes, I'll do
Daniele> it. Maybe not before next week, though. Sorry.:(

OK, no problems.  Thanks for your efforts anyway!


Daniele> Hm. No, I just ran the testsuite like that, everything should
Daniele> be in.  Strange...

So maybe it's something else, I don't know.

Well, the previous patch was not enough actually, I'm applying this:

Index: ChangeLog
from  Akim Demaille  <address@hidden>
        * tests/aclocal.m4 (AT_STATE_SAVE): Don't even try to preserve
        egrep error messages, AT_CHECK will find them.

Index: tests/aclocal.m4
===================================================================
RCS file: /cvs/autoconf/tests/aclocal.m4,v
retrieving revision 1.9
diff -u -u -r1.9 aclocal.m4
--- tests/aclocal.m4 2000/12/06 14:05:47 1.9
+++ tests/aclocal.m4 2000/12/06 15:13:10
@@ -15,9 +15,17 @@
 # - AWK|LEX|LEXLIB|LEX_OUTPUT_ROOT|LN_S|M4|RANLIB|SET_MAKE|YACC
 #   AC_PROGs from acspecific
 # - _|@|.[*#?].|LINENO|OLDPWD|PIPESTATUS|RANDOM|SECONDS
-#   Some variables some shells use and change
+#   Some variables some shells use and change.
+#   `.[*#?].' catches `$#' etc. which are displayed like this:
+#      | '!'=18186
+#      | '#'=0
+#      | '$'=6908
 # - POW_LIB
 #   From acfunctions.m4.
+#
+# Some `egrep' choke on such a big regex (e.g., SunOS 4.1.3).  In this
+# case just don't pay attention to the env.  It would be great
+# to keep the error message but we can't: that would break AT_CHECK.
 m4_defun([AC_STATE_SAVE],
 [(set) 2>&1 |
   egrep -v -e 'm4_join([|],
@@ -33,12 +41,10 @@
       [^(f77_(case|underscore))=],
       [^(ALLOCA|GETLOADAVG_LIBS|KMEM_GROUP|NEED_SETGID|POW_LIB)=],
       [^(AWK|LEX|LEXLIB|LEX_OUTPUT_ROOT|LN_S|M4|RANLIB|SET_MAKE|YACC)=],
-      [^(_|@|.[*#?].|LINENO|OLDPWD|PIPESTATUS|RANDOM|SECONDS)=])' |
+      [^(_|@|.[*#?].|LINENO|OLDPWD|PIPESTATUS|RANDOM|SECONDS)=])' 2>/dev/null |
   # There maybe variables spread on several lines, eg IFS, remove the dead
   # lines.
   fgrep = >state-env.$1
-# Some `egrep' choke on such a big regex (e.g., SunOS 4.1.3).  In this
-# case just don't pay attention to the env.
 test $? = 0 || rm -f state-env.$1
 
 ls -1 | grep -v '^state' | sort >state-ls.$1