Consolidate the locations where we have our internal perl modules
authorRichard Levitte <levitte@openssl.org>
Tue, 1 Aug 2017 20:10:39 +0000 (22:10 +0200)
committerRichard Levitte <levitte@openssl.org>
Tue, 15 Aug 2017 09:30:47 +0000 (11:30 +0200)
Instead of having perl modules under test/testlib, util and util/perl,
consolidate them all to be inside util/perl.

Reviewed-by: Andy Polyakov <appro@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/4069)

22 files changed:
Configure
test/generate_ssl_tests.pl
test/recipes/80-test_ssl_new.t
test/recipes/tconversion.pl
test/run_tests.pl
util/dofile.pl
util/perl/OpenSSL/Test.pm [moved from test/testlib/OpenSSL/Test.pm with 100% similarity]
util/perl/OpenSSL/Test/Simple.pm [moved from test/testlib/OpenSSL/Test/Simple.pm with 100% similarity]
util/perl/OpenSSL/Test/Utils.pm [moved from test/testlib/OpenSSL/Test/Utils.pm with 100% similarity]
util/perl/TLSProxy/Certificate.pm [moved from util/TLSProxy/Certificate.pm with 100% similarity]
util/perl/TLSProxy/CertificateVerify.pm [moved from util/TLSProxy/CertificateVerify.pm with 100% similarity]
util/perl/TLSProxy/ClientHello.pm [moved from util/TLSProxy/ClientHello.pm with 100% similarity]
util/perl/TLSProxy/EncryptedExtensions.pm [moved from util/TLSProxy/EncryptedExtensions.pm with 100% similarity]
util/perl/TLSProxy/HelloRetryRequest.pm [moved from util/TLSProxy/HelloRetryRequest.pm with 100% similarity]
util/perl/TLSProxy/Message.pm [moved from util/TLSProxy/Message.pm with 100% similarity]
util/perl/TLSProxy/NewSessionTicket.pm [moved from util/TLSProxy/NewSessionTicket.pm with 100% similarity]
util/perl/TLSProxy/Proxy.pm [moved from util/TLSProxy/Proxy.pm with 100% similarity]
util/perl/TLSProxy/Record.pm [moved from util/TLSProxy/Record.pm with 100% similarity]
util/perl/TLSProxy/ServerHello.pm [moved from util/TLSProxy/ServerHello.pm with 100% similarity]
util/perl/TLSProxy/ServerKeyExchange.pm [moved from util/TLSProxy/ServerKeyExchange.pm with 100% similarity]
util/perl/checkhandshake.pm [moved from test/testlib/checkhandshake.pm with 100% similarity]
util/perl/with_fallback.pm [moved from util/with_fallback.pm with 83% similarity]

index 61b86c4..ada1472 100755 (executable)
--- a/Configure
+++ b/Configure
@@ -12,6 +12,8 @@
 use 5.10.0;
 use strict;
 use Config;
+use FindBin;
+use lib "$FindBin::Bin/util/perl";
 use File::Basename;
 use File::Spec::Functions qw/:DEFAULT abs2rel rel2abs/;
 use File::Path qw/mkpath/;
