From 7d3d8de1f3f9d3da5c8a80db2c18493dd1be048e Mon Sep 17 00:00:00 2001 From: Peter Kokot Date: Sun, 21 Apr 2024 17:23:22 +0200 Subject: [PATCH] Fix erroneous dnl appended in configure (#14013) This is a backport of commit 03f15534a17c7031b89dac7aaa21d59474517321 to PHP-8.2 due to GH-14002 and fixes the PHP_CXX_COMPILE_STDCXX check in ext/intl whether the specified C++ standard is mandatory or optional. The `dnl` (Discard to Next Line) M4 macro in this combination of `m4_if` macros and arguments isn't properly replaced and a literal `dnl` string is appended in the configure script. The `[]dnl` works ok. --- NEWS | 1 + build/php_cxx_compile_stdcxx.m4 | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 9a6bf178e8d..33ce2e0bd37 100644 --- a/NEWS +++ b/NEWS @@ -11,6 +11,7 @@ PHP NEWS other timeout implementations). (Kévin Dunglas) . Fixed bug GH-14003 (Broken cleanup of unfinished calls with callable convert parameters). (ilutov) + . Fixed bug GH-14013 (Erroneous dnl appended in configure). (Peter Kokot) - Fibers: . Fixed bug GH-13903 (ASAN false positive underflow when executing copy()). diff --git a/build/php_cxx_compile_stdcxx.m4 b/build/php_cxx_compile_stdcxx.m4 index f8e97fce708..8f723266b2c 100644 --- a/build/php_cxx_compile_stdcxx.m4 +++ b/build/php_cxx_compile_stdcxx.m4 @@ -27,11 +27,11 @@ AC_DEFUN([PHP_CXX_COMPILE_STDCXX], [dnl [$1], [14], [ax_cxx_compile_alternatives="14 1y"], [$1], [17], [ax_cxx_compile_alternatives="17 1z"], [$1], [20], [ax_cxx_compile_alternatives="20"], - [m4_fatal([invalid first argument `$1' to PHP_CXX_COMPILE_STDCXX])])dnl + [m4_fatal([invalid first argument `$1' to PHP_CXX_COMPILE_STDCXX])])[]dnl m4_if([$2], [], [ax_cxx_compile_cxx$1_required=true], [$2], [mandatory], [ax_cxx_compile_cxx$1_required=true], [$2], [optional], [ax_cxx_compile_cxx$1_required=false], - [m4_fatal([invalid third argument `$2' to PHP_CXX_COMPILE_STDCXX])])dnl + [m4_fatal([invalid third argument `$2' to PHP_CXX_COMPILE_STDCXX])])[]dnl AC_LANG_PUSH([C++])dnl ac_success=no