From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id B6521C433F5 for ; Mon, 11 Oct 2021 16:53:57 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 91F9F60F38 for ; Mon, 11 Oct 2021 16:53:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231954AbhJKQz5 (ORCPT ); Mon, 11 Oct 2021 12:55:57 -0400 Received: from mail.kernel.org ([198.145.29.99]:41086 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229816AbhJKQz4 (ORCPT ); Mon, 11 Oct 2021 12:55:56 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 42EE160F35; Mon, 11 Oct 2021 16:53:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1633971236; bh=L3inRigEWdVCQ5SFbI+5XM7lBfBsn1YWkAZ8HH6pMWo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=WjwZma++piznE8kd2YCF/gBCrGVi1BVWKtzWzyKGV0CSvLmR6ZOHzjzRL8TzjM8LE Jh9tq9AX9DmhbcexKmU+feVGDWxzPpa5/pGHukS6OevPolJdx9k7HVrGHj/Ut7QF6O rr3lylC4G7S3jJEvI/t9HyqmFyI6rxsUn9Of13zN8ixjROAXnUGOwuo1hGUKB2nQuA Yhwo25L8E9nbHHTm32zrm4Z5Czqlnm0+bOQbSGSYC85s0cb5Q8pIMWJLD+ywXAjorf yFgdUOA6LT/AOG6Z/LVSSg5UHx3CcHrFqAfdhDzQU8H4jTn3F4YOIsu8bpG4cVKAzC DJJnH6ipPgU0g== Date: Mon, 11 Oct 2021 09:53:55 -0700 From: "Darrick J. Wong" To: Dave Chinner Cc: fstests@vger.kernel.org Subject: Re: [PATCH 1/2] fstests: autoconf has deprecated a bunch of macros Message-ID: <20211011165355.GG24282@magnolia> References: <20211010230659.216166-1-david@fromorbit.com> <20211010230659.216166-2-david@fromorbit.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20211010230659.216166-2-david@fromorbit.com> Precedence: bulk List-ID: X-Mailing-List: fstests@vger.kernel.org On Mon, Oct 11, 2021 at 10:06:58AM +1100, Dave Chinner wrote: > From: Dave Chinner > > Latest debian unstable environment results in these build warnings: > > autoconf > configure.ac:8: warning: The macro `AC_HEADER_STDC' is obsolete. > configure.ac:8: You should run autoupdate. > ./lib/autoconf/headers.m4:704: AC_HEADER_STDC is expanded from... > configure.ac:8: the top level > configure.ac:50: warning: The macro `AC_TRY_LINK' is obsolete. > configure.ac:50: You should run autoupdate. > ./lib/autoconf/general.m4:2920: AC_TRY_LINK is expanded from... > m4/package_xfslibs.m4:95: AC_PACKAGE_NEED_XFSCTL_MACRO is expanded from... > configure.ac:50: the top level > configure.ac:63: warning: The macro `AC_TRY_LINK' is obsolete. > configure.ac:63: You should run autoupdate. > ./lib/autoconf/general.m4:2920: AC_TRY_LINK is expanded from... > acinclude.m4:19: AC_PACKAGE_WANT_FALLOCATE is expanded from... > configure.ac:63: the top level > configure.ac:64: warning: The macro `AC_TRY_LINK' is obsolete. > configure.ac:64: You should run autoupdate. > ./lib/autoconf/general.m4:2920: AC_TRY_LINK is expanded from... > acinclude.m4:32: AC_PACKAGE_WANT_OPEN_BY_HANDLE_AT is expanded from... > configure.ac:64: the top level > configure.ac:69: warning: The macro `AC_TRY_LINK' is obsolete. > configure.ac:69: You should run autoupdate. > ./lib/autoconf/general.m4:2920: AC_TRY_LINK is expanded from... > m4/package_libcdev.m4:106: AC_HAVE_COPY_FILE_RANGE is expanded from... > configure.ac:69: the top level > configure.ac:107: warning: The macro `AC_CONFIG_HEADER' is obsolete. > configure.ac:107: You should run autoupdate. > ./lib/autoconf/status.m4:719: AC_CONFIG_HEADER is expanded from... > configure.ac:107: the top level > > So, run autoupdate and then fix all the warnings. > > Signed-off-by: Dave Chinner Much as I hate autoconf, this actually does look correct now that I've sifted through the latest documentation. Surprisingly it even works on Ubuntu 20.04... Reviewed-by: Darrick J. Wong --D > --- > acinclude.m4 | 16 ++++------- > configure.ac | 9 +++---- > m4/package_libcdev.m4 | 63 +++++++++++++++++++------------------------ > m4/package_ncurses.m4 | 7 ++--- > m4/package_xfslibs.m4 | 7 ++--- > 5 files changed, 41 insertions(+), 61 deletions(-) > > diff --git a/acinclude.m4 b/acinclude.m4 > index a605c01c..fd92f0d5 100644 > --- a/acinclude.m4 > +++ b/acinclude.m4 > @@ -18,28 +18,22 @@ AC_DEFUN([AC_PACKAGE_WANT_LINUX_FS_H], > > AC_DEFUN([AC_PACKAGE_WANT_FALLOCATE], > [ AC_MSG_CHECKING([for fallocate]) > - AC_TRY_LINK([ > + AC_LINK_IFELSE([AC_LANG_PROGRAM([[ > #define _GNU_SOURCE > #define _FILE_OFFSET_BITS 64 > #include > -#include ], > - [ fallocate(0, 0, 0, 0); ], > - [ have_fallocate=true; AC_MSG_RESULT(yes) ], > - [ have_fallocate=false; AC_MSG_RESULT(no) ]) > +#include ]], [[ fallocate(0, 0, 0, 0); ]])],[ have_fallocate=true; AC_MSG_RESULT(yes) ],[ have_fallocate=false; AC_MSG_RESULT(no) ]) > AC_SUBST(have_fallocate) > ]) > > AC_DEFUN([AC_PACKAGE_WANT_OPEN_BY_HANDLE_AT], > [ AC_MSG_CHECKING([for open_by_handle_at]) > - AC_TRY_LINK([ > + AC_LINK_IFELSE([AC_LANG_PROGRAM([[ > #define _GNU_SOURCE > #include > - ], > - [ > + ]], [[ > struct file_handle fh; > open_by_handle_at(0, &fh, 0); > - ], > - [ have_open_by_handle_at=true; AC_MSG_RESULT(yes) ], > - [ have_open_by_handle_at=false; AC_MSG_RESULT(no) ]) > + ]])],[ have_open_by_handle_at=true; AC_MSG_RESULT(yes) ],[ have_open_by_handle_at=false; AC_MSG_RESULT(no) ]) > AC_SUBST(have_open_by_handle_at) > ]) > diff --git a/configure.ac b/configure.ac > index 6e5ab397..cbf83779 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -1,12 +1,11 @@ > -AC_INIT([xfstests], [1.1.1]) > +AC_INIT([xfstests],[1.1.2]) > AC_CONFIG_MACRO_DIR([m4]) > LT_INIT > AC_CONFIG_SRCDIR([src/xfsctl.c]) > AC_PACKAGE_GLOBALS(xfstests) > AC_PACKAGE_UTILITIES(xfstests) > > -AC_HEADER_STDC > - AC_CHECK_HEADERS([ assert.h \ > +AC_CHECK_HEADERS([ assert.h \ > bstring.h \ > libgen.h \ > dirent.h \ > @@ -34,7 +33,7 @@ AC_HEADER_STDC > btrfs/ioctl.h \ > cifs/ioctl.h \ > sys/mman.h \ > - ]) > +]) > > AC_CHECK_HEADERS([xfs/xfs_log_format.h],,,[ > #define _GNU_SOURCE > @@ -104,6 +103,6 @@ AC_CHECK_MEMBERS([struct btrfs_ioctl_vol_args_v2.subvolid], [], [], [[ > #include > ]]) > > -AC_CONFIG_HEADER(include/config.h) > +AC_CONFIG_HEADERS([include/config.h]) > AC_CONFIG_FILES([include/builddefs]) > AC_OUTPUT > diff --git a/m4/package_libcdev.m4 b/m4/package_libcdev.m4 > index 14e67e18..5c76c0f7 100644 > --- a/m4/package_libcdev.m4 > +++ b/m4/package_libcdev.m4 > @@ -3,15 +3,14 @@ > # > AC_DEFUN([AC_HAVE_FADVISE], > [ AC_MSG_CHECKING([for fadvise ]) > - AC_TRY_COMPILE([ > + AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ > #define _GNU_SOURCE > #define _FILE_OFFSET_BITS 64 > #include > - ], [ > + ]], [[ > posix_fadvise(0, 1, 0, POSIX_FADV_NORMAL); > - ], have_fadvise=yes > - AC_MSG_RESULT(yes), > - AC_MSG_RESULT(no)) > + ]])],[have_fadvise=yes > + AC_MSG_RESULT(yes)],[AC_MSG_RESULT(no)]) > AC_SUBST(have_fadvise) > ]) > > @@ -20,15 +19,14 @@ AC_DEFUN([AC_HAVE_FADVISE], > # > AC_DEFUN([AC_HAVE_MADVISE], > [ AC_MSG_CHECKING([for madvise ]) > - AC_TRY_COMPILE([ > + AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ > #define _GNU_SOURCE > #define _FILE_OFFSET_BITS 64 > #include > - ], [ > + ]], [[ > posix_madvise(0, 0, MADV_NORMAL); > - ], have_madvise=yes > - AC_MSG_RESULT(yes), > - AC_MSG_RESULT(no)) > + ]])],[have_madvise=yes > + AC_MSG_RESULT(yes)],[AC_MSG_RESULT(no)]) > AC_SUBST(have_madvise) > ]) > > @@ -37,15 +35,14 @@ AC_DEFUN([AC_HAVE_MADVISE], > # > AC_DEFUN([AC_HAVE_MINCORE], > [ AC_MSG_CHECKING([for mincore ]) > - AC_TRY_COMPILE([ > + AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ > #define _GNU_SOURCE > #define _FILE_OFFSET_BITS 64 > #include > - ], [ > + ]], [[ > mincore(0, 0, 0); > - ], have_mincore=yes > - AC_MSG_RESULT(yes), > - AC_MSG_RESULT(no)) > + ]])],[have_mincore=yes > + AC_MSG_RESULT(yes)],[AC_MSG_RESULT(no)]) > AC_SUBST(have_mincore) > ]) > > @@ -54,15 +51,14 @@ AC_DEFUN([AC_HAVE_MINCORE], > # > AC_DEFUN([AC_HAVE_SENDFILE], > [ AC_MSG_CHECKING([for sendfile ]) > - AC_TRY_COMPILE([ > + AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ > #define _GNU_SOURCE > #define _FILE_OFFSET_BITS 64 > #include > - ], [ > + ]], [[ > sendfile(0, 0, 0, 0); > - ], have_sendfile=yes > - AC_MSG_RESULT(yes), > - AC_MSG_RESULT(no)) > + ]])],[have_sendfile=yes > + AC_MSG_RESULT(yes)],[AC_MSG_RESULT(no)]) > AC_SUBST(have_sendfile) > ]) > > @@ -71,14 +67,13 @@ AC_DEFUN([AC_HAVE_SENDFILE], > # > AC_DEFUN([AC_HAVE_GETMNTENT], > [ AC_MSG_CHECKING([for getmntent ]) > - AC_TRY_COMPILE([ > + AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ > #include > #include > - ], [ > + ]], [[ > getmntent(0); > - ], have_getmntent=yes > - AC_MSG_RESULT(yes), > - AC_MSG_RESULT(no)) > + ]])],[have_getmntent=yes > + AC_MSG_RESULT(yes)],[AC_MSG_RESULT(no)]) > AC_SUBST(have_getmntent) > ]) > > @@ -87,15 +82,14 @@ AC_DEFUN([AC_HAVE_GETMNTENT], > # > AC_DEFUN([AC_HAVE_GETMNTINFO], > [ AC_MSG_CHECKING([for getmntinfo ]) > - AC_TRY_COMPILE([ > + AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ > #include > #include > #include > - ], [ > + ]], [[ > getmntinfo(0, 0); > - ], have_getmntinfo=yes > - AC_MSG_RESULT(yes), > - AC_MSG_RESULT(no)) > + ]])],[have_getmntinfo=yes > + AC_MSG_RESULT(yes)],[AC_MSG_RESULT(no)]) > AC_SUBST(have_getmntinfo) > ]) > > @@ -105,15 +99,14 @@ AC_DEFUN([AC_HAVE_GETMNTINFO], > # > AC_DEFUN([AC_HAVE_COPY_FILE_RANGE], > [ AC_MSG_CHECKING([for copy_file_range]) > - AC_TRY_LINK([ > + AC_LINK_IFELSE([AC_LANG_PROGRAM([[ > #define _GNU_SOURCE > #include > #include > - ], [ > + ]], [[ > syscall(__NR_copy_file_range, 0, 0, 0, 0, 0, 0); > - ], have_copy_file_range=yes > - AC_MSG_RESULT(yes), > - AC_MSG_RESULT(no)) > + ]])],[have_copy_file_range=yes > + AC_MSG_RESULT(yes)],[AC_MSG_RESULT(no)]) > AC_SUBST(have_copy_file_range) > ]) > > diff --git a/m4/package_ncurses.m4 b/m4/package_ncurses.m4 > index 83a2c382..b220dd64 100644 > --- a/m4/package_ncurses.m4 > +++ b/m4/package_ncurses.m4 > @@ -19,12 +19,9 @@ AC_DEFUN([AC_PACKAGE_WANT_WORKING_LIBNCURSES], > libcurses="-lncurses" > LIBS="$LIBS $libcurses" > CFLAGS="$CFLAGS -D_GNU_SOURCE" > - AC_TRY_LINK([ > + AC_LINK_IFELSE([AC_LANG_PROGRAM([[ > #include > -#include ], > - [wgetch(stdscr);], > - [enable_curses=yes; AC_MSG_RESULT([ok])], > - [enable_curses=no; libcurses=""; AC_MSG_RESULT([disabling curses])]) > +#include ]], [[wgetch(stdscr);]])],[enable_curses=yes; AC_MSG_RESULT([ok])],[enable_curses=no; libcurses=""; AC_MSG_RESULT([disabling curses])]) > AC_SUBST(enable_curses) > AC_SUBST(libcurses) > ]) > diff --git a/m4/package_xfslibs.m4 b/m4/package_xfslibs.m4 > index 9be56e11..0746cd1d 100644 > --- a/m4/package_xfslibs.m4 > +++ b/m4/package_xfslibs.m4 > @@ -94,13 +94,10 @@ AC_DEFUN([AC_PACKAGE_NEED_ATTRLIST_LIBHANDLE], > > AC_DEFUN([AC_PACKAGE_NEED_XFSCTL_MACRO], > [ AC_MSG_CHECKING([xfsctl from xfs/xfs.h]) > - AC_TRY_LINK([ > + AC_LINK_IFELSE([AC_LANG_PROGRAM([[ > #define _GNU_SOURCE > #define _FILE_OFFSET_BITS 64 > -#include ], > - [ int x = xfsctl(0, 0, 0, 0); ], > - [ echo ok ], > - [ echo > +#include ]], [[ int x = xfsctl(0, 0, 0, 0); ]])],[ echo ok ],[ echo > echo 'FATAL ERROR: cannot find required macros in the XFS headers.' > echo 'Upgrade your XFS programs (xfsprogs) development package.' > echo 'Alternatively, run "make install-dev" from the xfsprogs source.' > -- > 2.33.0 >