Bugfix: configure: Quote SUPPRESS_WARNINGS sufficiently to preserve brackets

The regex includes [/ ] which is supposed to match either a forward slash or a
space, but m4 treats the brackets as special characters and effectively strips
them out, leading to -isystem /usr/include paths except for in the typical
scenario where it is the final parameter in the flag string.
24.x
Luke Dashjr 3 years ago
parent 831675c8dc
commit 556ee6f2fa

@ -1175,7 +1175,7 @@ dnl Do not change "-I/usr/include" to "-isystem /usr/include" because that
dnl is not necessary (/usr/include is already a system directory) and because dnl is not necessary (/usr/include is already a system directory) and because
dnl it would break GCC's #include_next. dnl it would break GCC's #include_next.
AC_DEFUN([SUPPRESS_WARNINGS], AC_DEFUN([SUPPRESS_WARNINGS],
[$(echo $1 |${SED} -E -e 's/(^| )-I/\1-isystem /g' -e 's;-isystem /usr/include([/ ]|$);-I/usr/include\1;g')]) [[$(echo $1 |${SED} -E -e 's/(^| )-I/\1-isystem /g' -e 's;-isystem /usr/include([/ ]|$);-I/usr/include\1;g')]])
dnl enable-fuzz should disable all other targets dnl enable-fuzz should disable all other targets
if test "x$enable_fuzz" = "xyes"; then if test "x$enable_fuzz" = "xyes"; then

Loading…
Cancel
Save