Reads fine.

I'd like to test this patch (perhaps later this week?).

Would 
    make davautocheck ENVVAR=short_circuit
be a better interface?

Noorul Islam K M wrote on Tue, Jun 28, 2011 at 12:03:36 +0530:
> Index: Makefile.in
> ===================================================================
> --- Makefile.in       (revision 1140431)
> +++ Makefile.in       (working copy)
> @@ -501,7 +501,8 @@
>  # Automatically configure and run Apache httpd on a random port, and then
>  # run make check.
>  davautocheck: bin $(TEST_DEPS) @BDB_TEST_DEPS@ apache-mod
> -     @# Takes MODULE_PATH and USE_HTTPV1 in the environment.
> +     @# Takes MODULE_PATH, USE_HTTPV1 and USE_AUTHZ_SHORT_CIRCUIT in the 
> +     @# environment.
>       @APXS=$(APXS) bash 
> $(top_srcdir)/subversion/tests/cmdline/davautocheck.sh
>  
>  # First, run:
> Index: subversion/tests/cmdline/davautocheck.sh
> ===================================================================
> --- subversion/tests/cmdline/davautocheck.sh  (revision 1140431)
> +++ subversion/tests/cmdline/davautocheck.sh  (working copy)
> @@ -67,6 +67,9 @@
>  #
>  # To prevent the server from advertising httpv2, pass USE_HTTPV1 in
>  # the environment.
> +# 
> +# To use "short_circuit" value for "SvnPathAuthz" directive set
> +# USE_AUTHZ_SHORT_CIRCUIT=yes in the environment.
>  #
>  # Passing --no-tests as argv[1] will have the script start a server
>  # but not run any tests.
> @@ -160,6 +163,12 @@
>   ADVERTISE_V2_PROTOCOL=off
>  fi
>  
> +# Pick up $USE_AUTHZ_SHORT_CIRCUIT
> +SVN_PATH_AUTHZ=""
> +if [ ${USE_AUTHZ_SHORT_CIRCUIT:+set} ]; then
> + SVN_PATH_AUTHZ="SVNPathAuthz      short_circuit"
> +fi

Reply via email to