Skip Menu |

This queue is for tickets about the Crypt-SSLeay CPAN distribution.

Report information
The Basics
Id: 30268
Status: resolved
Priority: 0/
Queue: Crypt-SSLeay

People
Owner: nanis [...] runu.moc.invalid
Requestors: rnorwood [...] redhat.com
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.57
Fixed in: 0.57_01



Subject: Makefile.PL should not always prompt for user input
Hi, Attached is a patch that adds a flag to Makefile.PL to allow the package builder to enable or disable the live tests. This lets package builders build the package without having it prompt for user input. We're using this patch on Fedora for that purpose. Thanks, -RN
Subject: perl-Crypt-SSLeay-0.57-live-tests.patch
--- Crypt-SSLeay-0.57/Makefile.PL Mon Sep 17 16:36:59 2007 +++ Crypt-SSLeay-0.57/Makefile.PL Thu Oct 25 13:45:53 2007 @@ -9,12 +9,13 @@ eval "use ExtUtils::MakeMaker::Coverage"; $@ or print "Adding testcover target\n"; -use vars qw($opt_default $opt_libpath $opt_static); +use vars qw($opt_default $opt_libpath $opt_static $opt_live_tests); GetOptions( "default", \$opt_default, "lib=s", \$opt_libpath, "static", \$opt_static, + "live-tests!", \$opt_live_tests, ); $opt_default ||= $ENV{CRYPT_SSLEAY_DEFAULT}; @@ -275,17 +276,23 @@ cc $Config{cc} INFO - print <<"INFO"; + my $network_tests = 'n'; + if (not defined $opt_live_tests) { + print <<"INFO"; The test suite can attempt to connect to public servers to ensure that the code is working properly. If you are behind a strict firewall or have no network connectivity, these tests may fail (through no fault of the code). - INFO - my $network_tests = prompt - "Do you want to run the live tests (y/N) ?", - 'N'; + + $network_tests = prompt "Do you want to run the live tests (y/N) ?", 'N'; + } + elsif ($opt_live_tests) { + $network_tests = 'y'; + } + print OUT "network_tests ", ($network_tests =~ /y/i) ? 1 : 0, "\n"; + close OUT; }
Sorry, I've been sitting on this patch for quite a while. I'm going through the tickets that have piled up over the last couple of months. I shall address them all and push out a new release soon. Thanks for your patience, David Landgren
On Thu Oct 25 13:57:14 2007, rnorwood wrote: Show quoted text
> Hi, > > Attached is a patch that adds a flag to Makefile.PL to allow the package > builder to enable or disable the live tests. This lets package builders > build the package without having it prompt for user input. We're using > this patch on Fedora for that purpose. > > Thanks, > > -RN
Hi, I've included this patch in 0.57_01. If the smoke tests this week are all white, I'll release 0.58 in short order. Thanks, David