-----Original Message----- From: Ed J via RT

Confirmation from #perl on irc.perl.org - it's a deliberate change in perl 5.20.0. A quick fix would be either to explicitly set $ENV{PATH} to '/bin:/usr/bin', or skip the test for 5.20.0.

Really ? I thought it was purely dependent upon system configuration, and completely independent of perl version. On my Windows 7, Ubuntu 12.04, and Debian Wheezy systems the 08taint.t tests pass (for perl-5.20.0 as well as earlier versions of perl).

I hoped there would be a sensible value available in %Config, but there isn't.

I would happily dismantle Inline's attempted taint handling if:
a) Ingy gives his blessing for that to happen;
&&
b) there's a consensus that this is the right thing to do.

So far neither has happened.
In the meantime, patches are welcome.

I guess there are other things we could do - eg skip the 08taint.t test script if (eg) $ENV{INLINE_NTT} was set. ("NTT" being a mnemonic for "No Taint Testing"). I've no objection to doing that. In fact, I think I might do just that - it comes at no cost to those who don't want to make use of the option.

However, I don't think I would like to force those tests to be skipped for 5.20. Someone might not notice that - and then get really annoyed because the test suite didn't disclose to them that taint did not work.

Cheers,
Rob

Reply via email to