You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by "Plüm, Rüdiger, VF-Group" <ru...@vodafone.com> on 2008/11/04 13:23:38 UTC
Re: SSL toolkit detection in acinclude.m4
> -----Ursprüngliche Nachricht-----
> Von: Ruediger Pluem
> Gesendet: Samstag, 25. Oktober 2008 15:18
> An: dev@httpd.apache.org
> Betreff: Re: SSL toolkit detection in acinclude.m4
>
>
>
> On 10/16/2008 12:04 PM, Rainer Jung wrote:
> > Ruediger Pluem wrote:
> >> While investigating PR46018 I came across the following in
> acinclude.m4
> >>
> >> dnl Run header and version checks
> >> saved_CPPFLAGS="$CPPFLAGS"
> >> saved_LIBS="$LIBS"
> >> if test "x$ap_ssltk_base" != "x"; then
> >> APR_ADDTO(CPPFLAGS, [-I$ap_ssltk_base/include])
> >> APR_ADDTO(INCLUDES, [-I$ap_ssltk_base/include])
> >> APR_ADDTO(LDFLAGS, [-L$ap_ssltk_base/lib])
> >>
> >>
> >> Is there any reason why we only save / restore CPPFLAGS
> and LIBS and
> >> not INCLUDES and LDFLAGS which are also modified?
> >>
> >> The next thing I don't get is why we do not add
> `$apr_config --libs`
> >> to LIBS
> >> any longer (since r669924). This causes -ldl to miss and thus the
> >> checks to fail
> >> (I guess only if there is no pkgconfig file).
> >
> > Yes, I ran into a similar issue, when compiling 2.2.10
> against static
> > openssl libs on Solaris, having no pkgconfig. Then you need
> to add -ldl,
> > -lnsl, -lsocket during the configure tests, because
> otherwise tehy fail.
> > That was not necessary until 2.2.9.
>
> Any further comments, ideas how to fix this?
I added a possible fix to the PR (https://issues.apache.org/bugzilla/attachment.cgi?id=22826).
Does somebody mind checking if this is the correct thing to do?
Regards
Rüdiger
Re: SSL toolkit detection in acinclude.m4
Posted by Ruediger Pluem <rp...@apache.org>.
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
Re: SSL toolkit detection in acinclude.m4
Posted by Rainer Jung <ra...@kippdata.de>.
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.
Re: SSL toolkit detection in acinclude.m4
Posted by Ruediger Pluem <rp...@apache.org>.
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?
Regards
Rüdiger
Re: SSL toolkit detection in acinclude.m4
Posted by Rainer Jung <ra...@kippdata.de>.
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.
Regards,
Rainer
Re: SSL toolkit detection in acinclude.m4
Posted by Rainer Jung <ra...@kippdata.de>.
Plüm, Rüdiger, VF-Group schrieb:
>>> On 10/16/2008 12:04 PM, Rainer Jung wrote:
>>>> Ruediger Pluem wrote:
>>>>> While investigating PR46018 I came across the following in
>>> acinclude.m4
>>>>> dnl Run header and version checks
>>>>> saved_CPPFLAGS="$CPPFLAGS"
>>>>> saved_LIBS="$LIBS"
>>>>> if test "x$ap_ssltk_base" != "x"; then
>>>>> APR_ADDTO(CPPFLAGS, [-I$ap_ssltk_base/include])
>>>>> APR_ADDTO(INCLUDES, [-I$ap_ssltk_base/include])
>>>>> APR_ADDTO(LDFLAGS, [-L$ap_ssltk_base/lib])
>>>>>
>>>>>
>>>>> Is there any reason why we only save / restore CPPFLAGS
>>> and LIBS and
>>>>> not INCLUDES and LDFLAGS which are also modified?
>>>>>
>>>>> The next thing I don't get is why we do not add
>>> `$apr_config --libs`
>>>>> to LIBS
>>>>> any longer (since r669924). This causes -ldl to miss and thus the
>>>>> checks to fail
>>>>> (I guess only if there is no pkgconfig file).
>>>> Yes, I ran into a similar issue, when compiling 2.2.10
>>> against static
>>>> openssl libs on Solaris, having no pkgconfig. Then you need
>>> to add -ldl,
>>>> -lnsl, -lsocket during the configure tests, because
>>> otherwise tehy fail.
>>>> That was not necessary until 2.2.9.
>>> Any further comments, ideas how to fix this?
>> I added a possible fix to the PR
>> (https://issues.apache.org/bugzilla/attachment.cgi?id=22826).
>> Does somebody mind checking if this is the correct thing to do?
>
> Rainer does the patch fix your issues on Solaris?
The additional `$apr_config --libs` seems fine to fix the issue during
configure itself.
Saving INCLUDES and LDFLAGS break the build. Without saving, entries are
added to EXTRA_INCLUDES and EXTRA_LDFLAGS in build/config_vars.mk. Those
entries are needed to build ab and mod_ssl and to link them.
For the linking issue one could instead of adding the SSL-specifix -L...
to EXTRA_LDFLAGS add them to ab_LDADD and to MOD_SSL_LDADD: it seems not
to be needed for anything else. Since both of those vars include
SSL_LIBS, it could also go in there.
The include flag is harder, since at the moment the build system does ot
include a component specific include variable (like ab_INCADD or
MOD_SSL_INCADD).
So now we know, why INCLUDES and LDFLAGS where not saved in configure ;)
I'll test once more without the save-related parts of the patch and let
you know.
Regards,
Rainer
Re: SSL toolkit detection in acinclude.m4
Posted by Rainer Jung <ra...@kippdata.de>.
Plüm, Rüdiger, VF-Group schrieb:
>
>
>> -----Ursprüngliche Nachricht-----
>> Von: "Plüm, Rüdiger, VF-Group"
>> Gesendet: Dienstag, 4. November 2008 13:24
>> An: dev@httpd.apache.org
>> Betreff: Re: SSL toolkit detection in acinclude.m4
>>
>>
>>
>>> -----Ursprüngliche Nachricht-----
>>> Von: Ruediger Pluem
>>> Gesendet: Samstag, 25. Oktober 2008 15:18
>>> An: dev@httpd.apache.org
>>> Betreff: Re: SSL toolkit detection in acinclude.m4
>>>
>>>
>>>
>>> On 10/16/2008 12:04 PM, Rainer Jung wrote:
>>>> Ruediger Pluem wrote:
>>>>> While investigating PR46018 I came across the following in
>>> acinclude.m4
>>>>> dnl Run header and version checks
>>>>> saved_CPPFLAGS="$CPPFLAGS"
>>>>> saved_LIBS="$LIBS"
>>>>> if test "x$ap_ssltk_base" != "x"; then
>>>>> APR_ADDTO(CPPFLAGS, [-I$ap_ssltk_base/include])
>>>>> APR_ADDTO(INCLUDES, [-I$ap_ssltk_base/include])
>>>>> APR_ADDTO(LDFLAGS, [-L$ap_ssltk_base/lib])
>>>>>
>>>>>
>>>>> Is there any reason why we only save / restore CPPFLAGS
>>> and LIBS and
>>>>> not INCLUDES and LDFLAGS which are also modified?
>>>>>
>>>>> The next thing I don't get is why we do not add
>>> `$apr_config --libs`
>>>>> to LIBS
>>>>> any longer (since r669924). This causes -ldl to miss and thus the
>>>>> checks to fail
>>>>> (I guess only if there is no pkgconfig file).
>>>> Yes, I ran into a similar issue, when compiling 2.2.10
>>> against static
>>>> openssl libs on Solaris, having no pkgconfig. Then you need
>>> to add -ldl,
>>>> -lnsl, -lsocket during the configure tests, because
>>> otherwise tehy fail.
>>>> That was not necessary until 2.2.9.
>>> Any further comments, ideas how to fix this?
>> I added a possible fix to the PR
>> (https://issues.apache.org/bugzilla/attachment.cgi?id=22826).
>> Does somebody mind checking if this is the correct thing to do?
>
> Rainer does the patch fix your issues on Solaris?
I'm in New Orleans. Will be able to test remotely but only later. Will
report back in 24 hours.
Regards,
Rainer
Re: SSL toolkit detection in acinclude.m4
Posted by "Plüm, Rüdiger, VF-Group" <ru...@vodafone.com>.
> -----Ursprüngliche Nachricht-----
> Von: "Plüm, Rüdiger, VF-Group"
> Gesendet: Dienstag, 4. November 2008 13:24
> An: dev@httpd.apache.org
> Betreff: Re: SSL toolkit detection in acinclude.m4
>
>
>
> > -----Ursprüngliche Nachricht-----
> > Von: Ruediger Pluem
> > Gesendet: Samstag, 25. Oktober 2008 15:18
> > An: dev@httpd.apache.org
> > Betreff: Re: SSL toolkit detection in acinclude.m4
> >
> >
> >
> > On 10/16/2008 12:04 PM, Rainer Jung wrote:
> > > Ruediger Pluem wrote:
> > >> While investigating PR46018 I came across the following in
> > acinclude.m4
> > >>
> > >> dnl Run header and version checks
> > >> saved_CPPFLAGS="$CPPFLAGS"
> > >> saved_LIBS="$LIBS"
> > >> if test "x$ap_ssltk_base" != "x"; then
> > >> APR_ADDTO(CPPFLAGS, [-I$ap_ssltk_base/include])
> > >> APR_ADDTO(INCLUDES, [-I$ap_ssltk_base/include])
> > >> APR_ADDTO(LDFLAGS, [-L$ap_ssltk_base/lib])
> > >>
> > >>
> > >> Is there any reason why we only save / restore CPPFLAGS
> > and LIBS and
> > >> not INCLUDES and LDFLAGS which are also modified?
> > >>
> > >> The next thing I don't get is why we do not add
> > `$apr_config --libs`
> > >> to LIBS
> > >> any longer (since r669924). This causes -ldl to miss and thus the
> > >> checks to fail
> > >> (I guess only if there is no pkgconfig file).
> > >
> > > Yes, I ran into a similar issue, when compiling 2.2.10
> > against static
> > > openssl libs on Solaris, having no pkgconfig. Then you need
> > to add -ldl,
> > > -lnsl, -lsocket during the configure tests, because
> > otherwise tehy fail.
> > > That was not necessary until 2.2.9.
> >
> > Any further comments, ideas how to fix this?
>
> I added a possible fix to the PR
> (https://issues.apache.org/bugzilla/attachment.cgi?id=22826).
> Does somebody mind checking if this is the correct thing to do?
Rainer does the patch fix your issues on Solaris?
Regards
Rüdiger