ap-php changes break php74

[Jonathan Perkin <jperkin%mnx.io@localhost>]

There was some work recently on ap-php.  I don't recall what the 
original reasons for the change were, but I remember having to fix 
issues with the configure script that didn't happen previously.

It turns out there's a bigger problem.  A user reported seeing startup 
failure when trying to use php74 modules that previously worked fine:

  PHP Warning:  PHP Startup: Unable to load dynamic library 'zip.so' (tried: /opt/local/lib/php/20191128/zip.so (ld.so.1: httpd: fatal: relocation error: file /opt/local/lib/php/20191128/zip.so: symbol executor_globals: referenced symbol not found), /opt/local/lib/php/20191128/zip.so.so (ld.so.1: httpd: fatal: /opt/local/lib/php/20191128/zip.so.so: open failed: No such file or directory)) in Unknown on line 0

This turns out to be changing how the thread-safety ZTS option is 
handled.  In earlier versions of the same build:

  configure:83063: checking whether to enable thread-safety
  configure:83065: result: no

and now in pkgsrc-trunk:

  configure:88161: checking whether to enable thread-safety
  configure:88163: result: yes

This is what causes the symbols to change, and there may be other 
lurking issues (there are other differences in php_config.h).

I'll see if I can manually disable the ZTS option, but I'd appreciate 
if folks who understand PHP can look into this more deeply and check 
that nothing else has been broken, and whether it's possible to revert 
this fully back to the previously behaviour.

Thanks,

--
Jonathan Perkin   -   mnx.io   -   pkgsrc.smartos.org
Open Source Complete Cloud   www.tritondatacenter.com