With the apr trunk gen-build.py this should support building DBD drivers
as DSOs if configured with --enable-dbd-dso; basic testing only,
comments welcome!
Index: Makefile.in
===================================================================
--- Makefile.in (revision 501084)
+++ Makefile.in (working copy)
@@ -22,8 +22,19 @@
INSTALL = @INSTALL@
INSTALL_DATA = @INSTALL_DATA@
-TARGETS = $(TARGET_LIB) aprutil.exp apu-config.out
+APU_MODULES = @APU_MODULES@
+LINK_MODULE = $(LIBTOOL) $(LTFLAGS) --mode=link $(CC) $(LT_LDFLAGS)
$(ALL_CFLAGS) $(ALL_LDFLAGS) -release $(APRUTIL_MAJOR_VERSION) -module -rpath
$(APU_DSO_LIBDIR)
+APU_DSO_LIBDIR = @APU_DSO_LIBDIR@
+EXTRA_OBJECTS = @EXTRA_OBJECTS@
+
+LDADD_dbd_pgsql = @LDADD_dbd_pgsql@
+LDADD_dbd_oracle = @LDADD_dbd_oracle@
+LDADD_dbd_sqlite2 = @LDADD_dbd_sqlite2@
+LDADD_dbd_sqlite3 = @LDADD_dbd_sqlite3@
+
+TARGETS = $(TARGET_LIB) aprutil.exp apu-config.out $(APU_MODULES)
+
# bring in rules.mk for standard functionality
@INCLUDE_RULES@
@INCLUDE_OUTPUTS@
@@ -53,7 +64,7 @@
apu-config.out: $(APU_CONFIG)
sed 's,^\(location=\).*$$,\1installed,' < $(APU_CONFIG) > $@
-install: $(TARGETS)
+install: $(TARGETS) install-modules
$(APR_MKDIR) $(DESTDIR)$(includedir) $(DESTDIR)$(libdir)/pkgconfig \
$(DESTDIR)$(libdir) $(DESTDIR)$(bindir)
for f in $(top_srcdir)/include/*.h $(top_blddir)/include/*.h; do \
@@ -67,9 +78,17 @@
$(INSTALL_DATA) aprutil.exp $(DESTDIR)$(libdir)
$(INSTALL) -m 755 apu-config.out $(DESTDIR)$(bindir)/$(APU_CONFIG)
-$(TARGET_LIB): $(OBJECTS)
- $(LINK) @lib_target@ $(ALL_LIBS) $(APRUTIL_LDFLAGS) $(APRUTIL_LIBS)
+$(TARGET_LIB): $(OBJECTS) $(EXTRA_OBJECTS)
+ $(LINK) @lib_target@ $(EXTRA_OBJECTS) $(ALL_LIBS) $(APRUTIL_LDFLAGS)
$(APRUTIL_LIBS)
+install-modules: [EMAIL PROTECTED]@
+
+install-modules-no:
+
+install-modules-yes: $(APU_MODULES)
+ $(APR_MKDIR) $(DESTDIR)$(APU_DSO_LIBDIR)
+ @for m in $(APU_MODULES); do $(LIBTOOL) $(LTFLAGS) --mode=install
$(INSTALL) -m 755 $$m $(DESTDIR)$(APU_DSO_LIBDIR); done
+
exports.c: $(HEADERS)
$(APR_MKEXPORT) $(HEADERS) > $@
Index: build.conf
===================================================================
--- build.conf (revision 501084)
+++ build.conf (working copy)
@@ -19,7 +19,7 @@
xml/*.c
strmatch/*.c
xlate/*.c
- dbd/*.c
+ dbd/apr_dbd.c
ssl/*.c
# we have no platform-specific subdirs
@@ -28,7 +28,26 @@
# the public headers
headers = include/*.h include/private/*.h
+modules = dbd_pgsql dbd_sqlite2 dbd_sqlite3 dbd_oracle
+
# gen_uri_delim.c
# we have a recursive makefile for the test files (for now)
# test/*.c
+
+[dbd_pgsql]
+paths = dbd/apr_dbd_pgsql.c
+target = dbd/apr_dbd_pgsql.la
+
+[dbd_sqlite2]
+paths = dbd/apr_dbd_sqlite2.c
+target = dbd/apr_dbd_sqlite2.la
+
+[dbd_sqlite3]
+paths = dbd/apr_dbd_sqlite3.c
+target = dbd/apr_dbd_sqlite3.la
+
+[dbd_oracle]
+paths = dbd/apr_dbd_oracle.c
+target = dbd/apr_dbd_oracle.la
+
Index: configure.in
===================================================================
--- configure.in (revision 501084)
+++ configure.in (working copy)
@@ -141,6 +141,8 @@
APU_CHECK_DBD_SQLITE3
APU_CHECK_DBD_SQLITE2
APU_CHECK_DBD_ORACLE
+dnl Enable DSO build; must be last:
+APU_CHECK_DBD_DSO
APU_FIND_EXPAT
APU_FIND_ICONV
@@ -169,6 +171,24 @@
APRUTIL_LIBNAME="aprutil${libsuffix}"
AC_SUBST(APRUTIL_LIBNAME)
+# Set up destination directory for DSOs.
+APU_DSO_LIBDIR="\${libdir}/apr-util-${APRUTIL_MAJOR_VERSION}"
+# Set APU_HAVE_MODULES appropriate for the Makefile
+if test -n "$APU_MODULES"; then
+ APU_HAVE_MODULES=yes
+else
+ APU_HAVE_MODULES=no
+fi
+# Define expanded libdir for apu_config.h
+APR_EXPAND_VAR(abs_dso_libdir, $APU_DSO_LIBDIR)
+AC_DEFINE_UNQUOTED([APU_DSO_LIBDIR], ["$abs_dso_libdir"],
+ [Define to be absolute path to DSO directory])
+AC_SUBST(APU_HAVE_MODULES)
+AC_SUBST(APU_DSO_LIBDIR)
+AC_SUBST(APU_MODULES)
+
+AC_SUBST(EXTRA_OBJECTS)
+
dnl
dnl Prep all the flags and stuff for compilation and export to other builds
dnl
Index: dbd/apr_dbd.c
===================================================================
--- dbd/apr_dbd.c (revision 501084)
+++ dbd/apr_dbd.c (working copy)
@@ -16,23 +16,23 @@
#include <stdio.h>
+#include "apu_config.h"
#include "apu.h"
+
#include "apr_pools.h"
-#include "apr_dbd_internal.h"
-#include "apr_dbd.h"
+#include "apr_dso.h"
+#include "apr_strings.h"
#include "apr_hash.h"
#include "apr_thread_mutex.h"
-#include "apr_dso.h"
-#include "apr_strings.h"
+#include "apr_dbd_internal.h"
+#include "apr_dbd.h"
+#include "apu_version.h"
+
static apr_hash_t *drivers = NULL;
#define CLEANUP_CAST (apr_status_t (*)(void*))
-/* Once the autofoo supports building it for dynamic load, we can use
- * #define APR_DSO_BUILD APR_HAS_DSO
- */
-
#if APR_HAS_THREADS
static apr_thread_mutex_t* mutex = NULL;
apr_status_t apr_dbd_mutex_lock()
@@ -52,7 +52,7 @@
}
#endif
-#ifndef APR_DSO_BUILD
+#ifndef APU_DSO_BUILD
#define DRIVER_LOAD(name,driver,pool) \
{ \
extern const apr_dbd_driver_t driver; \
@@ -91,8 +91,8 @@
/* This already registers a pool cleanup */
#endif
-#ifndef APR_DSO_BUILD
-
+#ifndef APU_DSO_BUILD
+ /* Load statically-linked drivers: */
#if APU_HAVE_MYSQL
DRIVER_LOAD("mysql", apr_dbd_mysql_driver, pool);
#endif
@@ -111,15 +111,17 @@
#if APU_HAVE_SOME_OTHER_BACKEND
DRIVER_LOAD("firebird", apr_dbd_other_driver, pool);
#endif
-#endif /* APR_DSO_BUILD */
+#endif /* APU_DSO_BUILD */
+
return ret;
}
APU_DECLARE(apr_status_t) apr_dbd_get_driver(apr_pool_t *pool, const char
*name,
const apr_dbd_driver_t **driver)
{
-#if APR_DSO_BUILD
+#if APR_HAS_DSO
char path[80];
apr_dso_handle_t *dlhandle = NULL;
+ apr_dso_handle_sym_t symbol;
#endif
apr_status_t rv;
@@ -128,12 +130,12 @@
return APR_SUCCESS;
}
-#if APR_DSO_BUILD
+#if APR_HAS_DSO
#if APR_HAS_THREADS
rv = apr_thread_mutex_lock(mutex);
if (rv != APR_SUCCESS) {
- goto unlock;
+ return rv;
}
*driver = apr_hash_get(drivers, name, APR_HASH_KEY_STRING);
if (*driver) {
@@ -144,18 +146,19 @@
#ifdef WIN32
sprintf(path, "apr_dbd_%s.dll", name);
#else
- sprintf(path, "apr_dbd_%s.so", name);
+ apr_snprintf(path, sizeof path, APU_DSO_LIBDIR "/apr_dbd_%s.so", name);
#endif
rv = apr_dso_load(&dlhandle, path, pool);
if (rv != APR_SUCCESS) { /* APR_EDSOOPEN */
goto unlock;
}
sprintf(path, "apr_dbd_%s_driver", name);
- rv = apr_dso_sym((void*)driver, dlhandle, path);
+ rv = apr_dso_sym(&symbol, dlhandle, path);
if (rv != APR_SUCCESS) { /* APR_ESYMNOTFOUND */
apr_dso_unload(dlhandle);
goto unlock;
}
+ *driver = symbol;
if ((*driver)->init) {
(*driver)->init(pool);
}
@@ -166,7 +169,7 @@
apr_thread_mutex_unlock(mutex);
#endif
-#else /* APR_DSO_BUILD - so if it wasn't already loaded, it's NOTIMPL */
+#else /* not builtin and !APR_HAS_DSO => not implemented */
rv = APR_ENOTIMPL;
#endif
Index: build/dbd.m4
===================================================================
--- build/dbd.m4 (revision 501084)
+++ build/dbd.m4 (working copy)
@@ -69,9 +69,9 @@
dnl Since we have already done the AC_CHECK_LIB tests, if we have it,
dnl we know the library is there.
if test "$apu_have_pgsql" = "1"; then
- APR_ADDTO(APRUTIL_EXPORT_LIBS,[-lpq])
- APR_ADDTO(APRUTIL_LIBS,[-lpq])
+ LDADD_dbd_pgsql=-lpq
fi
+ AC_SUBST(LDADD_dbd_pgsql)
])
dnl
AC_DEFUN([APU_CHECK_DBD_MYSQL], [
@@ -221,9 +221,9 @@
dnl Since we have already done the AC_CHECK_LIB tests, if we have it,
dnl we know the library is there.
if test "$apu_have_sqlite3" = "1"; then
- APR_ADDTO(APRUTIL_EXPORT_LIBS,[-lsqlite3])
- APR_ADDTO(APRUTIL_LIBS,[-lsqlite3])
+ LDADD_dbd_sqlite3="-lsqlite3"
fi
+ AC_SUBST(LDADD_dbd_sqlite3)
])
dnl
AC_DEFUN([APU_CHECK_DBD_SQLITE2], [
@@ -266,9 +266,9 @@
dnl Since we have already done the AC_CHECK_LIB tests, if we have it,
dnl we know the library is there.
if test "$apu_have_sqlite2" = "1"; then
- APR_ADDTO(APRUTIL_EXPORT_LIBS,[-lsqlite])
- APR_ADDTO(APRUTIL_LIBS,[-lsqlite])
+ LDADD_dbd_sqlite2="-lsqlite"
fi
+ AC_SUBST(LDADD_dbd_sqlite2)
])
dnl
AC_DEFUN([APU_CHECK_DBD_ORACLE], [
@@ -294,8 +294,7 @@
unset ac_cv_lib_clntsh_OCIEnvCreate
AC_CHECK_LIB(clntsh, OCIEnvCreate, [
apu_have_oracle=1
- APR_ADDTO(APRUTIL_EXPORT_LIBS,[-lnnz10])
- APR_ADDTO(APRUTIL_LIBS,[-lnnz10])
+ LDADD_dbd_oracle="-lnnz10"
],,[-lnnz10])
]))
@@ -322,8 +321,7 @@
unset ac_cv_lib_clntsh_OCIEnvCreate
AC_CHECK_LIB(clntsh, OCIEnvCreate, [
apu_have_oracle=1
- APR_ADDTO(APRUTIL_EXPORT_LIBS,[-lnnz10])
- APR_ADDTO(APRUTIL_LIBS,[-lnnz10])
+ LDADD_dbd_oracle="-lnnz10"
],,[-lnnz10])
]))
if test "$apu_have_oracle" != "0"; then
@@ -354,8 +352,7 @@
unset ac_cv_lib_clntsh_OCIEnvCreate
AC_CHECK_LIB(clntsh, OCIEnvCreate, [
apu_have_oracle=1
- APR_ADDTO(APRUTIL_EXPORT_LIBS,[-lnnz10])
- APR_ADDTO(APRUTIL_LIBS,[-lnnz10])
+ LDADD_dbd_oracle=-lnnz10
],,[-lnnz10])
]))
@@ -367,8 +364,40 @@
dnl Since we have already done the AC_CHECK_LIB tests, if we have it,
dnl we know the library is there.
if test "$apu_have_oracle" = "1"; then
- APR_ADDTO(APRUTIL_EXPORT_LIBS,[-lclntsh])
- APR_ADDTO(APRUTIL_LIBS,[-lclntsh])
+ LDADD_dbd_oracle="$LDADD_dbd_oracle -lclntsh"
fi
+ AC_SUBST(LDADD_dbd_oracle)
])
-dnl
+
+AC_DEFUN([APU_CHECK_DBD_DSO], [
+
+ AC_ARG_ENABLE([dbd-dso],
+ APR_HELP_STRING([--enable-dbd-dso], [build DBD drivers as DSOs]))
+
+ if test $enable_dbd_dso = yes; then
+ AC_DEFINE([APU_DSO_BUILD], 1, [Define if DBD drivers are built as DSOs])
+ apu_dbd_dso=1
+
+ dsos=
+ test $apu_have_oracle = 1 && dsos="$dsos dbd/apr_dbd_oracle.la"
+ test $apu_have_pgsql = 1 && dsos="$dsos dbd/apr_dbd_pgsql.la"
+ test $apu_have_mysql = 1 && dsos="$dsos dbd/apr_dbd_mysql.la"
+ test $apu_have_sqlite2 = 1 && dsos="$dsos dbd/apr_dbd_sqlite2.la"
+ test $apu_have_sqlite3 = 1 && dsos="$dsos dbd/apr_dbd_sqlite3.la"
+
+ APU_MODULES="$APU_MODULES $dsos"
+ else
+ # Statically link the DBD drivers:
+
+ objs=
+ test $apu_have_oracle = 1 && objs="$objs dbd/apr_dbd_oracle.lo"
+ test $apu_have_pgsql = 1 && objs="$objs dbd/apr_dbd_pgsql.lo"
+ test $apu_have_mysql = 1 && objs="$objs dbd/apr_dbd_mysql.lo"
+ test $apu_have_sqlite2 = 1 && objs="$objs dbd/apr_dbd_sqlite2.lo"
+ test $apu_have_sqlite3 = 1 && objs="$objs dbd/apr_dbd_sqlite3.lo"
+ EXTRA_OBJECTS="$EXTRA_OBJECTS $objs"
+
+ APRUTIL_LIBS="$APRUTIL_LIBS '$(LDADD_dbd_pgsql) $(LDADD_dbd_sqlite2)
$(LDADD_dbd_sqlite3) $(LDADD_dbd_oracle)'"
+ APRUTIL_EXPORT_LIBS="$APRUTIL_EXPORT_LIBS '$(LDADD_dbd_pgsql)
$(LDADD_dbd_sqlite2) $(LDADD_dbd_sqlite3) $(LDADD_dbd_oracle)'"
+ fi
+])
