httpd-apreq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Schaefer <joe+gm...@sunstarsys.com>
Subject Re: [apreq-2] Apache::Test preinstall issues
Date Sun, 20 Jul 2003 08:59:16 GMT
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).

-- 
Joe Schaefer

Mime
View raw message