[prev in list] [next in list] [prev in thread] [next in thread] 

List:       apr-dev
Subject:    Re: Makefile.in does not take external LIBS LDFLAGS
From:       Henry Jen <henryjen () ztune ! net>
Date:       2006-04-30 23:03:11
Message-ID: 4455422F.4050607 () ztune ! net
[Download RAW message or body]

Ryan Bloom wrote:
> Didn't we solve this problem by adding EXTRA_LIBS and EXTRA_LDFLAGS? I 
> haven't looked at the code, but I remember (from years past), that
> we didn't want people to set CFLAGS, LIBS, LDFLAGS, etc for some
> reason.  Instead, we asked people to set EXTRA_CFLAGS, etc and we
> picked those up.
> 

I am relative new to apr, so not sure about history and no one pointed 
me to EXTRA_xxx. :-)

Include CFLAGS may not be appropriate, but I think LDFLAGS/LIBS should 
be OK although I don't feel very comfortable with it. Would some one 
please enlighten me a bit on this issue?

Have a quick look at those EXTRA flags, don't know how to use those. Ideas?

Cheers,
Henry


> I could be mis-remembering though.
> 
> Ryan
> 
> On 4/28/06, Henry Jen <henryjen@ztune.net> wrote:
>> Here is the new patch.
>>
>> Cheers,
>> Henry
>>
>>
>>
>> On 4/9/06, William A. Rowe, Jr. <wrowe@rowe-clan.net> 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
>> >
>>
>>
>>
> 
> 
> -- 
> Ryan Bloom
> rbb@apache.org
> rbb@rkbloom.net
> rbloom@gmail.com
> 

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic