Return-Path: Delivered-To: apmail-httpd-dev-archive@www.apache.org Received: (qmail 75888 invoked from network); 3 Jan 2009 23:11:11 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 3 Jan 2009 23:11:11 -0000 Received: (qmail 39485 invoked by uid 500); 3 Jan 2009 23:11:00 -0000 Delivered-To: apmail-httpd-dev-archive@httpd.apache.org Received: (qmail 39440 invoked by uid 500); 3 Jan 2009 23:11:00 -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 39352 invoked by uid 99); 3 Jan 2009 23:11:00 -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:11:00 -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:10:56 +0000 Received: (qmail 75755 invoked by uid 2161); 3 Jan 2009 23:10:35 -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 1353B1721C for ; Sun, 4 Jan 2009 00:10:25 +0100 (CET) Message-ID: <495FF062.6060108@apache.org> Date: Sun, 04 Jan 2009 00:10:26 +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> <495FEE4B.9050406@apache.org> In-Reply-To: <495FEE4B.9050406@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/04/2009 12:01 AM, Ruediger Pluem wrote: > > 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" Ahh, sorry. Out of coffee error. Regards Rüdiger