From 0954557e1f8354c1beb30fe86992c24355c7d951 Mon Sep 17 00:00:00 2001 From: Peter Kokot Date: Thu, 29 Aug 2024 20:15:26 +0200 Subject: [PATCH] Autotools: Replace backticks command substitutions with $(...) This is a follow-up of previous backticks replacement commits. This replaces backticks command substitutions in remaining ext config M4 files, phpize, and configure.ac with the recommended $(...). Note that $(...) still does not work on some obsolete shells that ship with Solaris 10, for example. Elsewhere they should work. However, for these obsolete shells Autoconf also re-executes the shell script under the supported shell so it can make them work regardless. Additionally, few comments CS are also adjusted to not cause confusion when searching for backticks usages and one indentation sync done. As of Autoconf 2.72 the backticks in macro help texts are also replaced with single quotes. --- build/config-stubs | 8 +++--- configure.ac | 65 ++++++++++++++++++++++------------------------ ext/ldap/config.m4 | 2 +- scripts/phpize.m4 | 14 +++++----- 4 files changed, 43 insertions(+), 46 deletions(-) diff --git a/build/config-stubs b/build/config-stubs index 0dc77879ccff7..6d8d3c1ea1e7e 100755 --- a/build/config-stubs +++ b/build/config-stubs @@ -2,9 +2,9 @@ dir=$1; shift for stubfile in $dir/*/config0.m4 $dir/*/config.m4 $dir/*/config9.m4; do - echo "dnl Define where extension directories are located in the configure context -AC_DEFUN([PHP_EXT_BUILDDIR], [`dirname $stubfile`]) -AC_DEFUN([PHP_EXT_DIR], [`dirname $stubfile`]) -AC_DEFUN([PHP_EXT_SRCDIR], [\$abs_srcdir/`dirname $stubfile`]) + echo "dnl Define where extension directories are located in the configure context +AC_DEFUN([PHP_EXT_BUILDDIR], [$(dirname $stubfile)]) +AC_DEFUN([PHP_EXT_DIR], [$(dirname $stubfile)]) +AC_DEFUN([PHP_EXT_SRCDIR], [\$abs_srcdir/$(dirname $stubfile)]) sinclude($stubfile)" done diff --git a/configure.ac b/configure.ac index d7afb3544e1b2..893aa23638f16 100644 --- a/configure.ac +++ b/configure.ac @@ -68,7 +68,7 @@ dnl Allow overriding PHP_EXTRA_VERSION through the homonymous env var AC_ARG_VAR([PHP_EXTRA_VERSION],[Extra PHP version label suffix, e.g. '-dev', 'rc1', '-acme'])dnl AS_IF([test -z "$PHP_EXTRA_VERSION"],[PHP_EXTRA_VERSION=[$]4]) PHP_VERSION="$PHP_MAJOR_VERSION.$PHP_MINOR_VERSION.$PHP_RELEASE_VERSION$PHP_EXTRA_VERSION" -PHP_VERSION_ID=`expr [$]PHP_MAJOR_VERSION \* 10000 + [$]PHP_MINOR_VERSION \* 100 + [$]PHP_RELEASE_VERSION` +PHP_VERSION_ID=$(expr [$]PHP_MAJOR_VERSION \* 10000 + [$]PHP_MINOR_VERSION \* 100 + [$]PHP_RELEASE_VERSION) dnl Allow version values to be used in Makefile. PHP_SUBST([PHP_MAJOR_VERSION]) @@ -100,7 +100,7 @@ dnl ---------------------------------------------------------------------------- PHP_INIT_BUILD_SYSTEM -dnl Because `make install` is often performed by the superuser, we create the +dnl Because 'make install' is often performed by the superuser, we create the dnl libs subdirectory as the user who configures PHP. Otherwise, the current dnl user will not be able to delete libs or the contents of libs. @@ -499,7 +499,7 @@ dnl Check for structure members. AC_CHECK_MEMBERS([struct tm.tm_gmtoff],,,[#include ]) AC_CHECK_MEMBERS([struct stat.st_blksize, struct stat.st_rdev]) dnl AC_STRUCT_ST_BLOCKS will screw QNX because fileblocks.o does not exist. -if test "`uname -s 2>/dev/null`" != "QNX"; then +if test "$(uname -s 2>/dev/null)" != "QNX"; then AC_STRUCT_ST_BLOCKS fi @@ -516,7 +516,7 @@ dnl dnl At least some versions of FreeBSD seem to have buggy ifunc support, see dnl bug #77284. Conservatively don't use ifuncs on FreeBSD prior to version 12. AS_CASE([$host_alias], [*-*-*android*|*-*-*uclibc*|*-*-*musl*|*openbsd*], [true], [ - if test "`uname -s 2>/dev/null`" != "FreeBSD" || test "`uname -U 2>/dev/null`" -ge 1200000; then + if test "$(uname -s 2>/dev/null)" != "FreeBSD" || test "$(uname -U 2>/dev/null)" -ge 1200000; then AX_GCC_FUNC_ATTRIBUTE([ifunc]) AX_GCC_FUNC_ATTRIBUTE([target]) fi @@ -769,10 +769,9 @@ if test "$PHP_GCOV" = "yes"; then [AC_MSG_ERROR([GNU C compatible compiler is required for --enable-gcov])]) dnl Check if ccache is being used. - case `$php_shtool path $CC` in - *ccache*[)] gcc_ccache=yes;; - *[)] gcc_ccache=no;; - esac + AS_CASE([$($php_shtool path $CC)], + [*ccache*], [gcc_ccache=yes], + [gcc_ccache=no]) if test "$gcc_ccache" = "yes" && (test -z "$CCACHE_DISABLE" || test "$CCACHE_DISABLE" != "1"); then AC_MSG_ERROR(m4_text_wrap([ @@ -787,8 +786,8 @@ if test "$PHP_GCOV" = "yes"; then dnl Remove all optimization flags from CFLAGS. changequote({,}) dnl Discard known '-O...' flags, including just '-O', but do not remove only '-O' in '-Ounknown' - CFLAGS=`echo "$CFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g'` - CXXFLAGS=`echo "$CXXFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g'` + CFLAGS=$(echo "$CFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g') + CXXFLAGS=$(echo "$CXXFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g') changequote([,]) dnl Add the special gcc flags. @@ -808,8 +807,8 @@ AS_VAR_IF([PHP_DEBUG], [yes], [ ZEND_DEBUG=yes changequote({,}) dnl Discard known '-O...' flags, including just '-O', but do not remove only '-O' in '-Ounknown' - CFLAGS=`echo "$CFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g'` - CXXFLAGS=`echo "$CXXFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g'` + CFLAGS=$(echo "$CFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g') + CXXFLAGS=$(echo "$CXXFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g') changequote([,]) dnl Add -O0 only if GCC or ICC is used. if test "$GCC" = "yes" || test "$ICC" = "yes"; then @@ -1323,23 +1322,21 @@ case $libdir in libdir=$libdir/php ;; esac -case `eval echo $datadir` in - '${prefix}/share') - datadir=$datadir/php - ;; -esac -phptempdir=`pwd`/libs +AS_CASE([$(eval echo $datadir)], + ['${prefix}/share'], [datadir=$datadir/php]) + +phptempdir=$(pwd)/libs old_exec_prefix=$exec_prefix old_libdir=$libdir old_datadir=$datadir -exec_prefix=`eval echo $exec_prefix` -libdir=`eval echo $libdir` -datadir=`eval eval echo $datadir` +exec_prefix=$(eval echo $exec_prefix) +libdir=$(eval echo $libdir) +datadir=$(eval eval echo $datadir) dnl Build extension directory path. -ZEND_MODULE_API_NO=`$EGREP '#define ZEND_MODULE_API_NO ' $srcdir/Zend/zend_modules.h|"${SED}" 's/#define ZEND_MODULE_API_NO //'` +ZEND_MODULE_API_NO=$($EGREP '#define ZEND_MODULE_API_NO ' $srcdir/Zend/zend_modules.h|"${SED}" 's/#define ZEND_MODULE_API_NO //') AS_VAR_IF([EXTENSION_DIR],, [ extbasedir=$ZEND_MODULE_API_NO @@ -1360,17 +1357,17 @@ AS_CASE([$PHP_LAYOUT], [datarootdir=$prefix/php]) dnl Expand all directory names for use in macros/constants. -EXPANDED_PEAR_INSTALLDIR=`eval echo $PEAR_INSTALLDIR` -EXPANDED_EXTENSION_DIR=`eval echo $EXTENSION_DIR` -EXPANDED_LOCALSTATEDIR=`eval echo $localstatedir` -EXPANDED_BINDIR=`eval echo $bindir` -EXPANDED_SBINDIR=`eval echo $sbindir` -EXPANDED_MANDIR=`eval echo $mandir` +EXPANDED_PEAR_INSTALLDIR=$(eval echo $PEAR_INSTALLDIR) +EXPANDED_EXTENSION_DIR=$(eval echo $EXTENSION_DIR) +EXPANDED_LOCALSTATEDIR=$(eval echo $localstatedir) +EXPANDED_BINDIR=$(eval echo $bindir) +EXPANDED_SBINDIR=$(eval echo $sbindir) +EXPANDED_MANDIR=$(eval echo $mandir) EXPANDED_LIBDIR=$libdir -EXPANDED_SYSCONFDIR=`eval echo $sysconfdir` +EXPANDED_SYSCONFDIR=$(eval echo $sysconfdir) EXPANDED_DATADIR=$datadir -EXPANDED_PHP_CONFIG_FILE_PATH=`eval echo "$PHP_CONFIG_FILE_PATH"` -EXPANDED_PHP_CONFIG_FILE_SCAN_DIR=`eval echo "$PHP_CONFIG_FILE_SCAN_DIR"` +EXPANDED_PHP_CONFIG_FILE_PATH=$(eval echo "$PHP_CONFIG_FILE_PATH") +EXPANDED_PHP_CONFIG_FILE_SCAN_DIR=$(eval echo "$PHP_CONFIG_FILE_SCAN_DIR") INCLUDE_PATH=.:$EXPANDED_PEAR_INSTALLDIR exec_prefix=$old_exec_prefix @@ -1404,10 +1401,10 @@ PHP_REMOVE_USR_LIB([LDFLAGS]) EXTRA_LDFLAGS="$EXTRA_LDFLAGS $PHP_LDFLAGS" EXTRA_LDFLAGS_PROGRAM="$EXTRA_LDFLAGS_PROGRAM $PHP_LDFLAGS" -UNAME=`uname -a | xargs` +UNAME=$(uname -a | xargs) PHP_UNAME=${PHP_UNAME:-$UNAME} AC_DEFINE_UNQUOTED([PHP_UNAME], ["$PHP_UNAME"], [The 'uname -a' output.]) -PHP_OS=`uname | xargs` +PHP_OS=$(uname | xargs) AC_DEFINE_UNQUOTED([PHP_OS], ["$PHP_OS"], [The 'uname' output.]) PHP_BUILD_SYSTEM=${PHP_BUILD_SYSTEM:-$PHP_UNAME} AC_DEFINE_UNQUOTED([PHP_BUILD_SYSTEM], ["$PHP_BUILD_SYSTEM"], @@ -1562,7 +1559,7 @@ dnl PHP_HELP_SEPARATOR([Libtool:]) PHP_CONFIGURE_PART([Configuring libtool]) -dnl Silence warning: `ar: 'u' modifier ignored since 'D' is the default` +dnl Silence warning: "ar: 'u' modifier ignored since 'D' is the default". dnl See https://github.com/php/php-src/pull/3017 AC_SUBST([AR_FLAGS], [cr]) diff --git a/ext/ldap/config.m4 b/ext/ldap/config.m4 index 447e9dda2d454..bb57d636dde54 100644 --- a/ext/ldap/config.m4 +++ b/ext/ldap/config.m4 @@ -24,7 +24,7 @@ AC_DEFUN([PHP_LDAP_CHECKS], [ PHP_OCI8_IC_LIBDIR_SUFFIX=64 ]) - OCISDKRPMINC=`echo "$1" | $SED -e 's!^/usr/lib/oracle/\(.*\)/client\('${PHP_OCI8_IC_LIBDIR_SUFFIX}'\)*/lib[/]*$!/usr/include/oracle/\1/client\2!'` + OCISDKRPMINC=$(echo "$1" | $SED -e 's!^/usr/lib/oracle/\(.*\)/client\('${PHP_OCI8_IC_LIBDIR_SUFFIX}'\)*/lib[/]*$!/usr/include/oracle/\1/client\2!') dnl Check for Oracle Instant Client RPM install if test -f $OCISDKRPMINC/ldap.h; then diff --git a/scripts/phpize.m4 b/scripts/phpize.m4 index 09adfa787b54c..e59a1c03a5789 100644 --- a/scripts/phpize.m4 +++ b/scripts/phpize.m4 @@ -53,11 +53,11 @@ PHP_ARG_WITH([php-config],, dnl For BC. PHP_CONFIG=$PHP_PHP_CONFIG -prefix=`$PHP_CONFIG --prefix 2>/dev/null` -phpincludedir=`$PHP_CONFIG --include-dir 2>/dev/null` -INCLUDES=`$PHP_CONFIG --includes 2>/dev/null` -EXTENSION_DIR=`$PHP_CONFIG --extension-dir 2>/dev/null` -PHP_EXECUTABLE=`$PHP_CONFIG --php-binary 2>/dev/null` +prefix=$($PHP_CONFIG --prefix 2>/dev/null) +phpincludedir=$($PHP_CONFIG --include-dir 2>/dev/null) +INCLUDES=$($PHP_CONFIG --includes 2>/dev/null) +EXTENSION_DIR=$($PHP_CONFIG --extension-dir 2>/dev/null) +PHP_EXECUTABLE=$($PHP_CONFIG --php-binary 2>/dev/null) AS_VAR_IF([prefix],, [AC_MSG_ERROR([Cannot find php-config. Please use --with-php-config=PATH])]) @@ -106,8 +106,8 @@ AS_VAR_IF([PHP_DEBUG], [yes], [ ZEND_DEBUG=yes changequote({,}) dnl Discard known '-O...' flags, including just '-O', but do not remove only '-O' in '-Ounknown' - CFLAGS=`echo "$CFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g'` - CXXFLAGS=`echo "$CXXFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g'` + CFLAGS=$(echo "$CFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g') + CXXFLAGS=$(echo "$CXXFLAGS" | $SED -e 's/-O\([0-9gsz]\|fast\|\)\([\t ]\|$\)//g') changequote([,]) dnl Add -O0 only if GCC or ICC is used. if test "$GCC" = "yes" || test "$ICC" = "yes"; then