Here is the new patch.

Cheers,
Henry


On 4/9/06, William A. Rowe, Jr. <[EMAIL PROTECTED]> wrote:
Henry Jen wrote:
>
> will not have the -R flag in the Makefile.

Your patch is invalid because we need to proxy the whole accumulated
LDFLAGS off to the client who's trying to compile against apr[-util].
Fixing Makefile isn't enough, LDFLAGS must be 'sticky' within the
accumulated apr-1-config/apu-1-config syntax.

Please revisit your patch and determine where your desired flags are
being dumped by autoconf, and I'd be happy to commit a patch that
ensures those user-given flags percolate into our APRUTIL_LDFLAGS
throughout the configuration, as opposed to committing a bandaid.

Yours,

Bill

Index: configure.in
===================================================================
--- configure.in	(revision 398058)
+++ configure.in	(working copy)
@@ -97,6 +97,10 @@
 APR_ADDTO(CFLAGS, `$apr_config --cflags`)
 APR_ADDTO(CPPFLAGS, `$apr_config --cppflags`)
 
+dnl carry user defined LDFLAGS
+APR_ADDTO(APRUTIL_LDFLAGS, "$LDFLAGS")
+APR_ADDTO(APRUTIL_LIBS, "$LIBS")
+
 dnl
 dnl  Find the APR-ICONV directory.
 dnl

Reply via email to