Stas Bekman wrote:

Steve Hay wrote:
[...]

This makes sense after reading

http://perl.apache.org/docs/2.0/user/config/config.html#C_modperl_
http://perl.apache.org/docs/2.0/user/config/config.html#C_perl_script_


and sure enough the various tests that fail, e.g.

   apache\scanhdrs
   api\rflush
   modperl\post_utf8
   modules\cgi

specify "SetHandler perl-script" in their .pm files, while the test above that succeeded (api\lookup_ui) specifies "SetHandler modperl" in its .pm file.


Not a single test that uses 'perl-script' passes?

Without trawling through all the tests trying each one individually under t/SMOKE and looking to see how it is run (perl-script/modperl) I couldn't say for sure. I just happened to spot that the few failing ones that I've been looking at were all perl-script and a working one was modperl.




So it looks like rather than us having the redirection stuff oddly working for some tests and not for others, what we actually have is a case of it never working! The tests that work only do so because they use the "modperl" response handler which skips the tie()'ing of STDIN/STDOUT.


But you said that they did work from 'make test'/ t/TEST, for some reason they only fail when run from t/SMOKE. Is that correct? So it's some sort of condition that t/SMOKE creates, creating the grounds for this problem.

Yes, that's correct - "nmake test" runs the entire testsuite without error, "perl t/SMOKE" falls over on various tests.


I assumed it was something to do with the IPC::Run3 stuff in TestSmoke.pm that we've been playing with (specifically the fact that it introduces new redirections of its own), and I was under the impression that it is only t/SMOKE that uses this TestSmoke module. Is that not the case?

My thinking was that maybe the redirections in TestSmoke clash with the redirections done by perl-script, but they're fine with the modperl handler because that doesn't do any redirections of its own.

- Steve


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



Reply via email to