[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: CVS update: subversion/ac-helpers svn-apache.m4

From: Greg Stein <gstein_at_lyra.org>
Date: 2000-10-06 09:37:52 CEST

On Fri, Oct 06, 2000 at 05:50:20AM -0000, daniel@tigris.org wrote:
> User: daniel
> Date: 00/10/05 22:50:20
>
> Modified: ac-helpers svn-apache.m4
> Log:
> * svn-apache.m4: after the apache check has found an apache 2 directory, it
> now checks for the presense of apr.h to make sure that the apache installation
> has been configured. If apr.h isn't found, a warning message is produced. This
> may instead switch off the "apache present" variables.

It probably shouldn't switch it off. The user can just go to Apache and
configure the sucker to fix the thing. We wouldn't want to make the user
then go and reconfig subversion just for that.

You may want to consider just using "test -r ..." instead of
AC_CHECK_HEADER(). The former will be faster and will avoid the monkeying
with flags. If we want to cache the result, then we can do that separately.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
Received on Sat Oct 21 14:36:10 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.