Skip to content

Commit

Permalink
Bug 1137669: 003safesys.t doesn't test any file due to a missing -T a…
Browse files Browse the repository at this point in the history
…rgument

r=dylan a=glob
  • Loading branch information
LpSolit committed Mar 16, 2015
1 parent f0c2b6b commit fc4a6dd
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion t/003safesys.t
Expand Up @@ -38,7 +38,16 @@ my $perlapp = "\"$^X\"";
foreach my $file (@testitems) {
$file =~ s/\s.*$//; # nuke everything after the first space (#comment)
next if (!$file); # skip null entries
my $command = "$perlapp -c -It -MSupport::Systemexec $file 2>&1";

open(my $fh2, '<', $file);
my $bang = <$fh2>;
close $fh2;

my $T = "";
if ($bang =~ m/#!\S*perl\s+-.*T/) {
$T = "T";
}
my $command = "$perlapp -c$T -It -MSupport::Systemexec $file 2>&1";
my $loginfo=`$command`;
if ($loginfo =~ /arguments for Support::Systemexec::(system|exec)/im) {
ok(0,"$file DOES NOT use proper system or exec calls");
Expand Down

0 comments on commit fc4a6dd

Please sign in to comment.