[Buildroot] [PATCH v6,1/1] php: add apache support
Fabrice Fontaine
fontaine.fabrice at gmail.com
Sun Dec 11 20:42:07 UTC 2016
Continue work started by Bernd Kuhls in
https://patchwork.ozlabs.org/patch/437544/
Signed-off-by: Bernd Kuhls <bernd.kuhls at t-online.de>
Signed-off-by: Fabrice Fontaine <fabrice.fontaine at orange.com>
---
Changes v5 -> v6 (after review of Thomas Petazzoni):
- Remove BR2_PACKAGE_PHP_HAS_SAPI as this option no longer exists
- Remove hooks on apxs configuration file as this solution was ugly
- Clearly state in comment that apxs is an apache script
- Update configure script to pass down -S PREFIX='$(INSTALL_ROOT)/usr'
to apxs
Changes v4 -> v5 (after review of Arnout Vandecappelle):
- Patch on pthread detection has been merged upstream and is available
in php 7.0.14 so https://patchwork.ozlabs.org/patch/670567/
has been set as "Not Applicable"
- Remove hook added in v4 and let php updates the apache configuration
file
- Patch apxs configuration file to update httpd.conf in the target
directory with the correct relative path. Without these hooks, apxs
will only update correctly the httpd.conf in the staging directory
Changes v3 -> v4 (after review of Yann Morin):
- Do not select apache but depends on it
- Move pthread detection fix in a separate patch
- Remove unneeded configuration options such as --with-config-file-path
- Do not allow php to update apache configuration file by removing -a
argument from apxs call
Changes v2 -> v3 (after review of Thomas Petazzoni):
- Remove unneeded php-04-apache.patch
- Fix pthread detection if Apache MPM is event or worker
- Update pthread detection mechanism (--enable-pthreads does not exist
anymore)
- Remove unneeded --oldincludedir
- Fix php module path
package/php/Config.in | 7 +++++++
package/php/php.mk | 28 ++++++++++++++++++++++++++++
2 files changed, 35 insertions(+)
diff --git a/package/php/Config.in b/package/php/Config.in
index 57dabc4..0fb8006 100644
--- a/package/php/Config.in
+++ b/package/php/Config.in
@@ -1,6 +1,7 @@
config BR2_PACKAGE_PHP
bool "php"
select BR2_PACKAGE_PHP_SAPI_CGI if \
+ !BR2_PACKAGE_PHP_SAPI_APACHE && \
!BR2_PACKAGE_PHP_SAPI_CLI && \
!BR2_PACKAGE_PHP_SAPI_FPM && \
BR2_USE_MMU
@@ -14,6 +15,12 @@ config BR2_PACKAGE_PHP
if BR2_PACKAGE_PHP
+config BR2_PACKAGE_PHP_SAPI_APACHE
+ bool "Apache interface"
+ depends on BR2_PACKAGE_APACHE
+ help
+ Apache module
+
config BR2_PACKAGE_PHP_SAPI_CGI
bool "CGI interface"
# CGI uses fork()
diff --git a/package/php/php.mk b/package/php/php.mk
index 8c1ccef..9c9d3bb 100644
--- a/package/php/php.mk
+++ b/package/php/php.mk
@@ -81,6 +81,34 @@ PHP_CONF_OPTS += $(if $(BR2_PACKAGE_PHP_SAPI_CLI),--enable-cli,--disable-cli)
PHP_CONF_OPTS += $(if $(BR2_PACKAGE_PHP_SAPI_CGI),--enable-cgi,--disable-cgi)
PHP_CONF_OPTS += $(if $(BR2_PACKAGE_PHP_SAPI_FPM),--enable-fpm,--disable-fpm)
+ifeq ($(BR2_PACKAGE_PHP_SAPI_APACHE),y)
+PHP_DEPENDENCIES += apache
+PHP_CONF_OPTS += --with-apxs2=$(STAGING_DIR)/usr/bin/apxs
+
+# Enable thread safety option if Apache MPM is event or worker
+ifeq ($(BR2_PACKAGE_APACHE_MPM_EVENT)$(BR2_PACKAGE_APACHE_MPM_WORKER),y)
+PHP_CONF_OPTS += --enable-maintainer-zts
+endif
+
+# php uses apache's apxs script from staging directory to install libphp
+# dynamic library and update /etc/apache2/httpd.conf in the staging and target
+# directories. Here is the full command line:
+# "apxs -S LIBEXECDIR='$(INSTALL_ROOT)/usr/modules'
+# -S SYSCONFDIR='$(INSTALL_ROOT)/etc/apache2' -i -a -n php7"
+# This does not work for target directory as apxs sets the full path of the
+# library and not the relative one. Indeed, apxs is smart enough to substitute
+# away the prefix specified in $(STAGING_DIR)/usr/build/config_vars.mk so
+# httpd.conf will only be correct in the staging directory.
+# To fix this, add -S PREFIX='$(INSTALL_ROOT)/usr' to apxs call in configure
+define PHP_FIX_APXS_CALL
+ $(SED) "s, APXS_LIBEXECDIR=, APXS_PREFIX='\$$(INSTALL_ROOT)/usr' APXS_LIBEXECDIR=, \
+ ;s,-i -a -n php,-i -a -S PREFIX='\$$APXS_PREFIX' -n php," \
+ $(@D)/configure
+endef
+
+PHP_PRE_CONFIGURE_HOOKS += PHP_FIX_APXS_CALL
+endif
+
### Extensions
PHP_CONF_OPTS += \
$(if $(BR2_PACKAGE_PHP_EXT_SOCKETS),--enable-sockets) \
--
2.5.0
More information about the buildroot
mailing list