php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #75722 Convert valgrind detection to configure option
Submitted: 2017-12-22 22:38 UTC Modified: -
From: mhei at heimpold dot de Assigned:
Status: Closed Package: *Compile Issues
PHP Version: 7.2.0 OS: Linux / OpenWrt
Private report: No CVE-ID: None
 [2017-12-22 22:38 UTC] 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.



Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2018-03-23 21:04 UTC] nikic@php.net
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
 [2018-03-23 21:04 UTC] nikic@php.net
-Status: Open +Status: Closed
 
PHP Copyright © 2001-2018 The PHP Group
All rights reserved.
Last updated: Sun Nov 19 01:31:42 2017 UTC