@@ -1384,7 +1386,6 @@ my %unified_info = ();
 
 my $buildinfo_debug = defined($ENV{CONFIGURE_DEBUG_BUILDINFO});
 if ($builder eq "unified") {
-    use lib catdir(dirname(__FILE__),"util");
     use with_fallback qw(Text::Template);
 
     sub cleandir {
@@ -1512,8 +1513,10 @@ if ($builder eq "unified") {
         my %generate = ();
 
         push @{$config{build_infos}}, catfile(abs2rel($sourced, $blddir), $f);
-        my $template = Text::Template->new(TYPE => 'FILE',
-                                           SOURCE => catfile($sourced, $f));
+        my $template =
+            Text::Template->new(TYPE => 'FILE',
+                                SOURCE => catfile($sourced, $f),
+                                PREPEND => qq{use lib "$FindBin::Bin/util/perl";});
         die "Something went wrong with $sourced/$f: $!\n" unless $template;
         my @text =
             split /^/m,
index fd785b7..47a328c 100644 (file)
@@ -22,7 +22,7 @@ BEGIN {
     OpenSSL::Test::setup("no_test_here");
 }
 
-use lib srctop_dir("util");  # for with_fallback
+use lib srctop_dir("util", "perl");  # for with_fallback
 use lib srctop_dir("test", "ssl-tests");  # for ssltests_base
 
 use with_fallback qw(Text::Template);
index de35190..f86e509 100644 (file)
@@ -120,7 +120,7 @@ sub test_conf {
 
       skip 'failure', 2 unless
         ok(run(perltest(["generate_ssl_tests.pl", $input_file],
-                        interpreter_args => [ "-I", srctop_dir("test", "testlib")],
+                        interpreter_args => [ "-I", srctop_dir("util", "perl")],
                         stdout => $tmp_file)),
            "Getting output from generate_ssl_tests.pl.");
 
index 1a308e2..fb0b766 100644 (file)
@@ -12,7 +12,6 @@ use warnings;
 
 use File::Compare qw/compare_text/;
 use File::Copy;
-use lib 'testlib';
 use OpenSSL::Test qw/:DEFAULT/;
 
 my %conversionforms = (
index 1695729..1171eec 100644 (file)
@@ -25,12 +25,11 @@ my $TAP_Harness = can_load(modules => { 'TAP::Harness' => undef })
 my $srctop = $ENV{SRCTOP} || $ENV{TOP};
 my $bldtop = $ENV{BLDTOP} || $ENV{TOP};
 my $recipesdir = catdir($srctop, "test", "recipes");
-my $testlib = catdir($srctop, "test", "testlib");
-my $utillib = catdir($srctop, "util");
+my $libdir = rel2abs(catdir($srctop, "util", "perl"));
 
 my %tapargs =
     ( verbosity => $ENV{VERBOSE} || $ENV{V} || $ENV{HARNESS_VERBOSE} ? 1 : 0,
-      lib       => [ $testlib, $utillib ],
+      lib       => [ $libdir ],
       switches  => '-w',
       merge     => 1
     );
index 0d05574..36827e3 100644 (file)
@@ -14,6 +14,7 @@
 use strict;
 use warnings;
 
+use FindBin;
 use Getopt::Std;
 
 # We actually expect to get the following hash tables from configdata:
@@ -38,7 +39,7 @@ package OpenSSL::Template;
 # a fallback in case it's not installed on the system
 use File::Basename;
 use File::Spec::Functions;
-use lib catdir(dirname(__FILE__));
+use lib "$FindBin::Bin/perl";
 use with_fallback qw(Text::Template);
 
 #use parent qw/Text::Template/;
@@ -175,7 +176,10 @@ my $text =
 # Load the full template (combination of files) into Text::Template
 # and fill it up with our data.  Output goes directly to STDOUT
 
-my $template = OpenSSL::Template->new(TYPE => 'STRING', SOURCE => $text );
+my $template =
+    OpenSSL::Template->new(TYPE => 'STRING',
+                           SOURCE => $text,
+                           PREPEND => qq{use lib "$FindBin::Bin/perl";});
 
 sub output_reset_on {
     $template->output_reset_on();
similarity index 83%
rename from util/with_fallback.pm
rename to util/perl/with_fallback.pm
index b6deb20..2af1d5f 100644 (file)
@@ -13,7 +13,8 @@ sub import {
     foreach (@_) {
        eval "require $_";
        if ($@) {
-           unshift @INC, catdir(dirname(__FILE__), "..", "external", "perl");
+           unshift @INC, catdir(dirname(__FILE__),
+                                 "..", "..", "external", "perl");
            my $transfer = "transfer::$_";
            eval "require $transfer";
            shift @INC;