httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ruediger Pluem <rpl...@apache.org>
Subject Re: SSL toolkit detection in acinclude.m4
Date Thu, 06 Nov 2008 20:42:00 GMT


On 11/06/2008 08:57 PM, Rainer Jung wrote:
> Ruediger Pluem schrieb:
>> On 11/06/2008 05:50 PM, Rainer Jung wrote:
>>> Rainer Jung schrieb:
>>>> Plüm, Rüdiger, VF-Group schrieb:
>>>>> Rainer does the patch fix your issues on Solaris?
>>>> I'll test once more without the save-related parts of the patch and let
>>>> you know.
>>> I now removed the changes for INCLUDES and LDFLAGS and only kept the
>>> additional `$apr_config --libs`. That works fine.
>> Thanks for testing Rainer. Just for clarification: In fact you used the
>> following patch in your successful test.
>>
>> Index: acinclude.m4
>> ===================================================================
>> --- acinclude.m4        (Revision 711951)
>> +++ acinclude.m4        (Arbeitskopie)
>> @@ -447,10 +447,10 @@
>>          pkglookup="`$PKGCONFIG --libs-only-L --libs-only-other openssl`"
>>          APR_ADDTO(LDFLAGS, [$pkglookup])
>>        else
>> -        ap_ssltk_libs="-lssl -lcrypto"
>> +        ap_ssltk_libs="-lssl -lcrypto `$apr_config --libs`"
>>        fi
>>      else
>> -      ap_ssltk_libs="-lssl -lcrypto"
>> +      ap_ssltk_libs="-lssl -lcrypto `$apr_config --libs`"
>>      fi
>>    fi
>>    APR_SETVAR(SSL_LIBS, [$ap_ssltk_libs])
>>
>> Correct?
> 
> Exactly, should have included it in my response directly. Sorry.

No problem. I updated PR46018 (https://issues.apache.org/bugzilla/show_bug.cgi?id=46018)
and will wait some time for feedback. After that I will commit.

Regards

Rüdiger


Mime
View raw message