Mailing-List: contact new-httpd-help@apache.org; run by ezmlm
Precedence: bulk
Reply-To: new-httpd@apache.org
Date: Mon, 21 May 2001 12:47:43 -0700
From: Greg Stein <gstein@lyra.org>
To: new-httpd@apache.org, Brian Havard <brianh@kheldar.apana.org.au>
Subject: Re: DAV build
Message-ID: <20010521124742.M6482@lyra.org>
Mail-Followup-To: new-httpd@apache.org,
	Brian Havard <brianh@kheldar.apana.org.au>
References: <200105200802.f4K82LC05084@silk.apana.org.au>
 <200105201629.f4KGTsC19833@silk.apana.org.au>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.2i
In-Reply-To: <200105201629.f4KGTsC19833@silk.apana.org.au>;
 from brianh@kheldar.apana.org.au on Mon, May 21, 2001 at 02:29:46AM +1000

On Mon, May 21, 2001 at 02:29:46AM +1000, Brian Havard wrote:
>...
> Nope, that doesn't do it. It doesn't work right if using --enable-dav 
> explicitly. Getting the right behaviour in all situations I tested required 
> the patch below. Is there any good reason why we can't combine these into one
> module?

They don't combine -- they are two separate modules. One provides DAV
functionality, the other provides the backend repository. At the moment,
there are at least four backends in development that I'm aware of.

>...
> --- config6.m4	2001/05/18 20:27:45	1.3
> +++ config6.m4	2001/05/20 16:27:38
> @@ -7,7 +7,15 @@
>  if test "$enable_dav" = "no"; then
>    dav_fs_enable=no
>  else
> -  dav_fs_enable="$enable_dav"
> +  dav_fs_enable=most
> +
> +  if test -z "$enable_dav_fs"; then
> +    if test -z "$enable_dav"; then
> +      enable_dav_fs="shared"
> +    else
> +      enable_dav_fs="$enable_dav"
> +    fi
> +  fi
>  fi
>  
>  APACHE_MODULE(dav_fs, DAV provider for the filesystem, $dav_fs_objects, , $dav_fs_enable)

Hmm. In that inner block, the hard-coded "shared" doesn't feel right. The
default for shared/non-shared is defined by $module_default. Also, the
variable should probably be dav_fs_enable for passing to the APACHE_MODULE
macro, right?

Looks like I'll have to get in there and start trying the combos...

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/