httpd-apreq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jamie Krasnoo <krasn...@socal.rr.com>
Subject Re: [apreq-2] Apache::Test preinstall issues
Date Sun, 20 Jul 2003 11:21:10 GMT
On Sun, 2003-07-20 at 01:59, Joe Schaefer wrote:
> Randy Kobes <randy@theoryx5.uwinnipeg.ca> writes:
> 
> [...]
> 
> > Does the following work?
> > =================================================================
> > Index: TEST.PL
> > ================================================================
> > RCS file: /home/cvs/httpd-apreq-2/env/t/TEST.PL,v
> > retrieving revision 1.2
> > diff -u -r1.2 TEST.PL
> > --- TEST.PL	30 Jun 2003 20:42:14 -0000	1.2
> > +++ TEST.PL	20 Jul 2003 04:05:22 -0000
> > @@ -18,6 +18,12 @@
> >  $Apache::TestTrace::Level = 'debug';
> >  main::->new->run(@ARGV);
> > 
> > +sub pre_configure {
> > +    my $self = shift;
> > +    # Don't load an installed mod_apreq
> > +    Apache::TestConfig::autoconfig_skip_module_add('mod_apreq.c');
> > +}
> > +
> >  sub configure {
> >      my $self = shift;
> >      my $cfg = $self->{test_config};
> 
> Yes, it does.
> 
> > >
> > > The second problem is with the perl_tests:
> > >
> > >   % tail glue/perl/t/conf/modperl_inc.pl
> > >   ...
> > >   use lib '/path/to/httpd-apreq-2/glue/perl/t/response';
> > >   use lib '/path/to/httpd-apreq-2/glue/perl/blib/arch';
> > >   use lib '/path/to/httpd-apreq-2/glue/perl/blib/lib';
> > >   use Apache2;
> > >   1;
> > >
> > > The 'use Apache 2' directive needs to occur before
> > > the 'use lib ...' directives to avoid using a previously-
> > > installed Apache::Request.
> > 
> > This is being written from Apache::TestConfigPerl, in the
> > configure_startup_pl sub. The Apache2 entry comes from the call
> > to Apache::TestConfig->modperl_2_inc_fixup() - moving the
> > printing of $fixup to $fh to somewhere before the line
> >    print $fh "use lib '$_';\n";
> > would get the order you want, but I'm not sure how that
> > would affect, eg, the mod_perl tests.
> 
> It certainly won't affect the modperl build tests, since 
> modperl_2_inc_fixup is a noop there.  I suspect that
> this misordering is a bug in configure_startup_pl,
> but for the time being we can simply override it.
> 
> I've applied the relevant patches to cvs- please give it
> a try (you need to % make clean to clear out the old test
> configs).

I pulled down the changes and rebuilt it. All tests passed.

Jamie


Mime
View raw message