php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

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
View Developer Edit
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: mhei at heimpold dot de
New email:
PHP Version: OS:

 

 [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

Pull Requests

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-2024 The PHP Group
All rights reserved. 		Last updated: Thu Nov 21 13:01:29 2024 UTC