>>>>> On Sun, 09 Jul 2006 01:15:40 -0700, "Philip M. Gollucci" <[EMAIL >>>>> PROTECTED]> said:
> Philip M. Gollucci wrote: >> Andreas J. Koenig wrote: >>> > http://marc2.theaimsgroup.com/?l=apache-modperl-dev&m=114021879222434&w=2 >> Since Geoff was previously involved, I'll let him handle this, but I'm +1 >> for the patch >> as add_config() takes a scalar string not an array, so dump_special() as >> writting is clearly wrong. >> >> The only thing I could thing of differently would be if you wanted instead >> of the foreach() >> $self->add_config(join('', @data)) >> >> I've not traced what dump_special is supposed to be doing yet, so I don't >> know which is correct. >> >> > I take that back, > do something like this instead: > <Perl > > use Apache::Test (); > if (Apache::Test::have_module('mod_alias.c')) { > push @Alias, ['/perl_sections', '@DocumentRoot@']; > $Location{'/perl_sections'} = { > 'PerlInitHandler' => 'ModPerl::Test::add_config', > 'AuthType' => 'Basic', > 'AuthName' => 'PerlSection', > 'PerlAuthenHandler' => 'TestHooks::authen_basic', > }; > } > </Perl> I'm not sure what you want to say. The above has not much to do with my bugreport, at least on the surface. Are you suggesting I should not use $PerlConfig and prefer the %Location/@Alias syntax? Then I must say, I'm not a fan of these constructs, I *muchly* prefer $PerlConfig. -- andreas
