Conversation
config.m4
Outdated
| APCU_CFLAGS="-D_GNU_SOURCE -DZEND_ENABLE_STATIC_TSRMLS_CACHE=1" | ||
| PHP_ADD_LIBRARY(pthread) | ||
| PHP_ADD_LIBRARY(pthread,, APCU_SHARED_LIBADD) | ||
| PHP_LDFLAGS="$PHP_LDFLAGS -lpthread" |
There was a problem hiding this comment.
Hm, do we need the PHP_LDFLAGS adjustment? As I understand it, if apcu is not build shared PHP_ADD_LIBRARY should already be adding it to LIBS.
There was a problem hiding this comment.
I'm assuming not, and can't think of any reason to have this, but not 100% sure and haven't checked back to 7.0. It gets added to the Makefile but nothing in the Makefile uses PHP_LDFLAGS.
Even when compiling apcu statically this still wouldn't affect other modules
There was a problem hiding this comment.
In that case I think we should drop these lines and see if anyone complains...
TysonAndre
force-pushed
the
shared-libadd
branch
from
6ef5f79 to
fdfe78c
Compare
Noticed in php 8.3-dev configured with --disable-all The "-pthread" flag wasn't actually getting used in the makefile.
TysonAndre
force-pushed
the
shared-libadd
branch
from
fdfe78c to
b27e1ce
Compare
|
php/php-src@ea372e7 is also relevant - neither the old nor new version work in php < 7.4.14. A possible alternative is to manually concatenate the string instead |
2 participants
Noticed in php 8.3-dev configured with --disable-all
The "-pthread" flag wasn't actually getting used in the makefile.