Hello,

regarding the following patch:

On Sun, May 21, 2006 at 09:46:34PM -0700, Paul Eggert wrote:
>       2006-01-13  Jim Meyering  <[EMAIL PROTECTED]>
> 
>       Invoke AC_CHECK_FUNCS(getmntent) unconditionally so that tests of
>       $ac_cv_func_getmntent (e.g., in gl_LIST_MOUNTED_FILE_SYSTEMS) need
>       not double-quote uses of that variable, to accommodate the rare
>       case in which getmntent is available in none of the libraries
>       checked.  This happens at least on FreeBSD 5.0.
> 
> --- functions.m4.~1.100.~     2006-05-18 15:37:22.000000000 -0700
> +++ functions.m4      2006-05-21 21:41:06.000000000 -0700
> @@ -799,7 +799,8 @@ AN_FUNCTION([getmntent], [AC_FUNC_GETMNT
>  AC_DEFUN([AC_FUNC_GETMNTENT],
>  [# getmntent is in the standard C library on UNICOS, in -lsun on Irix 4,
>  # -lseq on Dynix/PTX, -lgen on Unixware.
> -AC_SEARCH_LIBS(getmntent, [sun seq gen], [AC_CHECK_FUNCS(getmntent)])
> +AC_SEARCH_LIBS(getmntent, [sun seq gen])
> +AC_CHECK_FUNCS(getmntent)
>  ])
>  
>  

If I understand it correctly, the AC_CHECK_FUNCS macro is redundant,
so we can eliminate it.
We have to set ac_cv_func_getmntent and HAVE_GETMNTENT, for
compatibility reasons, but we eliminate one compilation.

If there are no objections, I'll install the following patch a few weeks
after the 2.60 release, when all the guys from TIME and CNN will allow
Ralf to get some sleep again.

Have a nice day,
        Stepan
2006-05-22  Stepan Kasal  <[EMAIL PROTECTED]>

        * lib/autoconf/functions.m4 (AC_FUNC_GETMNTENT): Eliminate the
        expansion of AC_CHECK_FUNCS.

Index: lib/autoconf/functions.m4
===================================================================
RCS file: /cvsroot/autoconf/autoconf/lib/autoconf/functions.m4,v
retrieving revision 1.101
diff -u -r1.101 functions.m4
--- lib/autoconf/functions.m4   22 May 2006 04:45:03 -0000      1.101
+++ lib/autoconf/functions.m4   22 May 2006 09:34:34 -0000
@@ -799,8 +799,11 @@
 AC_DEFUN([AC_FUNC_GETMNTENT],
 [# getmntent is in the standard C library on UNICOS, in -lsun on Irix 4,
 # -lseq on Dynix/PTX, -lgen on Unixware.
-AC_SEARCH_LIBS(getmntent, [sun seq gen])
-AC_CHECK_FUNCS(getmntent)
+AC_SEARCH_LIBS(getmntent, [sun seq gen],
+              [ac_cv_func_getmntent=yes
+               AC_DEFINE([HAVE_GETMNTENT], [],
+                         [Define to 1 if you have the `getmntent' function.])],
+              [ac_cv_func_getmntent=no])
 ])
 
 

Reply via email to