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
