apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Cliff Woolley <jwool...@virginia.edu>
Subject Re: [PATCH] bucket debugging.
Date Wed, 27 Aug 2003 21:56:13 GMT
On Wed, 13 Aug 2003, Jeff Trawick wrote:

> Cliff Woolley wrote:
> > Any objections to the following patch?  All you do is pass
> > -DAPR_BUCKET_DEBUG to ./configure and then several things happen:
>
> is --enable-bucket-debug preferable?  a couple of positive aspects of that:
>
> a) natural place for help text (apr-util/configure --help)
> b) after introducing configure logic, it becomes natural for the
> compile-time setting to go in a header file instead of making the
> compiler command lines even longer

So it turns out this is kind of a pain because CPPFLAGS comes directly
from APR's rules.mk, which is copied in from APR as-is.  apr-util has
APRUTIL_INCLUDES, APRUTIL_LDFLAGS, etc, but no APRUTIL_CPPFLAGS.  And I
don't see a particularly clean way to add an APRUTIL_CPPFLAGS.  I could
just stick the -DAPR_BUCKET_DEBUG in APRUTIL_INCLUDES, which works, but is
a horrible hack.  Any ideas?

--Cliff

Mime
View raw message