Req #75722 [Opn->Csd]: Convert valgrind detection to configure option

Edit report at https://bugs.php.net/bug.php?id=75722&edit=1 ID: 75722 Updated by: nikic@php.net Reported by: mhei at heimpold dot de Summary: Convert valgrind detection to configure option -Status: Open +Status: Closed Type: Feature/Change Request Package: *Compile Issues Operating System: Linux / OpenWrt PHP Version: 7.2.0 Block user comment: N Private report: N New Comment: Automatic comment on behalf of mhei@heimpold.de Revision: http://git.php.net/?p=php-src.git;a=commit;h=25ba60b1479eb332a7388732f3ee5467000fbe73 Log: Fixed bug #75722: Rework valgrind detection Previous Comments: ------------------------------------------------------------------------ [2017-12-22 22:38:49] mhei at heimpold dot de Description: ------------ The configure script (acinclude.m4) currently searches for valgrind header file and enables valgrin support if found. When cross-compiling the searched paths are invalid for the target platform because they below to the host system. There seems to be no way to give the build system a dedicated path where to look. This leads to the issue, that when cross-compiling eg. for ARMv5 platform, that valgrind header file is detected - because host system is amd64 - and support is enabled but target platform will never support valgrind (valgrind requires e.g. at least ARMv7). So the proposal is to fix/rework the detection, or better add a configuration option for this, e.g. --without-valgrind-support or --with-valgrind-support=<dir> as used for other libraries should be enough. ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=75722&edit=1