Stepan Kasal <[EMAIL PROTECTED]> writes:

> First, you made a typo, so you are in fact deleting variables with values
> starting with a dollar.

Sorry about that.  It was a missing apostrophe.
I installed the corrected version below.

> Second, the intention to delete all lines matching /^[^=]*=''$/ should be
> documented in the comment above the macro.

That's already done.  Here's the comment:

# Furthermore, it is okay for a non-cache variable initialized to empty in one
# run to be unset in another run.  This happens when, for example, cache update
# code tries a number of values in LIBS and eventually restores LIBS to its
# original value. If LIBS was previously unset, it will have become set and
# empty.  (OTOH, cache variables indicate the result of the test even if they
# are empty, so we have to be strict about them.)

Nicolas's problem is that his shell's "set" command outputs "X=''"
whereas other shells say "X=".

Nicolas, can you please check that this revised patch does the trick?
Thanks.


2005-07-07  Paul Eggert  <[EMAIL PROTECTED]>

        * tests/local.at (AT_CONFIG_CMP): Ignore lines like "LIBS=''" too.
        This is a corrected version of yesterday's patch.

--- tests/local.at.~1.20.~      2005-07-07 16:53:02.000000000 -0700
+++ tests/local.at      2005-07-07 16:53:36.000000000 -0700
@@ -331,6 +331,7 @@ m4_define([AT_CONFIG_CMP],
 do
   $SED '/^ac_cv_/ b
        /^m4_defn([m4_re_word])=./ !d
+       /^[[^=]]*='\'''\''$/ d
        /^a[[cs]]_/ d
        /^PPID=/ d
        /^RANDOM=/ d


Reply via email to