httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ruediger Pluem <rpl...@apache.org>
Subject Re: svn commit: r830765 - /httpd/httpd/trunk/modules/ssl/ssl_toolkit_compat.h
Date Tue, 03 Nov 2009 06:11:07 GMT


On 11/03/2009 12:28 AM, Guenter Knauf wrote:
> Hi Ruediger,
> Ruediger Pluem schrieb:
>> I now get the following warnings on Linux:
>>
>> In file included from ssl_private.h:57,
>>                  from mod_ssl.c:27:
>> ssl_toolkit_compat.h:43:1: warning: "HAVE_OCSP" redefined
>> In file included from /usr/src/apache/httpd-trunk/include/ap_config.h:234,
>>                  from /usr/src/apache/httpd-trunk/include/httpd.h:43,
>>                  from ssl_private.h:30,
>>                  from mod_ssl.c:27:
>> /usr/src/apache/httpd-trunk/include/ap_config_auto.h:102:1: warning: this is the
location of the previous definition
>>
>> So if the above is only needed for Netware and Windows we should make this either
conditional
>> on those two OS'es or we should add an
>>
>> #ifndef HAVE_OCSP
>>
>> #endif
>>
>> block around this.
> we can just remove the define from the config.m4 - though I was unsure
> if removing the lines:
> 
> AC_DEFUN([CHECK_OCSP], [
> AC_CHECK_HEADERS(openssl/ocsp.h,·
>   [AC_DEFINE([HAVE_OCSP], 1, [Define if OCSP is supported by OpenSSL])]
> )
> ])
> 
> would be sufficient?

Additionally we would need to remove at least the call of CHECK_OCSP. But
I am not sure if this is sufficient either.
Plus I don't know if we need the outcome of this test in other parts of
configure.


Regards

Rüdiger

Mime
View raw message