[Emacs-diffs] /srv/bzr/emacs/trunk r108401: Make 'configure' a bit smaller and faster.

[Paul Eggert]

------------------------------------------------------------
revno: 108401
committer: Paul Eggert <address@hidden>
branch nick: trunk
timestamp: Sun 2012-05-27 17:46:33 -0700
message:
  Make 'configure' a bit smaller and faster.
  
  * configure.in (INSTALL_INFO): Set it with one call to
  AC_PATH_PROG, not three.
  (PKG_CONFIG): Hoist this out of PKG_CHECK_MODULES, since it's
  so likely that it'll be needed.
  (AC_CHECK_HEADERS_ONCE): Prefer to AC_CHECK_HEADERS where either works.
  (AC_PROG_MAKE_SET): Remove; Automake does this.
  (sysinfo): Do not check for this function, as it's never used.
  (tzset): Check for this function just once.
  * admin/CPP-DEFINES: Remove HAVE_SYSINFO.
  * m4/manywarnings.m4: Sync from gnulib, incorporating the following:
  2012-05-27 manywarnings: remove duplicate -Wmultichar entry
modified:
  ChangeLog
  admin/CPP-DEFINES
  admin/ChangeLog
  configure.in
  m4/manywarnings.m4

=== modified file 'ChangeLog'
--- a/ChangeLog 2012-05-27 13:42:13 +0000
+++ b/ChangeLog 2012-05-28 00:46:33 +0000
@@ -1,11 +1,25 @@
+2012-05-28  Paul Eggert  <address@hidden>
+
+       Make 'configure' a bit smaller and faster.
+       * configure.in (INSTALL_INFO): Set it with one call to
+       AC_PATH_PROG, not three.
+       (PKG_CONFIG): Hoist this out of PKG_CHECK_MODULES, since it's
+       so likely that it'll be needed.
+       (AC_CHECK_HEADERS_ONCE): Prefer to AC_CHECK_HEADERS where either works.
+       (AC_PROG_MAKE_SET): Remove; Automake does this.
+       (sysinfo): Do not check for this function, as it's never used.
+       (tzset): Check for this function just once.
+       * m4/manywarnings.m4: Sync from gnulib, incorporating the following:
+       2012-05-27 manywarnings: remove duplicate -Wmultichar entry
+
 2012-05-27  Eli Zaretskii  <address@hidden>
 
        * config.bat (lib): Create/update lib/stdalign.in-h and
        lib/sys_types.in-h.
 
-       * lib/makefile.w32-in ($(BLD)/md5.$(O)): 
-       ($(BLD)/sha1.$(O)): 
-       ($(BLD)/sha256.$(O)): 
+       * lib/makefile.w32-in ($(BLD)/md5.$(O)):
+       ($(BLD)/sha1.$(O)):
+       ($(BLD)/sha256.$(O)):
        ($(BLD)/sha512.$(O)): Depend on $(EMACS_ROOT)/nt/inc/stdalign.h.
        Suggested by Christoph Scholtes <address@hidden>.
 

=== modified file 'admin/CPP-DEFINES'
--- a/admin/CPP-DEFINES 2012-05-22 16:20:27 +0000
+++ b/admin/CPP-DEFINES 2012-05-28 00:46:33 +0000
@@ -156,7 +156,6 @@
 HAVE_STRFTIME
 HAVE_STRING_H
 HAVE_STRUCT_UTIMBUF
-HAVE_SYSINFO
 HAVE_SYSV_SIGPAUSE
 HAVE_SYS_SELECT_H
 HAVE_SYS_SYSTEMINFO_H

=== modified file 'admin/ChangeLog'
--- a/admin/ChangeLog   2012-05-27 09:12:09 +0000
+++ b/admin/ChangeLog   2012-05-28 00:46:33 +0000
@@ -1,3 +1,7 @@
+2012-05-28  Paul Eggert  <address@hidden>
+
+       * CPP-DEFINES: Remove HAVE_SYSINFO.
+
 2012-05-27  Paul Eggert  <address@hidden>
 
        Assume gnulib does largefile.

=== modified file 'configure.in'
--- a/configure.in      2012-05-27 09:12:09 +0000
+++ b/configure.in      2012-05-28 00:46:33 +0000
@@ -708,9 +708,8 @@
 dnl   AC_PROG_RANLIB
 dnl fi
 
-AC_PATH_PROG(INSTALL_INFO, install-info)
-AC_PATH_PROG(INSTALL_INFO, install-info,, /usr/sbin)
-AC_PATH_PROG(INSTALL_INFO, install-info,:, /sbin)
+AC_PATH_PROG(INSTALL_INFO, install-info, :,
+  $PATH$PATH_SEPARATOR/usr/sbin$PATH_SEPARATOR/sbin)
 dnl Don't use GZIP, which is used by gzip for additional parameters.
 AC_PATH_PROG(GZIP_PROG, gzip)
 
@@ -1067,6 +1066,8 @@
 esac
 
 
+AC_PATH_PROG(PKG_CONFIG, pkg-config, no)
+
 dnl This function definition taken from Gnome 2.0
 dnl PKG_CHECK_MODULES(GSTUFF, gtk+-2.0 >= 1.3 glib = 1.3.4, action-if, 
action-not)
 dnl defines GSTUFF_LIBS, GSTUFF_CFLAGS, see pkg-config man page
