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
|