autoconf-patches
[Top][All Lists]
Advanced

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

Re: FYI: remove documentation for AU_DEFUN


From: Akim Demaille
Subject: Re: FYI: remove documentation for AU_DEFUN
Date: Wed, 26 Nov 2003 08:57:14 +0100
User-agent: Gnus/5.1002 (Gnus v5.10.2) Emacs/21.3 (gnu/linux)

 > I just found out that there is a small problem, actually: when
 > using "autoconf -W obsolete", AC_LINK_FILES now spits out the text
 > about the inelegance of autoupdate's replacement.  So I did revert
 > my patch, however...
 
 >> > Leaving the warning in the updated file has the advantage of reminding 
 >> > the Autoconf user if they invoke autoconf with warnings enabled. 
 >> 
 >> You seem to forget the common case: AU_ALIAS.  It should not leave any 
 >> such warning. 
 
 > It does not.  The only macro I touched is AC_LINK_FILES, for which
 > I moved the warning from the unused $3 parameter into $1.  I did
 > not modify the definition of AU_DEFUN.

Actually, you did modify AU_DEFUN, and that was the origin of my
doubts.  But indeed, the way you changed it is fine, my bad.  I seemed
to believe that the warning was not part of the AU macro defined, but
it is.

And therefore, we did lose at some point the ability to leave a
warning after autoupdate's run, so that the user is aware that the
code needs hand work.


 > When autoupdate runs, the warning is not printed because no -W
 > option is passed from autoupdate to autom4te; and AU_DEFUN's
 > AC_DIAGNOSE expands to nothing so it does not go into the output
 > file:

That's what I missed, thanks!
 
 > ~/autoconf/tests$ cat > configure.ac <<EOF 
 > AC_INIT 
 > AC_SET_MAKE 
 > AC_OUTPUT 
 > EOF 
 > ~/autoconf/tests$ ./autoconf -W obsolete 
 > configure.ac:2: warning: The macro `AC_SET_MAKE' is obsolete. 
 > configure.ac:2: You should run autoupdate. 
 > autoconf/oldnames.m4:93: AC_SET_MAKE is expanded from... 
 > ~/autoconf/tests$ ./autoupdate 
 > ~/autoconf/tests$ ./autoconf -W obsolete 
 > configure.ac:2: warning: The macro `AC_SET_MAKE' is obsolete. 
 > configure.ac:2: You should run autoupdate. 
 > autoconf/oldnames.m4:93: AC_SET_MAKE is expanded from... 
 > ~/autoconf/tests$ cat configure.ac 
 > AC_INIT 
 > AC_PROG_MAKE_SET([]) 
 > AC_OUTPUT 
 > ~/autoconf/tests$ ./autoconf -W obsolete 
 > ~/autoconf/tests$ 
 
 > Paolo 
 






reply via email to

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