autoconf-patches
[Top][All Lists]
Advanced

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

Re: Environment variable $U


From: Ralf Wildenhues
Subject: Re: Environment variable $U
Date: Mon, 22 Feb 2010 21:57:34 +0100
User-agent: Mutt/1.5.20 (2009-10-28)

* Eric Blake wrote on Tue, Feb 09, 2010 at 02:26:34PM CET:
> tests/semantics.at:AC_REPLACE_FUNCS also uses $U, so we could see a
> spurious test failure; I'm not sure what to do there.

Isn't that one using $U as designed?

> Thoughts before I apply this to autoconf?

Well that moment has already passed, but it would've been good to squash
in the patch below, to fix lots of testsuite failures.  OK?

> >  Or we could also go ahead and remove ansi2knr support (after
> >  deprecating it suitably).
> 
> It's always been a desire of mine to switch to C89 as default and require
> extra effort on the part of people still wanting K&R, rather than the
> current situation of trying to support both by default.  But it takes lots
> of time and energy to implement, and hasn't been a high priority for me.

I'm definitely for postponing this action to after the next release.

Cheers,
Ralf

    Fix testsuite failures due to setting of $U.
    
    * tests/local.at (_AT_CHECK_ENV): Ignore setting of $U.

diff --git a/tests/local.at b/tests/local.at
index 58a9069..dcfdf60 100644
--- a/tests/local.at
+++ b/tests/local.at
@@ -303,7 +303,7 @@ if test -f state-env.before && test -f state-env.after; then
       [CYGWIN|ISC|MINGW32|MINIX|EMXOS2|XENIX|EXEEXT|OBJEXT],
       [X_(CFLAGS|(EXTRA_|PRE_)?LIBS)|x_(includes|libraries)|(have|no)_x],
       [(host|build|target)(_(alias|cpu|vendor|os))?],
-      [cross_compiling],
+      [cross_compiling|U],
       [interpval|PATH_SEPARATOR],
       [F77_DUMMY_MAIN|f77_(case|underscore)],
       [FC(_DUMMY_MAIN|FLAGS|LIBS|FLAGS_f)?],





reply via email to

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