>>>>> On Sun, 09 Jul 2006 01:15:40 -0700, "Philip M. Gollucci" <suppressed> 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
Mail converted by mhonarc 2.6.15
This archive provided courtesy of JSW4.NET, Internet Hosting Services for Small Business.