Return-Path: Delivered-To: apmail-httpd-dev-archive@www.apache.org Received: (qmail 75082 invoked from network); 3 Jan 2009 23:02:08 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 3 Jan 2009 23:02:08 -0000 Received: (qmail 36542 invoked by uid 500); 3 Jan 2009 23:02:08 -0000 Delivered-To: apmail-httpd-dev-archive@httpd.apache.org Received: (qmail 35608 invoked by uid 500); 3 Jan 2009 23:02:06 -0000 Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm Precedence: bulk Reply-To: dev@httpd.apache.org list-help: list-unsubscribe: List-Post: List-Id: Delivered-To: mailing list dev@httpd.apache.org Received: (qmail 35599 invoked by uid 99); 3 Jan 2009 23:02:06 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 03 Jan 2009 15:02:06 -0800 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received: from [140.211.11.9] (HELO minotaur.apache.org) (140.211.11.9) by apache.org (qpsmtpd/0.29) with SMTP; Sat, 03 Jan 2009 23:02:04 +0000 Received: (qmail 74959 invoked by uid 2161); 3 Jan 2009 23:01:42 -0000 Received: from [192.168.2.4] (euler.heimnetz.de [192.168.2.4]) by cerberus.heimnetz.de (Postfix on SuSE Linux 7.0 (i386)) with ESMTP id 61F3B1721C for ; Sun, 4 Jan 2009 00:01:30 +0100 (CET) Message-ID: <495FEE4B.9050406@apache.org> Date: Sun, 04 Jan 2009 00:01:31 +0100 From: Ruediger Pluem User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.19) Gecko/20081204 SeaMonkey/1.1.14 MIME-Version: 1.0 To: dev@httpd.apache.org Subject: Re: svn commit: r731009 - /httpd/httpd/trunk/modules/session/config.m4 References: <20090103161255.801AE238899B@eris.apache.org> In-Reply-To: <20090103161255.801AE238899B@eris.apache.org> X-Enigmail-Version: 0.95.7 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Virus-Checked: Checked by ClamAV on apache.org On 01/03/2009 05:12 PM, rjung@apache.org wrote: > Author: rjung > Date: Sat Jan 3 08:12:55 2009 > New Revision: 731009 > > URL: http://svn.apache.org/viewvc?rev=731009&view=rev > Log: > Allow feature test for apr_ssl.h to succeed: > Add necessary search paths temporarily to CPPFLAGS. > > Addition to r730717. > > When switching to another header, no need to again > rename the ap_HAVE_APR_CRYPTO variable. > > Modified: > httpd/httpd/trunk/modules/session/config.m4 > > Modified: httpd/httpd/trunk/modules/session/config.m4 > URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/session/config.m4?rev=731009&r1=731008&r2=731009&view=diff > ============================================================================== > --- httpd/httpd/trunk/modules/session/config.m4 (original) > +++ httpd/httpd/trunk/modules/session/config.m4 Sat Jan 3 08:12:55 2009 > @@ -18,8 +18,11 @@ > APACHE_MODULE(session, session module, , , most) > APACHE_MODULE(session_cookie, session cookie module, , , $session_mods_enable) > APACHE_MODULE(session_crypto, session crypto module, , , no, [ > - AC_CHECK_HEADERS(apr_ssl.h, [ap_HAVE_APR_SSL_H="yes"], [ap_HAVE_APR_SSL_H="no"]) > - if test $ap_HAVE_APR_SSL_H = "no"; then > + saved_CPPFLAGS="$CPPFLAGS" Does it make sense to save these flags if you do not restore them later on? > + CPPFLAGS="$CPPFLAGS -I$APR_INCLUDEDIR -I$APU_INCLUDEDIR" > + AC_CHECK_HEADERS(apr_crypto.h, [ap_HAVE_APR_CRYPTO="yes"], [ap_HAVE_APR_CRYPTO="no"]) > + CPPFLAGS="$saved_CPPFLAGS" > + if test $ap_HAVE_APR_CRYPTO = "no"; then > AC_MSG_WARN([Your APR does not include SSL/EVP support.]) > enable_session_crypto="no" > fi Regards Rüdiger