apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Guenter Knauf <fua...@apache.org>
Subject Re: svn commit: r553027 - in /apr/apr-util/trunk: dbd/apr_dbd.c include/apu.h.in include/apu.hnw include/apu.hw
Date Thu, 05 Jul 2007 14:44:56 GMT
Hi Martin,
>> sure it can also go into apu_config.h - but this needs then some more
>> changes:
>> - apr_dbd.h needs to include apu_config.h

> .... which is explicitly "private/apu_config.h" and should
>      therefore IMO not be used from outside.

>> - create a netware-own apu_config.hnw (not a problem)
>> - tell apr-util/configure.in to add this define (I would need some help
>> from you for that)

> ... the latter is probably the best approach.
can you please confirm if this will work for Unix/Linux:

--- configure.in.orig	Wed Jun 27 14:53:14 2007
+++ configure.in	Thu Jul 05 15:42:26 2007
@@ -173,6 +173,8 @@
 
 # Set up destination directory for DSOs.
 APU_DSO_LIBDIR="\${libdir}/apr-util-${APRUTIL_MAJOR_VERSION}"
+# Set up DBD driver format to load and print errors for driver DSOs.
+APU_DBD_DRIVER_FMT="\${libdir}/apr-util-${APRUTIL_MAJOR_VERSION}/apr_dbd_%s.so"
 # Set APU_HAVE_MODULES appropriately for the Makefile
 if test -n "$APU_MODULES"; then 
    APU_HAVE_MODULES=yes

then I would include apu_config.h from apr_dbd.h and it should work again:

--- apr_dbd.h.orig	Wed Jun 27 14:53:00 2007
+++ apr_dbd.h	Thu Jul 05 15:37:10 2007
@@ -22,6 +22,7 @@
 #define APR_DBD_H
 
 #include "apu.h"
+#include "apu_config.h"
 #include "apr_pools.h"
 
 #ifdef __cplusplus

dont know though if then an additional include path to include/private somewhere is needed....

thanks, Guenter.



Mime
View raw message