[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: $PerlConfig broken


>>>>> 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.