perl-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Philip M. Gollucci" <>
Subject Re: mp2biug and httpd TestConfigData
Date Wed, 14 Sep 2005 08:13:57 GMT
Stas Bekman wrote:
> Philip M. Gollucci wrote:
> But I don't think you should ever load it manually. It's A-T's job to do 
> that. Just leave it there and talk to A-T if you need to get hold of 
> that info.
I've pinned it down now.


ModPerl::Config::as_string() ->

      if ($args->{thaw} and ref($thaw) ne 'HASH') {
         #dont generate any new config
         $thaw->{vars}->{$_} = $args->{$_} for keys %$args;
         $thaw->{server} = $thaw->new_test_server;
         return $thaw;

We return in this IF having _never_ loaded thus mp2bug doesn't ever
the path of httpd causing most of the missing info in the mp2bug reports we've seen to the
and my greif.

Here is an actual fix instead of a "quick hack" like my previous two attempts now that I understand
Thanks for the suggestions Stas! /home/pgollucci/dev/repos/asf/perl/modperl/Apache-Test/trunk/lib/Apache
rv=0 143 >svn diff
---       (revision 280754)
+++       (working copy)
@@ -252,6 +252,7 @@
          $thaw->{vars}->{$_} = $args->{$_} for keys %$args;
          $thaw->{server} = $thaw->new_test_server;
+        custom_config_add_conf_opts($args);
          return $thaw;

@@ -2153,6 +2154,8 @@
  sub custom_config_add_conf_opts {
      my $args = shift;

+    custom_config_load();
      return unless $Apache::TestConfigData::vars and
          keys %$Apache::TestConfigData::vars;

     What doesn't kill us can only make us stronger.
                 Nothing is impossible.
Philip M. Gollucci ( 301.254.5198
Consultant /
Senior Developer / Liquidity Services, Inc.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message