Matt, thanks for the quick response.  I still have some questions
though.

> 
>  run_tests:
> -       $(FULLPERL) $(MP_TEST_SCRIPT) $(TEST_VERBOSE)
> +       $(PERL5LIB) $(FULLPERL) $(MP_TEST_SCRIPT) $(TEST_VERBOSE)
> 

I understand the intent of the PERL5LIB macro now, but I'd still need
all the changes except for this: 

-       PERL5LIB => "PERL5LIB=$ENV{PERL5LIB}",
+       PERL5LIB => "$ENV{PERL5LIB}",

right?

If your patch was all I applied, all it would fix is the odd
'../PERL5LIB/xxx/xxx/xxx' entry in @INC.  It still wouldn't preserve my
original PERL5LIB without this change to Makefile.PL

-       $ENV{PERL5LIB} = "$PWD/lib";
+       $ENV{PERL5LIB} = "$PWD/lib" . ($ENV{PERL5LIB} ?
":$ENV{PERL5LIB}" : "");

And the addition of 

+       PassEnv PERL5LIB

to httpd.conf is still required.  Otherwise the cgi-bin tests will fail.
I think it's probably a good thing to not preserve the PERL5LIB under
'-T' and force the user to create the local.pl with explicit 'use lib
...' statements, so I don't consider that a bug.

I realize all this only affects the 'make test', but I like to know that
it's built cleanly before installing new versions, and I'd hope I wasn't
the only one out there who ran the tests.

Thanks for your time and your help.

Jim


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to