On Thu, 03 Apr 2008 21:18:26 +0200
Ruediger Pluem <[EMAIL PROTECTED]> wrote:

> > +    ap_parse_node_t *condition;   /* Conditionally merge <If>
> > sections */ } core_dir_config;
> 
> Does this work correctly without adjusting merge_core_dir_configs?

Good point - I'll take a look.  For sure there's more work to do.


> > +    cmd->path = old_path;
> 
> Who changes cmd->path?

Erm ... looks like a cut&paste remnant.

> Why does this belong in an else branch? I guess if the condition in
> entry_core->condition is true we still need to do the previous check.

No.  The other check applies to <Files> / <FilesMatch> sections,
whose code path I've hijacked for <If>.  The two are mutually exclusive.

-- 
Nick Kew

Application Development with Apache - the Apache Modules Book
http://www.apachetutor.org/

Reply via email to