Fix the check of test apps in util/mk1mf.pl
authorRichard Levitte <levitte@openssl.org>
Mon, 27 Apr 2015 09:10:17 +0000 (11:10 +0200)
committerRichard Levitte <levitte@openssl.org>
Mon, 27 Apr 2015 13:55:53 +0000 (15:55 +0200)
The previous check assumed that the variables for each test app, ending
with TEST would be indication enough.  Experience showed that this isn't
the best way.  Instead, simply look for the EXE variable in test/Makefile.

Reviewed-by: Rich Salz <rsalz@openssl.org>
util/mk1mf.pl

index 454dcd27907e169b904d25d54842f0934d284fb9..7b5705534d7b56d387bcc6c84ed261537d416675 100755 (executable)
@@ -396,8 +396,15 @@ for (;;)
        if ($key eq "EX_LIBS")
                { $ex_libs .= " $val" if $val ne "";}
 
-       if ($key =~ /^[A-Z0-9_]*TEST$/ && (!$fipscanisteronly || $dir =~ /^fips/ ))
-               { $test.=&var_add($dir,$val, 0); }
+       # There was a condition here before:
+       #       !$fipscanisteronly || $dir =~ /^fips/
+       # It currently fills no function and needs to be rewritten anyway, so
+       # removed for now.
+       if ($dir eq "test" && $key eq "EXE")
+               {
+               foreach my $t (split /\s+/, $val) {
+                       $test.=&var_add($dir,$t, 0) if $t; }
+               }
 
        if ($key eq "EXE_OBJ")
                { $e_exe.=&var_add($dir,$val, 0); }