Opened 5 years ago
Closed 4 years ago
#60003 closed defect (fixed)
php73-apache2handler: opportunistically uses valgrind
Reported by: | miken32 (Michael Newton) | Owned by: | ryandesign (Ryan Carsten Schmidt) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | ||
Port: | php73-apache2handler |
Description
Build fails with fatal error: 'valgrind/callgrind.h' file not found
. Log file attached. Attempted to install valgrind-devel
to work around it, but apparently valgrind-devel @3.14.0-r2017-11-21 is not presently compatible with macOS Mojave or newer
. Since valgrind is used for debugging only, I wouldn't expect it to be required for a normal build of anything.
Attachments (1)
Change History (5)
Changed 5 years ago by miken32 (Michael Newton)
comment:1 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)
Status: | assigned → accepted |
---|---|
Summary: | Can't built php73-apache2handler → php73-apache2handler: opportunistically uses valgrind |
I also would not expect valgrind to be used; it is not present or used on my system.
Your log says:
:info:configure checking whether to enable valgrind support... yes :info:configure checking for valgrind header... found in /usr/local
Please try removing valgrind and anything else you have in /usr/local, then sudo port clean php73-apache2handler
and try installing the port again.
I will see if there is a configure argument we can supply to tell php not to use valgrind even if it is found.
comment:2 Changed 5 years ago by miken32 (Michael Newton)
Yes, I had built it from source years ago (2012 apparently) and I guess it's been following me ever since (thanks Time Machine.) Removed /usr/local/bin/valgrind
and /usr/local/include/valgrind
and the package built as expected.
Must be new behaviour in PHP 7.3 because I never had problems with 7.2.
comment:3 Changed 5 years ago by miken32 (Michael Newton)
And looks like there is an argument: --with-valgrind=no
https://github.com/php/php-src/blob/PHP-7.3.13/configure.ac#L810
comment:4 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
build log