On 08/21/2015 04:31 PM, Yann Ylavic wrote: > On Fri, Aug 21, 2015 at 4:29 PM, Yann Ylavic <[email protected]> wrote: >> On Fri, Aug 21, 2015 at 4:25 PM, Ruediger Pluem <[email protected]> wrote: >>> Wasn't it intended for trunk to change the default behavior? >> >> Yes it was, still when inherit_before is set we need to do that. > > I meant, we need to do "apr_array_append(p, over->patterns, > base->patterns)" and not the inverse. >
But apr_array_append(p, over->patterns, base->patterns) means that we do NOT inherit the stuff from the base configuration before the configuration of our section. So the code before your recent patch was correct. Regards Rüdiger
