httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From r..@covalent.net
Subject Re: [PATCH] configuring APR threads fix
Date Tue, 31 Oct 2000 16:57:00 GMT
On Mon, 30 Oct 2000, Joe Orton wrote:

> The APR configure script seems to be caching whether --enable-threads
> was passed, which is confusing. This fixes that and simplifies a bit,
> replacing the cache variable and enable_apr_threads with a single
> enable_threads variable:

I realize that this is confusing, but it is also correct.  The problem is
that we have to be able to decide based on which MPM was chosen how to
configure APR.  For example, if Apache uses the prefork MPM, then we don't
want threads, but if Apache uses mpmt_pthread or dexter or perchild, then
we do want threads.  I have not been able to find a way to force APR to be
configured differently based on how Apache is configured unless Apache
sets a condition in the cache.

This could be solved much cleaner if Autoconf would allow us to specify
flags to sub-project configure scripts, but I haven't seen a way to do
that yet.

That is why we have to check the cache.  I applied this patch, and it does
detect threads properly, but I ALWAYS configure with prefork, because it
is easier to debug the protocol code with that MPM, and APR all of a
sudden was being configured with threads, which breaks all sorts of
compile-time things.

Ryan

_______________________________________________________________________________
Ryan Bloom                        	rbb@apache.org
406 29th St.
San Francisco, CA 94131
-------------------------------------------------------------------------------


Mime
View raw message