Re: svn commit: r886177 - in /httpd/test/framework/trunk/t/conf: extra.conf.in ssl/ssl.conf.in

[Jeff Trawick <tr...@gmail.com>]

On Wed, Dec 2, 2009 at 10:42 AM,  <tr...@apache.org> wrote:
> Author: trawick
> Date: Wed Dec  2 15:42:54 2009
> New Revision: 886177
>
> URL: http://svn.apache.org/viewvc?rev=886177&view=rev
> Log:
> Mutex directive updates:
>
> mod_ssl's session cache mutex and mod_rewrite's rewrite map
> mutex no longer have to be configured explicitly, so provide
> only a commented-out version with explanation

...

> +    <IfVersion >= 2.3.4>
> +    # mutex created automatically
> +    # config needed only if file-based mutexes are used and
> +    # default lock file dir is inappropriate
> +    # Mutex file:/path/to/lockdir ssl-cache

I misunderstood that there was no way to specify the path to the mutex
files just in case file-based mutexes were used.  But ap_parse_mutex()
has allowed "default:/path/to/lockdir" all along, and the new Mutex
directive code doesn't get in the way; thus, the way to update the
path to the mutex files just in case they are needed is simply "Mutex
default:/path/to/lockdir".

Secondly, the way to control the lock file dir in the test suite would
be via a single Mutex directive to cover mod_ssl, mod_rewrite, etc.;
that directive should be generated by
Apache-Test/lib/Apache/TestConfig.pm and not in module-specific config
snippets.

Sorry for the noise...