diff options
author | Jim Meyering <meyering@iou.iou> | 2008-12-30 19:16:08 +0100 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2009-01-01 02:15:07 +0100 |
commit | 7fc5292a291dab4a1b5bf5370de1e083003dd18b (patch) | |
tree | 419f9dc905fa4297761f0118520ca0a9e0cbc9e8 /m4/jm-macros.m4 | |
parent | cf5616a0aaa6f357b5d74e495dddcc0d8dac100c (diff) | |
download | coreutils-7fc5292a291dab4a1b5bf5370de1e083003dd18b.tar.gz coreutils-7fc5292a291dab4a1b5bf5370de1e083003dd18b.tar.bz2 coreutils-7fc5292a291dab4a1b5bf5370de1e083003dd18b.zip |
improve M4 quoting
* configure.ac: Add quotes.
* gl/m4/mgetgroups.m4 (gl_MGETGROUPS): Likewise.
* m4/boottime.m4 (GNULIB_BOOT_TIME): Likewise.
* m4/check-decl.m4 (gl_CHECK_DECLS): Likewise.
* m4/gmp.m4 (cu_GMP): Likewise.
* m4/jm-macros.m4 (coreutils_MACROS, gl_CHECK_ALL_HEADERS): Likewise.
(gl_CHECK_ALL_TYPES): Likewise.
* m4/lib-check.m4 (cu_LIB_CHECK): Likewise.
* m4/stat-prog.m4 (cu_PREREQ_STAT_PROG): Likewise.
FYI, I first ran this command:
git ls-files | grep -E '\.(m4|ac)$' | xargs perl -pi \
-e 's/(AC_[A-Z_]+\()([^[()]+?)([,)])/$1\[$2]$3/g;' \
-e 's/(AC_[A-Z_]+\(\[[^,]+?\], )([^,[()]+?)([,)])/$1\[$2]$3/g;' \
-e 's/(AC_[A-Z_]+\((?:\[[^,]+?\], ){2})([^,[()]+?)([,)])/$1\[$2]$3/g'
Then I updated serial numbers and copyright dates manually.
Also, I manually added two pairs of quotes in boottime.m4.
Diffstat (limited to 'm4/jm-macros.m4')
-rw-r--r-- | m4/jm-macros.m4 | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/m4/jm-macros.m4 b/m4/jm-macros.m4 index 38e19fcb5..335ec242a 100644 --- a/m4/jm-macros.m4 +++ b/m4/jm-macros.m4 @@ -1,4 +1,4 @@ -#serial 107 -*- autoconf -*- +#serial 108 -*- autoconf -*- dnl Misc type-related macros for coreutils. @@ -22,7 +22,7 @@ dnl Misc type-related macros for coreutils. AC_DEFUN([coreutils_MACROS], [ AM_MISSING_PROG(HELP2MAN, help2man) - AC_SUBST(MAN) + AC_SUBST([MAN]) dnl This macro actually runs replacement code. See isc-posix.m4. AC_REQUIRE([AC_ISC_POSIX])dnl @@ -44,7 +44,7 @@ AC_DEFUN([coreutils_MACROS], # used by ls AC_REQUIRE([gl_CLOCK_TIME]) # used by shred - AC_CHECK_FUNCS_ONCE(directio) + AC_CHECK_FUNCS_ONCE([directio]) # Used by install.c. AC_CHECK_FUNCS_ONCE([matchpathcon_init_prefix]) @@ -79,7 +79,7 @@ AC_DEFUN([coreutils_MACROS], [test "$ac_cv_search_fdatasync" = "none required" || LIB_FDATASYNC=$ac_cv_search_fdatasync]) AC_SUBST([LIB_FDATASYNC]) - AC_CHECK_FUNCS(fdatasync) + AC_CHECK_FUNCS([fdatasync]) LIBS=$coreutils_saved_libs # Check whether libcap is usable -- for ls --color support @@ -89,7 +89,7 @@ AC_DEFUN([coreutils_MACROS], [AC_CHECK_LIB([cap], [cap_get_file], [AC_CHECK_HEADER([sys/capability.h], [LIB_CAP=-lcap - AC_DEFINE([HAVE_CAP], 1, [libcap usability])], + AC_DEFINE([HAVE_CAP], [1], [libcap usability])], [AC_MSG_WARN([header sys/capability.h was not found, support for libcap will not be built])] )], [AC_MSG_WARN([libcap library was not found or not usable, support for libcap will not be built])]) @@ -101,16 +101,16 @@ AC_DEFUN([coreutils_MACROS], # BeOS which has all the math functions in the normal runtime library # and doesn't have a separate math library. - AC_SUBST(SEQ_LIBM) + AC_SUBST([SEQ_LIBM]) ac_seq_body=' static double x, y; x = floor (x); x = rint (x); x = modf (x, &y);' - AC_TRY_LINK([#include <math.h>], $ac_seq_body, , + AC_TRY_LINK([#include <math.h>], [$ac_seq_body], , [ac_seq_save_LIBS="$LIBS" LIBS="$LIBS -lm" - AC_TRY_LINK([#include <math.h>], $ac_seq_body, SEQ_LIBM=-lm) + AC_TRY_LINK([#include <math.h>], [$ac_seq_body], [SEQ_LIBM=-lm]) LIBS="$ac_seq_save_LIBS" ]) @@ -133,7 +133,7 @@ AC_DEFUN([gl_CHECK_ALL_HEADERS], syslog.h \ termios.h \ ) - AC_CHECK_HEADERS(sys/sysctl.h, [], [], + AC_CHECK_HEADERS([sys/sysctl.h], [], [], [AC_INCLUDES_DEFAULT [#if HAVE_SYS_PARAM_H #include <sys/param.h> @@ -174,12 +174,12 @@ AC_DEFUN([gl_CHECK_ALL_TYPES], AC_REQUIRE([AC_TYPE_PID_T]) AC_REQUIRE([AC_TYPE_SIZE_T]) AC_REQUIRE([AC_TYPE_UID_T]) - AC_CHECK_TYPE(ino_t, unsigned long int) + AC_CHECK_TYPE([ino_t], [unsigned long int]) dnl This relies on the fact that Autoconf's implementation of dnl AC_CHECK_TYPE checks includes unistd.h. - AC_CHECK_TYPE(major_t, unsigned int) - AC_CHECK_TYPE(minor_t, unsigned int) + AC_CHECK_TYPE([major_t], [unsigned int]) + AC_CHECK_TYPE([minor_t], [unsigned int]) AC_REQUIRE([AC_HEADER_MAJOR]) ]) |