rpluem at apache
Nov 2, 2009, 10:11 PM
Post #3 of 4
(120 views)
Permalink
|
|
Re: svn commit: r830765 - /httpd/httpd/trunk/modules/ssl/ssl_toolkit_compat.h
[In reply to]
|
|
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
|
