apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Tom Donovan <donov...@bellatlantic.net>
Subject Re: [PATCH] DBD drivers as DSOs
Date Wed, 28 Feb 2007 02:12:16 GMT
I'm puzzled by the implicit rule in apr_dbd.c that if one or more static dbd drivers are configured,

dso drivers aren't allowed.  Also by the fact that dso dbd drivers need to be explicitly enabled.

Now that it works to have several different dbd drivers loaded, this seems unnecessarily restrictive.

It makes more sense to me to always allow dso dbd drivers if the platform supports dso and
the 
driver name is not already in the 'drivers' hash.  Static drivers would, of course, get in
there 
first and take precedence.

This patch to 1.3 works for me on Windows.  I haven't tried it on Unix yet.
It dispenses with the whole --enable-dbd-dso/APU_DSO_BUILD thing and just enables dso dbd
drivers if 
dso is enabled - regardless of whether there are any static dbd drivers or not.

I hope I'm missing something here, or else I wish I brought this point up before all the work
went 
into changing the build.  Alas, I just started looking at the dbd thing in the last few days.

-tom-


Mime
View raw message