[PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Naohiro Tamura]

This patch replaced obsoleted AC_HELP_STRING to AS_HELP_STRING.
As of Glibc 2.33, INSTALL says "GNU 'autoconf' 2.69 (exactly)" is
required. So we can safely replace obsoleted macro without worrying
about older autoconf.
This change doesn't affect the contents of "configure" file.
---
 configure.ac | 60 ++++++++++++++++++++++++++--------------------------
 1 file changed, 30 insertions(+), 30 deletions(-)

diff --git a/configure.ac b/configure.ac
index dfebb8a7cc..b3f317f86b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -90,7 +90,7 @@ config_vars=
 
 # Check for a --with-gd argument and set libgd-LDFLAGS in config.make.
 AC_ARG_WITH([gd],
-	    AC_HELP_STRING([--with-gd=DIR],
+	    AS_HELP_STRING([--with-gd=DIR],
 			   [find libgd include dir and library with prefix DIR]),
 	    [dnl
 case "$with_gd" in
@@ -100,7 +100,7 @@ yes|''|no) ;;
 esac
 ])
 AC_ARG_WITH([gd-include],
-	    AC_HELP_STRING([--with-gd-include=DIR],
+	    AS_HELP_STRING([--with-gd-include=DIR],
 			   [find libgd include files in DIR]),
 	    [dnl
 case "$with_gd_include" in
@@ -109,7 +109,7 @@ case "$with_gd_include" in
 esac
 ])
 AC_ARG_WITH([gd-lib],
-	    AC_HELP_STRING([--with-gd-lib=DIR],
+	    AS_HELP_STRING([--with-gd-lib=DIR],
 			   [find libgd library files in DIR]),
 	    [dnl
 case "$with_gd_lib" in
@@ -129,18 +129,18 @@ fi
 
 dnl Arguments to specify presence of other packages/features.
 AC_ARG_WITH([binutils],
-	    AC_HELP_STRING([--with-binutils=PATH],
+	    AS_HELP_STRING([--with-binutils=PATH],
 			   [specify location of binutils (as and ld)]),
 	    [path_binutils=$withval],
 	    [path_binutils=''])
 AC_ARG_WITH([selinux],
-	    AC_HELP_STRING([--with-selinux],
+	    AS_HELP_STRING([--with-selinux],
 			   [if building with SELinux support]),
 	    [with_selinux=$withval],
 	    [with_selinux=auto])
 
 AC_ARG_WITH([headers],
-	    AC_HELP_STRING([--with-headers=PATH],
+	    AS_HELP_STRING([--with-headers=PATH],
 			   [location of system headers to use
 			    (for example /usr/src/linux/include)
 			    @<:@default=compiler default@:>@]),
@@ -150,56 +150,56 @@ AC_SUBST(sysheaders)
 
 AC_SUBST(use_default_link)
 AC_ARG_WITH([default-link],
-	    AC_HELP_STRING([--with-default-link],
+	    AS_HELP_STRING([--with-default-link],
 			   [do not use explicit linker scripts]),
 	    [use_default_link=$withval],
 	    [use_default_link=default])
 
 dnl Additional build flags injection.
 AC_ARG_WITH([nonshared-cflags],
-	    AC_HELP_STRING([--with-nonshared-cflags=CFLAGS],
+	    AS_HELP_STRING([--with-nonshared-cflags=CFLAGS],
 			   [build nonshared libraries with additional CFLAGS]),
 	    [extra_nonshared_cflags=$withval],
 	    [extra_nonshared_cflags=])
 AC_SUBST(extra_nonshared_cflags)
 
 AC_ARG_ENABLE([sanity-checks],
-	      AC_HELP_STRING([--disable-sanity-checks],
+	      AS_HELP_STRING([--disable-sanity-checks],
 			     [really do not use threads (should not be used except in special situations) @<:@default=yes@:>@]),
 	      [enable_sanity=$enableval],
 	      [enable_sanity=yes])
 
 AC_ARG_ENABLE([shared],
-	      AC_HELP_STRING([--enable-shared],
+	      AS_HELP_STRING([--enable-shared],
 			     [build shared library @<:@default=yes if GNU ld@:>@]),
 	      [shared=$enableval],
 	      [shared=yes])
 AC_ARG_ENABLE([profile],
-	      AC_HELP_STRING([--enable-profile],
+	      AS_HELP_STRING([--enable-profile],
 			     [build profiled library @<:@default=no@:>@]),
 	      [profile=$enableval],
 	      [profile=no])
 AC_ARG_ENABLE([static-pie],
-	      AC_HELP_STRING([--enable-static-pie],
+	      AS_HELP_STRING([--enable-static-pie],
 			     [enable static PIE support and use it in the testsuite @<:@default=no@:>@]),
 	      [static_pie=$enableval],
 	      [static_pie=no])
 AC_ARG_ENABLE([timezone-tools],
-	      AC_HELP_STRING([--disable-timezone-tools],
+	      AS_HELP_STRING([--disable-timezone-tools],
 			     [do not install timezone tools @<:@default=install@:>@]),
 	      [enable_timezone_tools=$enableval],
 	      [enable_timezone_tools=yes])
 AC_SUBST(enable_timezone_tools)
 
 AC_ARG_ENABLE([hardcoded-path-in-tests],
-	      AC_HELP_STRING([--enable-hardcoded-path-in-tests],
+	      AS_HELP_STRING([--enable-hardcoded-path-in-tests],
 			     [hardcode newly built glibc path in tests @<:@default=no@:>@]),
 	      [hardcoded_path_in_tests=$enableval],
 	      [hardcoded_path_in_tests=no])
 AC_SUBST(hardcoded_path_in_tests)
 
 AC_ARG_ENABLE([stackguard-randomization],
-	      AC_HELP_STRING([--enable-stackguard-randomization],
+	      AS_HELP_STRING([--enable-stackguard-randomization],
 			     [initialize __stack_chk_guard canary with a random number at program start]),
 	      [enable_stackguard_randomize=$enableval],
 	      [enable_stackguard_randomize=no])
@@ -208,7 +208,7 @@ if test "$enable_stackguard_randomize" = yes; then
 fi
 
 AC_ARG_ENABLE([hidden-plt],
-	      AC_HELP_STRING([--disable-hidden-plt],
+	      AS_HELP_STRING([--disable-hidden-plt],
 			     [do not hide internal function calls to avoid PLT]),
 	      [hidden=$enableval],
 	      [hidden=yes])
@@ -217,7 +217,7 @@ if test "x$hidden" = xno; then
 fi
 
 AC_ARG_ENABLE([bind-now],
-	      AC_HELP_STRING([--enable-bind-now],
+	      AS_HELP_STRING([--enable-bind-now],
 			     [disable lazy relocations in DSOs]),
 	      [bindnow=$enableval],
 	      [bindnow=no])
@@ -229,7 +229,7 @@ fi
 dnl Build glibc with -fstack-protector, -fstack-protector-all, or
 dnl -fstack-protector-strong.
 AC_ARG_ENABLE([stack-protector],
-	      AC_HELP_STRING([--enable-stack-protector=@<:@yes|no|all|strong@:>@],
+	      AS_HELP_STRING([--enable-stack-protector=@<:@yes|no|all|strong@:>@],
 			     [Use -fstack-protector[-all|-strong] to detect glibc buffer overflows]),
 	      [enable_stack_protector=$enableval],
 	      [enable_stack_protector=no])
@@ -241,7 +241,7 @@ esac
 dnl On some platforms we cannot use dynamic loading.  We must provide
 dnl static NSS modules.
 AC_ARG_ENABLE([static-nss],
-	      AC_HELP_STRING([--enable-static-nss],
+	      AS_HELP_STRING([--enable-static-nss],
 			     [build static NSS modules @<:@default=no@:>@]),
 	      [static_nss=$enableval],
 	      [static_nss=no])
@@ -252,14 +252,14 @@ if test x"$static_nss" = xyes || test x"$shared" = xno; then
 fi
 
 AC_ARG_ENABLE([force-install],
-	      AC_HELP_STRING([--disable-force-install],
+	      AS_HELP_STRING([--disable-force-install],
 			     [don't force installation of files from this package, even if they are older than the installed files]),
 	      [force_install=$enableval],
 	      [force_install=yes])
 AC_SUBST(force_install)
 
 AC_ARG_ENABLE([maintainer-mode],
-	      AC_HELP_STRING([--enable-maintainer-mode],
+	      AS_HELP_STRING([--enable-maintainer-mode],
 			     [enable make rules and dependencies not useful (and sometimes confusing) to the casual installer]),
 	      [maintainer=$enableval],
 	      [maintainer=no])
@@ -267,7 +267,7 @@ AC_ARG_ENABLE([maintainer-mode],
 dnl On some platforms we allow dropping compatibility with all kernel
 dnl versions.
 AC_ARG_ENABLE([kernel],
-	      AC_HELP_STRING([--enable-kernel=VERSION],
+	      AS_HELP_STRING([--enable-kernel=VERSION],
 			     [compile for compatibility with kernel not older than VERSION]),
 	      [minimum_kernel=$enableval],
 	      [])
@@ -285,34 +285,34 @@ dnl For the development we sometimes want gcc to issue even more warnings.
 dnl This is not the default since many of the extra warnings are not
 dnl appropriate.
 AC_ARG_ENABLE([all-warnings],
-	      AC_HELP_STRING([--enable-all-warnings],
+	      AS_HELP_STRING([--enable-all-warnings],
 			     [enable all useful warnings gcc can issue]),
 	      [all_warnings=$enableval],
 	      [])
 AC_SUBST(all_warnings)
 
 AC_ARG_ENABLE([werror],
-	      AC_HELP_STRING([--disable-werror],
+	      AS_HELP_STRING([--disable-werror],
 			     [do not build with -Werror]),
 	      [enable_werror=$enableval],
 	      [enable_werror=yes])
 AC_SUBST(enable_werror)
 
 AC_ARG_ENABLE([multi-arch],
-	      AC_HELP_STRING([--enable-multi-arch],
+	      AS_HELP_STRING([--enable-multi-arch],
 			     [enable single DSO with optimizations for multiple architectures]),
 	      [multi_arch=$enableval],
 	      [multi_arch=default])
 
 AC_ARG_ENABLE([experimental-malloc],
-	      AC_HELP_STRING([--disable-experimental-malloc],
+	      AS_HELP_STRING([--disable-experimental-malloc],
 			     [disable experimental malloc features]),
 	      [experimental_malloc=$enableval],
 	      [experimental_malloc=yes])
 AC_SUBST(experimental_malloc)
 
 AC_ARG_ENABLE([memory-tagging],
-	      AC_HELP_STRING([--enable-memory-tagging],
+	      AS_HELP_STRING([--enable-memory-tagging],
 			     [enable memory tagging if supported by the architecture @<:@default=no@:>@]),
 	      [memory_tagging=$enableval],
 	      [memory_tagging=no])
@@ -327,14 +327,14 @@ fi
 AC_SUBST(memory_tagging)
 
 AC_ARG_ENABLE([crypt],
-              AC_HELP_STRING([--disable-crypt],
+              AS_HELP_STRING([--disable-crypt],
                              [do not build nor install the passphrase hashing library, libcrypt]),
               [build_crypt=$enableval],
               [build_crypt=yes])
 AC_SUBST(build_crypt)
 
 AC_ARG_ENABLE([nss-crypt],
-	      AC_HELP_STRING([--enable-nss-crypt],
+	      AS_HELP_STRING([--enable-nss-crypt],
 			     [enable libcrypt to use nss]),
 	      [nss_crypt=$enableval],
 	      [nss_crypt=no])
@@ -474,7 +474,7 @@ AC_TRY_COMPILE([], [
 	       [libc_cv_compiler_default_cet=no])
 
 AC_ARG_ENABLE([cet],
-	      AC_HELP_STRING([--enable-cet],
+	      AS_HELP_STRING([--enable-cet],
 			     [enable Intel Control-flow Enforcement Technology (CET), x86 only]),
 	      [enable_cet=$enableval],
 	      [enable_cet=$libc_cv_compiler_default_cet])
-- 
2.17.1

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Adhemerval Zanella]

On 15/02/2021 09:31, Naohiro Tamura wrote:
> This patch replaced obsoleted AC_HELP_STRING to AS_HELP_STRING.
> As of Glibc 2.33, INSTALL says "GNU 'autoconf' 2.69 (exactly)" is
> required. So we can safely replace obsoleted macro without worrying
> about older autoconf.
> This change doesn't affect the contents of "configure" file.

If this does not change or improve configure, why change configure.ac
then?

> ---
>  configure.ac | 60 ++++++++++++++++++++++++++--------------------------
>  1 file changed, 30 insertions(+), 30 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index dfebb8a7cc..b3f317f86b 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -90,7 +90,7 @@ config_vars=
>  
>  # Check for a --with-gd argument and set libgd-LDFLAGS in config.make.
>  AC_ARG_WITH([gd],
> -	    AC_HELP_STRING([--with-gd=DIR],
> +	    AS_HELP_STRING([--with-gd=DIR],
>  			   [find libgd include dir and library with prefix DIR]),
>  	    [dnl
>  case "$with_gd" in
> @@ -100,7 +100,7 @@ yes|''|no) ;;
>  esac
>  ])
>  AC_ARG_WITH([gd-include],
> -	    AC_HELP_STRING([--with-gd-include=DIR],
> +	    AS_HELP_STRING([--with-gd-include=DIR],
>  			   [find libgd include files in DIR]),
>  	    [dnl
>  case "$with_gd_include" in
> @@ -109,7 +109,7 @@ case "$with_gd_include" in
>  esac
>  ])
>  AC_ARG_WITH([gd-lib],
> -	    AC_HELP_STRING([--with-gd-lib=DIR],
> +	    AS_HELP_STRING([--with-gd-lib=DIR],
>  			   [find libgd library files in DIR]),
>  	    [dnl
>  case "$with_gd_lib" in
> @@ -129,18 +129,18 @@ fi
>  
>  dnl Arguments to specify presence of other packages/features.
>  AC_ARG_WITH([binutils],
> -	    AC_HELP_STRING([--with-binutils=PATH],
> +	    AS_HELP_STRING([--with-binutils=PATH],
>  			   [specify location of binutils (as and ld)]),
>  	    [path_binutils=$withval],
>  	    [path_binutils=''])
>  AC_ARG_WITH([selinux],
> -	    AC_HELP_STRING([--with-selinux],
> +	    AS_HELP_STRING([--with-selinux],
>  			   [if building with SELinux support]),
>  	    [with_selinux=$withval],
>  	    [with_selinux=auto])
>  
>  AC_ARG_WITH([headers],
> -	    AC_HELP_STRING([--with-headers=PATH],
> +	    AS_HELP_STRING([--with-headers=PATH],
>  			   [location of system headers to use
>  			    (for example /usr/src/linux/include)
>  			    @<:@default=compiler default@:>@]),
> @@ -150,56 +150,56 @@ AC_SUBST(sysheaders)
>  
>  AC_SUBST(use_default_link)
>  AC_ARG_WITH([default-link],
> -	    AC_HELP_STRING([--with-default-link],
> +	    AS_HELP_STRING([--with-default-link],
>  			   [do not use explicit linker scripts]),
>  	    [use_default_link=$withval],
>  	    [use_default_link=default])
>  
>  dnl Additional build flags injection.
>  AC_ARG_WITH([nonshared-cflags],
> -	    AC_HELP_STRING([--with-nonshared-cflags=CFLAGS],
> +	    AS_HELP_STRING([--with-nonshared-cflags=CFLAGS],
>  			   [build nonshared libraries with additional CFLAGS]),
>  	    [extra_nonshared_cflags=$withval],
>  	    [extra_nonshared_cflags=])
>  AC_SUBST(extra_nonshared_cflags)
>  
>  AC_ARG_ENABLE([sanity-checks],
> -	      AC_HELP_STRING([--disable-sanity-checks],
> +	      AS_HELP_STRING([--disable-sanity-checks],
>  			     [really do not use threads (should not be used except in special situations) @<:@default=yes@:>@]),
>  	      [enable_sanity=$enableval],
>  	      [enable_sanity=yes])
>  
>  AC_ARG_ENABLE([shared],
> -	      AC_HELP_STRING([--enable-shared],
> +	      AS_HELP_STRING([--enable-shared],
>  			     [build shared library @<:@default=yes if GNU ld@:>@]),
>  	      [shared=$enableval],
>  	      [shared=yes])
>  AC_ARG_ENABLE([profile],
> -	      AC_HELP_STRING([--enable-profile],
> +	      AS_HELP_STRING([--enable-profile],
>  			     [build profiled library @<:@default=no@:>@]),
>  	      [profile=$enableval],
>  	      [profile=no])
>  AC_ARG_ENABLE([static-pie],
> -	      AC_HELP_STRING([--enable-static-pie],
> +	      AS_HELP_STRING([--enable-static-pie],
>  			     [enable static PIE support and use it in the testsuite @<:@default=no@:>@]),
>  	      [static_pie=$enableval],
>  	      [static_pie=no])
>  AC_ARG_ENABLE([timezone-tools],
> -	      AC_HELP_STRING([--disable-timezone-tools],
> +	      AS_HELP_STRING([--disable-timezone-tools],
>  			     [do not install timezone tools @<:@default=install@:>@]),
>  	      [enable_timezone_tools=$enableval],
>  	      [enable_timezone_tools=yes])
>  AC_SUBST(enable_timezone_tools)
>  
>  AC_ARG_ENABLE([hardcoded-path-in-tests],
> -	      AC_HELP_STRING([--enable-hardcoded-path-in-tests],
> +	      AS_HELP_STRING([--enable-hardcoded-path-in-tests],
>  			     [hardcode newly built glibc path in tests @<:@default=no@:>@]),
>  	      [hardcoded_path_in_tests=$enableval],
>  	      [hardcoded_path_in_tests=no])
>  AC_SUBST(hardcoded_path_in_tests)
>  
>  AC_ARG_ENABLE([stackguard-randomization],
> -	      AC_HELP_STRING([--enable-stackguard-randomization],
> +	      AS_HELP_STRING([--enable-stackguard-randomization],
>  			     [initialize __stack_chk_guard canary with a random number at program start]),
>  	      [enable_stackguard_randomize=$enableval],
>  	      [enable_stackguard_randomize=no])
> @@ -208,7 +208,7 @@ if test "$enable_stackguard_randomize" = yes; then
>  fi
>  
>  AC_ARG_ENABLE([hidden-plt],
> -	      AC_HELP_STRING([--disable-hidden-plt],
> +	      AS_HELP_STRING([--disable-hidden-plt],
>  			     [do not hide internal function calls to avoid PLT]),
>  	      [hidden=$enableval],
>  	      [hidden=yes])
> @@ -217,7 +217,7 @@ if test "x$hidden" = xno; then
>  fi
>  
>  AC_ARG_ENABLE([bind-now],
> -	      AC_HELP_STRING([--enable-bind-now],
> +	      AS_HELP_STRING([--enable-bind-now],
>  			     [disable lazy relocations in DSOs]),
>  	      [bindnow=$enableval],
>  	      [bindnow=no])
> @@ -229,7 +229,7 @@ fi
>  dnl Build glibc with -fstack-protector, -fstack-protector-all, or
>  dnl -fstack-protector-strong.
>  AC_ARG_ENABLE([stack-protector],
> -	      AC_HELP_STRING([--enable-stack-protector=@<:@yes|no|all|strong@:>@],
> +	      AS_HELP_STRING([--enable-stack-protector=@<:@yes|no|all|strong@:>@],
>  			     [Use -fstack-protector[-all|-strong] to detect glibc buffer overflows]),
>  	      [enable_stack_protector=$enableval],
>  	      [enable_stack_protector=no])
> @@ -241,7 +241,7 @@ esac
>  dnl On some platforms we cannot use dynamic loading.  We must provide
>  dnl static NSS modules.
>  AC_ARG_ENABLE([static-nss],
> -	      AC_HELP_STRING([--enable-static-nss],
> +	      AS_HELP_STRING([--enable-static-nss],
>  			     [build static NSS modules @<:@default=no@:>@]),
>  	      [static_nss=$enableval],
>  	      [static_nss=no])
> @@ -252,14 +252,14 @@ if test x"$static_nss" = xyes || test x"$shared" = xno; then
>  fi
>  
>  AC_ARG_ENABLE([force-install],
> -	      AC_HELP_STRING([--disable-force-install],
> +	      AS_HELP_STRING([--disable-force-install],
>  			     [don't force installation of files from this package, even if they are older than the installed files]),
>  	      [force_install=$enableval],
>  	      [force_install=yes])
>  AC_SUBST(force_install)
>  
>  AC_ARG_ENABLE([maintainer-mode],
> -	      AC_HELP_STRING([--enable-maintainer-mode],
> +	      AS_HELP_STRING([--enable-maintainer-mode],
>  			     [enable make rules and dependencies not useful (and sometimes confusing) to the casual installer]),
>  	      [maintainer=$enableval],
>  	      [maintainer=no])
> @@ -267,7 +267,7 @@ AC_ARG_ENABLE([maintainer-mode],
>  dnl On some platforms we allow dropping compatibility with all kernel
>  dnl versions.
>  AC_ARG_ENABLE([kernel],
> -	      AC_HELP_STRING([--enable-kernel=VERSION],
> +	      AS_HELP_STRING([--enable-kernel=VERSION],
>  			     [compile for compatibility with kernel not older than VERSION]),
>  	      [minimum_kernel=$enableval],
>  	      [])
> @@ -285,34 +285,34 @@ dnl For the development we sometimes want gcc to issue even more warnings.
>  dnl This is not the default since many of the extra warnings are not
>  dnl appropriate.
>  AC_ARG_ENABLE([all-warnings],
> -	      AC_HELP_STRING([--enable-all-warnings],
> +	      AS_HELP_STRING([--enable-all-warnings],
>  			     [enable all useful warnings gcc can issue]),
>  	      [all_warnings=$enableval],
>  	      [])
>  AC_SUBST(all_warnings)
>  
>  AC_ARG_ENABLE([werror],
> -	      AC_HELP_STRING([--disable-werror],
> +	      AS_HELP_STRING([--disable-werror],
>  			     [do not build with -Werror]),
>  	      [enable_werror=$enableval],
>  	      [enable_werror=yes])
>  AC_SUBST(enable_werror)
>  
>  AC_ARG_ENABLE([multi-arch],
> -	      AC_HELP_STRING([--enable-multi-arch],
> +	      AS_HELP_STRING([--enable-multi-arch],
>  			     [enable single DSO with optimizations for multiple architectures]),
>  	      [multi_arch=$enableval],
>  	      [multi_arch=default])
>  
>  AC_ARG_ENABLE([experimental-malloc],
> -	      AC_HELP_STRING([--disable-experimental-malloc],
> +	      AS_HELP_STRING([--disable-experimental-malloc],
>  			     [disable experimental malloc features]),
>  	      [experimental_malloc=$enableval],
>  	      [experimental_malloc=yes])
>  AC_SUBST(experimental_malloc)
>  
>  AC_ARG_ENABLE([memory-tagging],
> -	      AC_HELP_STRING([--enable-memory-tagging],
> +	      AS_HELP_STRING([--enable-memory-tagging],
>  			     [enable memory tagging if supported by the architecture @<:@default=no@:>@]),
>  	      [memory_tagging=$enableval],
>  	      [memory_tagging=no])
> @@ -327,14 +327,14 @@ fi
>  AC_SUBST(memory_tagging)
>  
>  AC_ARG_ENABLE([crypt],
> -              AC_HELP_STRING([--disable-crypt],
> +              AS_HELP_STRING([--disable-crypt],
>                               [do not build nor install the passphrase hashing library, libcrypt]),
>                [build_crypt=$enableval],
>                [build_crypt=yes])
>  AC_SUBST(build_crypt)
>  
>  AC_ARG_ENABLE([nss-crypt],
> -	      AC_HELP_STRING([--enable-nss-crypt],
> +	      AS_HELP_STRING([--enable-nss-crypt],
>  			     [enable libcrypt to use nss]),
>  	      [nss_crypt=$enableval],
>  	      [nss_crypt=no])
> @@ -474,7 +474,7 @@ AC_TRY_COMPILE([], [
>  	       [libc_cv_compiler_default_cet=no])
>  
>  AC_ARG_ENABLE([cet],
> -	      AC_HELP_STRING([--enable-cet],
> +	      AS_HELP_STRING([--enable-cet],
>  			     [enable Intel Control-flow Enforcement Technology (CET), x86 only]),
>  	      [enable_cet=$enableval],
>  	      [enable_cet=$libc_cv_compiler_default_cet])
> 

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Mike Frysinger]

On 15 Feb 2021 09:59, Adhemerval Zanella via Libc-alpha wrote:
> On 15/02/2021 09:31, Naohiro Tamura wrote:
> > This patch replaced obsoleted AC_HELP_STRING to AS_HELP_STRING.
> > As of Glibc 2.33, INSTALL says "GNU 'autoconf' 2.69 (exactly)" is
> > required. So we can safely replace obsoleted macro without worrying
> > about older autoconf.
> > This change doesn't affect the contents of "configure" file.
> 
> If this does not change or improve configure, why change configure.ac
> then?

why wait for autoconf to start warning that we're using a deprecated
macro and failing the build or confusing devs ?

patch LGTM
-mike

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Adhemerval Zanella]

On 15/02/2021 10:29, Mike Frysinger wrote:
> On 15 Feb 2021 09:59, Adhemerval Zanella via Libc-alpha wrote:
>> On 15/02/2021 09:31, Naohiro Tamura wrote:
>>> This patch replaced obsoleted AC_HELP_STRING to AS_HELP_STRING.
>>> As of Glibc 2.33, INSTALL says "GNU 'autoconf' 2.69 (exactly)" is
>>> required. So we can safely replace obsoleted macro without worrying
>>> about older autoconf.
>>> This change doesn't affect the contents of "configure" file.
>>
>> If this does not change or improve configure, why change configure.ac
>> then?
> 
> why wait for autoconf to start warning that we're using a deprecated
> macro and failing the build or confusing devs ?
> 
> patch LGTM

Because we use a macro to test for the specific autoconf version:

aclocal.m4:

  5 m4_define([GLIBC_AUTOCONF_VERSION], [2.69])
  6 m4_if(m4_defn([AC_AUTOCONF_VERSION]), GLIBC_AUTOCONF_VERSION, [],
  7       [m4_fatal(m4_flatten(
  8 Exactly version GLIBC_AUTOCONF_VERSION of Autoconf is required but you have
  9 m4_defn([AC_AUTOCONF_VERSION])

So autoconf will start to fail or dumping a warning only if the
distro actually patch it to do so.

I don't have a strong opinion about this patch, but due the *current*
constraints of the auto-generation (which requires an specific autoconf
version), this patch does not improve anything really.

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Mike Frysinger]

On 15 Feb 2021 10:39, Adhemerval Zanella wrote:
> On 15/02/2021 10:29, Mike Frysinger wrote:
> > On 15 Feb 2021 09:59, Adhemerval Zanella via Libc-alpha wrote:
> >> On 15/02/2021 09:31, Naohiro Tamura wrote:
> >>> This patch replaced obsoleted AC_HELP_STRING to AS_HELP_STRING.
> >>> As of Glibc 2.33, INSTALL says "GNU 'autoconf' 2.69 (exactly)" is
> >>> required. So we can safely replace obsoleted macro without worrying
> >>> about older autoconf.
> >>> This change doesn't affect the contents of "configure" file.
> >>
> >> If this does not change or improve configure, why change configure.ac
> >> then?
> > 
> > why wait for autoconf to start warning that we're using a deprecated
> > macro and failing the build or confusing devs ?
> > 
> > patch LGTM
> 
> Because we use a macro to test for the specific autoconf version:

that's not an argument for continuing to use deprecated macros

> aclocal.m4:
> 
>   5 m4_define([GLIBC_AUTOCONF_VERSION], [2.69])
>   6 m4_if(m4_defn([AC_AUTOCONF_VERSION]), GLIBC_AUTOCONF_VERSION, [],
>   7       [m4_fatal(m4_flatten(
>   8 Exactly version GLIBC_AUTOCONF_VERSION of Autoconf is required but you have
>   9 m4_defn([AC_AUTOCONF_VERSION])
> 
> So autoconf will start to fail or dumping a warning only if the
> distro actually patch it to do so.
> 
> I don't have a strong opinion about this patch, but due the *current*
> constraints of the auto-generation (which requires an specific autoconf
> version), this patch does not improve anything really.

the macro is clearly deprecated upstream.  the version we're pinned to supports
the non-deprecated version.  i don't see why we're debating this.

it's a lot easier to upgrade versions when you don't have to dig through a pile
of warnings in the process.  if people want to do migrations ahead of time (and
have even written the patches), then let's merge them.
-mike

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Adhemerval Zanella]

On 15/02/2021 12:38, Mike Frysinger wrote:
> On 15 Feb 2021 10:39, Adhemerval Zanella wrote:
>> On 15/02/2021 10:29, Mike Frysinger wrote:
>>> On 15 Feb 2021 09:59, Adhemerval Zanella via Libc-alpha wrote:
>>>> On 15/02/2021 09:31, Naohiro Tamura wrote:
>>>>> This patch replaced obsoleted AC_HELP_STRING to AS_HELP_STRING.
>>>>> As of Glibc 2.33, INSTALL says "GNU 'autoconf' 2.69 (exactly)" is
>>>>> required. So we can safely replace obsoleted macro without worrying
>>>>> about older autoconf.
>>>>> This change doesn't affect the contents of "configure" file.
>>>>
>>>> If this does not change or improve configure, why change configure.ac
>>>> then?
>>>
>>> why wait for autoconf to start warning that we're using a deprecated
>>> macro and failing the build or confusing devs ?
>>>
>>> patch LGTM
>>
>> Because we use a macro to test for the specific autoconf version:
> 
> that's not an argument for continuing to use deprecated macros
> 
>> aclocal.m4:
>>
>>   5 m4_define([GLIBC_AUTOCONF_VERSION], [2.69])
>>   6 m4_if(m4_defn([AC_AUTOCONF_VERSION]), GLIBC_AUTOCONF_VERSION, [],
>>   7       [m4_fatal(m4_flatten(
>>   8 Exactly version GLIBC_AUTOCONF_VERSION of Autoconf is required but you have
>>   9 m4_defn([AC_AUTOCONF_VERSION])
>>
>> So autoconf will start to fail or dumping a warning only if the
>> distro actually patch it to do so.
>>
>> I don't have a strong opinion about this patch, but due the *current*
>> constraints of the auto-generation (which requires an specific autoconf
>> version), this patch does not improve anything really.
> 
> the macro is clearly deprecated upstream.  the version we're pinned to supports
> the non-deprecated version.  i don't see why we're debating this.
> 
> it's a lot easier to upgrade versions when you don't have to dig through a pile
> of warnings in the process.  if people want to do migrations ahead of time (and
> have even written the patches), then let's merge them.

Again, I don't have a strong opinion about it so I not following your
slight aggressive tone here.  I just initially asked *why* change it,
since we already have a specific check for the version so any version
migration would require to check if these macros indeed would break
something (and I agree with you that this would make easier to upgrade).

In any case, it seems that the configure.ac already use a mix of both
the macro and the deprecated one, so it should be fix. Also please fix
it on the aclocal.m4 as well.

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Florian Weimer]

* Adhemerval Zanella via Libc-alpha:

> Again, I don't have a strong opinion about it so I not following your
> slight aggressive tone here.  I just initially asked *why* change it,
> since we already have a specific check for the version so any version
> migration would require to check if these macros indeed would break
> something (and I agree with you that this would make easier to upgrade).

I must say I find it odd that we build with -Werror, try to address all
kinds of compiler warnings, and then push back against fixing warnings
in other places.

If it's possible to get rid of the warning while retaining compatibility
with 2.69 for the time being, then let's do it?

Thanks,
Florian
-- 
Red Hat GmbH, https://de.redhat.com/ , Registered seat: Grasbrunn,
Commercial register: Amtsgericht Muenchen, HRB 153243,
Managing Directors: Charles Cachera, Brian Klemm, Laurie Krebs, Michael O'Neill

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Adhemerval Zanella]

On 15/02/2021 14:17, Florian Weimer wrote:
> * Adhemerval Zanella via Libc-alpha:
> 
>> Again, I don't have a strong opinion about it so I not following your
>> slight aggressive tone here.  I just initially asked *why* change it,
>> since we already have a specific check for the version so any version
>> migration would require to check if these macros indeed would break
>> something (and I agree with you that this would make easier to upgrade).
> 
> I must say I find it odd that we build with -Werror, try to address all
> kinds of compiler warnings, and then push back against fixing warnings
> in other places.
> 
> If it's possible to get rid of the warning while retaining compatibility
> with 2.69 for the time being, then let's do it?
> 
> Thanks,
> Florian
> 

Fair enough, I withdraw my objections.

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Mike Frysinger]

On 15 Feb 2021 13:39, Adhemerval Zanella wrote:
> Again, I don't have a strong opinion about it so I not following your
> slight aggressive tone here.

apologies if it comes across that way.  i'm just (in my view) stating
facts (they're deprecated) and that alone is sufficient justification.

> I just initially asked *why* change it,

as stated in the original patch: it's an obsolete macro, and there's a
not-obsolete replacement.  seems like we should always merge this sort
of thing ?

> Also please fix it on the aclocal.m4 as well.

aclocal only uses AS_HELP_STRING that i can see.  where are you seeing
use of that AC variant ?

on a somewhat related note, i can't help but feel like we should switch
to using aclocal to manage aclocal.m4 too.  splitting it up into sep m4
files under m4/ should help with the gnarliness of the language ?
-mike

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Adhemerval Zanella]

On 15/02/2021 17:21, Mike Frysinger wrote:
> On 15 Feb 2021 13:39, Adhemerval Zanella wrote:
>> Again, I don't have a strong opinion about it so I not following your
>> slight aggressive tone here.
> 
> apologies if it comes across that way.  i'm just (in my view) stating
> facts (they're deprecated) and that alone is sufficient justification.
> 
>> I just initially asked *why* change it,
> 
> as stated in the original patch: it's an obsolete macro, and there's a
> not-obsolete replacement.  seems like we should always merge this sort
> of thing ?
> 
>> Also please fix it on the aclocal.m4 as well.
> 
> aclocal only uses AS_HELP_STRING that i can see.  where are you seeing
> use of that AC variant ?
> 

It was a mistake from my part.

> on a somewhat related note, i can't help but feel like we should switch
> to using aclocal to manage aclocal.m4 too.  splitting it up into sep m4
> files under m4/ should help with the gnarliness of the language ?
> -mike
> 

RE: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[naohirot]

Adhemerval-san, Mike-san, Florian-san,

Thank you for all comments!
This is my second patch to glibc community, which I found during
thinking about how to check compiler option '-march=armv8.2-a+sve' to
add a64fx SoC multiarch code.

I took that consensus has been make among maintainers :-)
So I'd like to contribute to replace other obsolete macros too,
AC_TRY_COMPILE, AC_TRY_LINK and AC_CHECK_TOOL_PREFIX.

Thanks.
Naohiro

> -----Original Message-----
> From: Adhemerval Zanella <adhemerval.zanella@linaro.org>
> Sent: Tuesday, February 16, 2021 5:31 AM
> To: Tamura, Naohiro/田村 直広 <naohirot@fujitsu.com>;
> libc-alpha@sourceware.org; Mike Frysinger <vapier@gentoo.org>
> Subject: Re: [PATCH] Replaced obsoleted AC_HELP_STRING to
> AS_HELP_STRING
> 
> 
> 
> On 15/02/2021 17:21, Mike Frysinger wrote:
> > On 15 Feb 2021 13:39, Adhemerval Zanella wrote:
> >> Again, I don't have a strong opinion about it so I not following your
> >> slight aggressive tone here.
> >
> > apologies if it comes across that way.  i'm just (in my view) stating
> > facts (they're deprecated) and that alone is sufficient justification.
> >
> >> I just initially asked *why* change it,
> >
> > as stated in the original patch: it's an obsolete macro, and there's a
> > not-obsolete replacement.  seems like we should always merge this sort
> > of thing ?
> >
> >> Also please fix it on the aclocal.m4 as well.
> >
> > aclocal only uses AS_HELP_STRING that i can see.  where are you seeing
> > use of that AC variant ?
> >
> 
> It was a mistake from my part.
> 
> > on a somewhat related note, i can't help but feel like we should
> > switch to using aclocal to manage aclocal.m4 too.  splitting it up
> > into sep m4 files under m4/ should help with the gnarliness of the language ?
> > -mike
> >

[PATCH 3/3] Removed obsolete AC_CHECK_TOOL_PREFIX

[Naohiro Tamura]

This patch removed obsolete AC_CHECK_TOOL_PREFIX.
It has been confirmed that AC_CHECK_TOOL_PREFIX is defined as NOP in
GNU 'autoconf' 2.69 [1], so we can safely remove it.
The 'configure' file has been deleted one blank line by the 'autoconf'
2.69 command .

   [1] autoconf-2.69/share/autoconf/autoconf/programs.m4
   181 # AC_CHECK_TOOL_PREFIX
   182 # --------------------
   183 AU_DEFUN([AC_CHECK_TOOL_PREFIX])
---
 configure    | 1 -
 configure.ac | 1 -
 2 files changed, 2 deletions(-)

diff --git a/configure b/configure
index 1dc3af60b4..37cef37413 100755
--- a/configure
+++ b/configure
@@ -4732,7 +4732,6 @@ fi
 fi
 
 # These programs are version sensitive.
-
 for ac_prog in gnumake gmake make
 do
   # Extract the first word of "$ac_prog", so it can be a program name with args.
diff --git a/configure.ac b/configure.ac
index b7f6f8f008..a20c759241 100644
--- a/configure.ac
+++ b/configure.ac
@@ -993,7 +993,6 @@ else
 fi
 
 # These programs are version sensitive.
-AC_CHECK_TOOL_PREFIX
 AC_CHECK_PROG_VER(MAKE, gnumake gmake make, --version,
   [GNU Make[^0-9]*\([0-9][0-9.]*\)],
   [[4-9].* | [1-9][0-9]*], critic_missing="$critic_missing make")
-- 
2.17.1

Re: [PATCH 3/3] Removed obsolete AC_CHECK_TOOL_PREFIX

[Zack Weinberg]

On Tue, Feb 16, 2021 at 7:02 AM Naohiro Tamura <naohirot@fujitsu.com> wrote:
> This patch removed obsolete AC_CHECK_TOOL_PREFIX.
> It has been confirmed that AC_CHECK_TOOL_PREFIX is defined as NOP in
> GNU 'autoconf' 2.69 [1], so we can safely remove it.
> The 'configure' file has been deleted one blank line by the 'autoconf'
> 2.69 command .

OK, please commit.

zw

RE: [PATCH 3/3] Removed obsolete AC_CHECK_TOOL_PREFIX

[naohirot]

Zack-san,

Thanks!
Naohiro
> -----Original Message-----
> From: Zack Weinberg <zackw@panix.com>
> Sent: Wednesday, February 17, 2021 1:42 AM
> To: Tamura, Naohiro/田村 直広 <naohirot@fujitsu.com>
> Cc: GNU C Library <libc-alpha@sourceware.org>
> Subject: Re: [PATCH 3/3] Removed obsolete AC_CHECK_TOOL_PREFIX
> 
> On Tue, Feb 16, 2021 at 7:02 AM Naohiro Tamura <naohirot@fujitsu.com> wrote:
> > This patch removed obsolete AC_CHECK_TOOL_PREFIX.
> > It has been confirmed that AC_CHECK_TOOL_PREFIX is defined as NOP in
> > GNU 'autoconf' 2.69 [1], so we can safely remove it.
> > The 'configure' file has been deleted one blank line by the 'autoconf'
> > 2.69 command .
> 
> OK, please commit.
> 
> zw

Re: [PATCH 3/3] Removed obsolete AC_CHECK_TOOL_PREFIX

[Mike Frysinger]

On 16 Feb 2021 12:01, Naohiro Tamura wrote:
> This patch removed obsolete AC_CHECK_TOOL_PREFIX.

i've merged this now, thanks!
-mike

Re: [PATCH 3/3] Removed obsolete AC_CHECK_TOOL_PREFIX

[naohirot]

Mike-san,

Thanks for updating the first line and merging it!
Naohiro

________________________________________
From: Mike Frysinger <vapier@gentoo.org>
Sent: Monday, 22 February 2021 08:36
To: Tamura, Naohiro/田村 直広
Cc: libc-alpha@sourceware.org
Subject: Re: [PATCH 3/3] Removed obsolete AC_CHECK_TOOL_PREFIX

On 16 Feb 2021 12:01, Naohiro Tamura wrote:
> This patch removed obsolete AC_CHECK_TOOL_PREFIX.

i've merged this now, thanks!
-mike

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Mike Frysinger]

i've merged this AC_HELP_STRING->AS_HELP_STRING patch now, thanks!
-mike

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[naohirot]

Mike-san,

Thanks for updating the first line and merging it!
Naohiro
________________________________________
From: Mike Frysinger <vapier@gentoo.org>
Sent: Monday, 22 February 2021 06:31
To: Tamura, Naohiro/田村 直広
Cc: libc-alpha@sourceware.org
Subject: Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

i've merged this AC_HELP_STRING->AS_HELP_STRING patch now, thanks!
-mike

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Carlos O'Donell]

On 2/21/21 4:31 PM, Mike Frysinger via Libc-alpha wrote:
> i've merged this AC_HELP_STRING->AS_HELP_STRING patch now, thanks!

Mike,

Please hold on merging further changes from Fujitsu while we work
on copyright assignment with the FSF.

-- 
Cheers,
Carlos.

RE: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[naohirot]

Hi Mike, Carlos,

> From: Carlos O'Donell <carlos@redhat.com>
> Sent: Tuesday, February 23, 2021 11:03 AM
 
> On 2/21/21 4:31 PM, Mike Frysinger via Libc-alpha wrote:
> > i've merged this AC_HELP_STRING->AS_HELP_STRING patch now, thanks!
> 
> Mike,
> 
> Please hold on merging further changes from Fujitsu while we work on copyright
> assignment with the FSF.

Fujitsu submitted signed assignment finally.
It took longer than I thought.

So please kindly merge the following patch, I verified that it can be still applied
without rebase.

[PATCH v2 1/3] Replaced obsolete AC_TRY_COMPILE to AC_COMPILE_IFELSE
https://sourceware.org/pipermail/libc-alpha/2021-February/122740.html

Thanks.
Naohiro

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[Mike Frysinger]

On 10 May 2021 00:52, naohirot@fujitsu.com wrote:
> So please kindly merge the following patch, I verified that it can be still applied
> without rebase.
> 
> [PATCH v2 1/3] Replaced obsolete AC_TRY_COMPILE to AC_COMPILE_IFELSE
> https://sourceware.org/pipermail/libc-alpha/2021-February/122740.html

Zack Weinberg left feedback that doesn't seem to have been addressed ?
https://sourceware.org/pipermail/libc-alpha/2021-February/122722.html
-mike

RE: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[naohirot]

Hi Mike, Zack,

> From: Mike Frysinger <vapier@gentoo.org>
> 
> On 10 May 2021 00:52, naohirot@fujitsu.com wrote:
> > So please kindly merge the following patch, I verified that it can be
> > still applied without rebase.
> >
> > [PATCH v2 1/3] Replaced obsolete AC_TRY_COMPILE to
> AC_COMPILE_IFELSE
> > https://sourceware.org/pipermail/libc-alpha/2021-February/122740.html
> 
> Zack Weinberg left feedback that doesn't seem to have been addressed ?
> https://sourceware.org/pipermail/libc-alpha/2021-February/122722.html

Oh, I didn't answered regarding AC_PREPROC_IFELSE.
I'll fix that.

Thanks.
Naohiro

Re: [PATCH] Replaced obsoleted AC_HELP_STRING to AS_HELP_STRING

[naohirot]

Hi Mike, Zack,

> From: Tamura, Naohiro/田村 直広 <naohirot@fujitsu.com>
> 
> Hi Mike, Zack,
> 
> > From: Mike Frysinger <vapier@gentoo.org>
> >
> > On 10 May 2021 00:52, naohirot@fujitsu.com wrote:
> > > So please kindly merge the following patch, I verified that it can be
> > > still applied without rebase.
> > >
> > > [PATCH v2 1/3] Replaced obsolete AC_TRY_COMPILE to
> > AC_COMPILE_IFELSE
> > > https://sourceware.org/pipermail/libc-alpha/2021-February/122740.html
> >
> > Zack Weinberg left feedback that doesn't seem to have been addressed ?
> > https://sourceware.org/pipermail/libc-alpha/2021-February/122722.html
> 
> Oh, I didn't answered regarding AC_PREPROC_IFELSE.
> I'll fix that.

I've submitted the fixed patch.

https://sourceware.org/pipermail/libc-alpha/2021-May/126079.html

Thanks.
Naohiro