summaryrefslogtreecommitdiff
path: root/php
diff options
context:
space:
mode:
authorRuss Allbery <rra@stanford.edu>2008-09-12 15:36:53 -0700
committerRuss Allbery <rra@stanford.edu>2008-09-12 15:36:53 -0700
commit3786840efc18999b07ab7d021441c19846e89211 (patch)
tree61e0c9632217a9c228c7282f95f130925c9d3168 /php
parentbaeec5b5bba7aab9aba5e05a3c3d5fcbf5848ed1 (diff)
Switch to configuring PHP at build time
Handling the PHP directory as an Autoconf subdirectory just doesn't work with srcdir != builddir builds. The ordering is all wrong. Switch to running configure at build time, which also has the nice advantage that we can generate config.m4 with Autoconf and substitute in the correct link paths.
Diffstat (limited to 'php')
-rw-r--r--php/config.m4.in (renamed from php/config.m4)12
1 files changed, 5 insertions, 7 deletions
diff --git a/php/config.m4 b/php/config.m4.in
index 1a50fa4..c26d82b 100644
--- a/php/config.m4
+++ b/php/config.m4.in
@@ -1,4 +1,4 @@
-dnl remctl PECL extension for PHP configuration
+dnl remctl PECL extension for PHP configuration -*- autoconf -*-
dnl
dnl Provides additional configuration hooks for the PHP module build system.
dnl This file is used by the phpize frameowrk.
@@ -12,11 +12,9 @@ dnl See LICENSE for licensing terms.
PHP_ARG_ENABLE([remctl], [whether to enable remctl PHP extension],
[AC_HELP_STRING([--enable-remctl], [Enable recmtl PHP extension])], [yes])
-dnl The escaping on PHP_ADD_INCLUDE and PHP_ADD_LIBPATH seems to work, and
-dnl seems to be the only thing that works, but it makes me nervous. Needs
-dnl more testing and maybe a better approach that will still work for builddir
-dnl != srcdir builds.
+dnl The @...@ variables are substituted by the top-level configure. Yes, this
+dnl is a little odd, but it seems to work.
AS_IF([test "$PHP_REMCTL" != no],
[PHP_NEW_EXTENSION([remctl], [php_remctl.c], [$ext_shared])
- PHP_ADD_INCLUDE([\${abs_top_srcdir}/..])
- LDFLAGS="$LDFLAGS -L\${top_builddir}/../client/.libs -lremctl"])
+ PHP_ADD_INCLUDE([@abs_top_srcdir@])
+ LDFLAGS="$LDFLAGS -L@abs_top_builddir@/client/.libs -lremctl"])