>> hmm, what was the rationale from changing it from __FILE__ to /dev/null? >> just Apache2::Status? why not undef instead of some fake entry? it's >> just >> a hash afterall :) > > I originally had a patch for Apache2::Status for mod_perl.pm > specifically so that you didn't get two lines displayed for for > mod_perl2.pm and 1 for mod_perl.pm.... Very confusing if you had both in > the same perl tree. > > Stas suggested that rather then trying to patch each and every thing > that relies on it, we patch the source lib/mod_perl.pm with the SVN > commit above. I agreed.
ok. well, let's see if the patch for Apache2::Request helps a bit (I wasn't able to quickly reproduce the error in the mp2 test suite). if it does I guess we'll be ok. well, we might want to propagate a similar change to Apache::Reload on CPAN and Apache::StatINC in mp1... --Geoff