@@ -1074,8 +1075,6 @@
 AC_DEFUN([PKG_CHECK_MODULES], [
   succeeded=no
 
-  AC_PATH_PROG(PKG_CONFIG, pkg-config, no)
-
   if test "$PKG_CONFIG" = "no" ; then
      ifelse([$4], , [AC_MSG_ERROR([
       *** The pkg-config script could not be found. Make sure it is in your 
path, or give the full path to pkg-config with the PKG_CONFIG environment 
variable or --with-pkg-config-prog.  Or see 
http://www.freedesktop.org/software/pkgconfig to get pkg-config.])], [$4])
@@ -1176,10 +1175,11 @@
 fi
 
 dnl checks for header files
-AC_CHECK_HEADERS(sys/select.h sys/time.h unistd.h utime.h \
-  linux/version.h sys/systeminfo.h \
-  stdio_ext.h fcntl.h coff.h pty.h sys/mman.h \
-  sys/vlimit.h sys/resource.h locale.h sys/_mbstate_t.h \
+AC_CHECK_HEADERS_ONCE(
+  sys/select.h sys/time.h utime.h
+  linux/version.h sys/systeminfo.h
+  stdio_ext.h fcntl.h coff.h pty.h
+  sys/vlimit.h sys/resource.h
   sys/utsname.h pwd.h utmp.h dirent.h util.h)
 
 AC_MSG_CHECKING(if personality LINUX32 can be set)
@@ -1263,7 +1263,7 @@
   AC_DEFINE(NO_MATHERR, 1, [Define to 1 if you don't have struct exception in 
math.h.])
 fi
 
-AC_CHECK_HEADERS(sys/socket.h)
+AC_CHECK_HEADERS_ONCE(sys/socket.h)
 AC_CHECK_HEADERS(net/if.h, , , [AC_INCLUDES_DEFAULT
 #if HAVE_SYS_SOCKET_H
 #include <sys/socket.h>
@@ -1292,7 +1292,7 @@
 #endif])
 
 dnl Check for endianness.
-AC_C_BIGENDIAN
+dnl AC_C_BIGENDIAN is done by gnulib.
 
 AC_CACHE_CHECK([for  __attribute__ ((__aligned__ (expr)))],
   [emacs_cv_attribute_aligned],
@@ -1308,7 +1308,7 @@
 fi
 
 dnl check for Make feature
-AC_PROG_MAKE_SET
+dnl AC_PROG_MAKE_SET is done by Automake.
 
 DEPFLAGS=
 MKDEPDIR=":"
@@ -1664,7 +1664,7 @@
 
 dnl Check if pthreads is available.
 LIB_PTHREAD=
-AC_CHECK_HEADERS(pthread.h)
+AC_CHECK_HEADERS_ONCE(pthread.h)
 if test "$ac_cv_header_pthread_h"; then
   dnl gmalloc.c uses pthread_atfork, which is not available on older-style
   dnl hosts such as MirBSD 10, so test for pthread_atfork instead of merely
@@ -2537,7 +2537,7 @@
 AC_SUBST(LIBGPM)
 
 dnl Check for malloc/malloc.h on darwin
-AC_CHECK_HEADER(malloc/malloc.h, [AC_DEFINE(HAVE_MALLOC_MALLOC_H, 1, [Define 
to 1 if you have the <malloc/malloc.h> header file.])])
+AC_CHECK_HEADERS_ONCE(malloc/malloc.h)
 
 GNUSTEP_CFLAGS=
 ### Use NeXTstep API to implement GUI.
@@ -2642,7 +2642,7 @@
   fi
 fi
 AC_CHECK_FUNCS(touchlock)
-AC_CHECK_HEADERS(maillock.h)
+AC_CHECK_HEADERS_ONCE(maillock.h)
 AC_SUBST(LIBS_MAIL)
 
 ## Define MAIL_USE_FLOCK (or LOCKF) if the mailer uses flock (or lockf) to
@@ -2688,9 +2688,9 @@
 
 
 AC_CHECK_FUNCS(gethostname \
-rename closedir mkdir rmdir sysinfo getrusage get_current_dir_name \
+rename closedir mkdir rmdir getrusage get_current_dir_name \
 random lrand48 logb frexp fmod rint cbrt setsid \
-strerror fpathconf select euidaccess getpagesize tzset setlocale \
+strerror fpathconf select euidaccess getpagesize setlocale \
 utimes getrlimit setrlimit setpgid getcwd getwd shutdown getaddrinfo \
 __fpending strsignal setitimer \
 sendto recvfrom getsockname getpeername \
@@ -2710,7 +2710,7 @@
            [Define to 1 if you have the `__builtin_unwind_init' function.])
 fi
 
-AC_CHECK_HEADERS(sys/un.h)
+AC_CHECK_HEADERS_ONCE(sys/un.h)
 
 AC_FUNC_FSEEKO
 
@@ -2977,6 +2977,7 @@
 AC_SUBST(DESLIB)
 AC_SUBST(KRB4LIB)
 
+AC_CHECK_FUNCS_ONCE(tzset)
 AC_MSG_CHECKING(whether localtime caches TZ)
 AC_CACHE_VAL(emacs_cv_localtime_cache,
 [if test x$ac_cv_func_tzset = xyes; then

=== modified file 'm4/manywarnings.m4'
--- a/m4/manywarnings.m4        2012-04-09 07:45:59 +0000
+++ b/m4/manywarnings.m4        2012-05-28 00:46:33 +0000
@@ -1,4 +1,4 @@
-# manywarnings.m4 serial 3
+# manywarnings.m4 serial 4
 dnl Copyright (C) 2008-2012 Free Software Foundation, Inc.
 dnl This file is free software; the Free Software Foundation
 dnl gives unlimited permission to copy and/or distribute it,
@@ -143,7 +143,6 @@
   for gl_manywarn_item in \
     -Wattributes \
     -Wcoverage-mismatch \
-    -Wmultichar \
     -Wunused-macros \
     ; do
     gl_manywarn_set="$gl_manywarn_set $gl_manywarn_item"