Skip Menu |

This queue is for tickets about the Carp-Assert CPAN distribution.

Report information
The Basics
Id: 21170
Status: resolved
Priority: 0/
Queue: Carp-Assert

People
Owner: Nobody in particular
Requestors: cpan [...] tty.nl
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: (no value)
Fixed in: (no value)



Subject: PERL_NDEBUG was not deleted in t/30env_disabled.t
diff -u8pr Carp-Assert-0.18/t/30env_disabled.t Carp-Assert-0.18.org/t/30env_disabled.t --- Carp-Assert-0.18/t/30env_disabled.t 2006-08-24 19:32:04.000000000 +0200 +++ Carp-Assert-0.18.org/t/30env_disabled.t 2001-02-09 16:29:14.000000000 +0100 @@ -7,17 +7,17 @@ use strict; $| = 1; print "1..5\n"; my $t_num = 1; local $@; $@ = ''; -BEGIN { delete $ENV{PERL_NDEBUG}; $ENV{NDEBUG} = 1; } +BEGIN { $ENV{NDEBUG} = 1; } use Carp::Assert; eval { assert(1==0) if DEBUG; }; print "not " if $@; print "ok ",$t_num++,"\n"; $@ = ''; eval { assert(1==0); };
I've shielded the tests against the user's environment, this should be resolved by 0.20 when it comes out shortly. I could not use your patch because it is unsafe on VMS where %ENV persists between processes. I came up with something else.