[perl-CPAN-Inject] Expect an error if DNS does not work

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



commit 7fc3da260f4bb4cf0a2e233553704e7cfdc85ac8
Author: Petr Písař <ppisar@xxxxxxxxxx>
Date:   Thu Sep 11 10:42:16 2014 +0200

    Expect an error if DNS does not work

 ...own-exception-while-loading-CPAN-configur.patch |   81 ++++++++++++++++++++
 perl-CPAN-Inject.spec                              |    8 ++-
 2 files changed, 88 insertions(+), 1 deletions(-)
---
diff --git a/CPAN-Inject-1.14-Expect-unknown-exception-while-loading-CPAN-configur.patch b/CPAN-Inject-1.14-Expect-unknown-exception-while-loading-CPAN-configur.patch
new file mode 100644
index 0000000..dc454b1
--- /dev/null
+++ b/CPAN-Inject-1.14-Expect-unknown-exception-while-loading-CPAN-configur.patch
@@ -0,0 +1,81 @@
+From 1fd98154c60e0ab1d03f703f23b6775aed2bfc68 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Petr=20P=C3=ADsa=C5=99?= <ppisar@xxxxxxxxxx>
+Date: Thu, 11 Sep 2014 10:35:29 +0200
+Subject: [PATCH] Expect unknown exception while loading CPAN configuration
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+If DNS resolution does not work, CPAN run for the first time could not
+load its configuration and returns different exception than
+t/02_main.t expects.
+
+This patch removes the bogus test and wrappes the exception to
+provide better explanation pointing to the CPAN as the culprit.
+
+CPAN RT#98774
+
+Signed-off-by: Petr Písař <ppisar@xxxxxxxxxx>
+---
+ lib/CPAN/Inject.pm | 13 +++++++++----
+ t/02_main.t        | 11 ++---------
+ 2 files changed, 11 insertions(+), 13 deletions(-)
+
+diff --git a/lib/CPAN/Inject.pm b/lib/CPAN/Inject.pm
+index 86cf9ea..89fb120 100644
+--- a/lib/CPAN/Inject.pm
++++ b/lib/CPAN/Inject.pm
+@@ -209,12 +209,17 @@ sub from_cpan_config {
+ 	require Cwd;
+ 	my $origin_working_directory = Cwd::getcwd;
+ 
+-	# Support for different mechanisms depending on the version
+-	# of CPAN that is in use.
+-	if ( defined $CPAN::HandleConfig::VERSION ) {
++	eval {
++	    # Support for different mechanisms depending on the version
++	    # of CPAN that is in use.
++	    if ( defined $CPAN::HandleConfig::VERSION ) {
+ 		CPAN::HandleConfig->load;
+-	} else {
++	    } else {
+ 		CPAN::Config->load;
++	    }
++	};
++	if ($@) {
++	    die "CPAN could not load its configuration: $@";
+ 	}
+ 
+ 	# Restore working directory in case CPAN has changed it, RT#94963
+diff --git a/t/02_main.t b/t/02_main.t
+index 4686a7e..3e1b7a6 100644
+--- a/t/02_main.t
++++ b/t/02_main.t
+@@ -8,7 +8,7 @@ BEGIN {
+ 	$^W = 1;
+ }
+ 
+-use Test::More tests => 24;
++use Test::More tests => 23;
+ use File::Spec::Functions ':ALL';
+ use File::Remove          'remove';
+ use CPAN::Inject;
+@@ -76,14 +76,7 @@ SCOPE: {
+ 			);
+ 	};
+ 	SKIP: {
+-		skip( "Current user owns CPAN::Config", 1 ) unless $@;
+-		like($@,
+-			qr/(The directory .* does not exist|The sources directory is not owned by the current user)/, 
+-			'Got expected error',
+-		);
+-	}
+-	SKIP: {
+-		skip( "Current user does not own CPAN::Config", 2 ) if $@;
++		skip( "Error while loading CPAN configuration ($@)", 2 ) if $@;
+ 		isa_ok( $cpan, 'CPAN::Inject' );
+ 		is( $cpan->author,  'ADAMK',  '->author ok' );
+ 	}
+-- 
+1.9.3
+
diff --git a/perl-CPAN-Inject.spec b/perl-CPAN-Inject.spec
index 1c61d6e..7319c64 100644
--- a/perl-CPAN-Inject.spec
+++ b/perl-CPAN-Inject.spec
@@ -1,6 +1,6 @@
 Name:           perl-CPAN-Inject
 Version:        1.14
-Release:        7%{?dist}
+Release:        8%{?dist}
 Summary:        Base class for injecting distributions into CPAN sources
 License:        GPL+ or Artistic
 Group:          Development/Libraries
@@ -8,6 +8,8 @@ URL:            http://search.cpan.org/dist/CPAN-Inject/
 Source0:        http://www.cpan.org/authors/id/P/PS/PSHANGOV/CPAN-Inject-%{version}.tar.gz
 # Work around CPAN bug mangling working directory, bug #1084093, CPAN RT#94963
 Patch0:         CPAN-Inject-1.14-Restore-working-directory-after-loading-CPAN-configu.patch
+# Expect en error if DNS does not work, bug #1138562, CPAN RT#98774
+Patch1:         CPAN-Inject-1.14-Expect-unknown-exception-while-loading-CPAN-configur.patch
 BuildArch:      noarch
 BuildRequires:  perl(CPAN) >= 1.36
 BuildRequires:  perl(CPAN::Checksums) >= 1.05
@@ -44,6 +46,7 @@ created to add additional distributions into a minicpan mirror.
 %prep
 %setup -q -n CPAN-Inject-%{version}
 %patch0 -p1
+%patch1 -p1
 
 # Remove bundled libraries
 rm -r inc
@@ -72,6 +75,9 @@ make test </dev/null
 %{_mandir}/man3/*
 
 %changelog
+* Thu Sep 11 2014 Petr Pisar <ppisar@xxxxxxxxxx> - 1.14-8
+- Expect an error if DNS does not work (bug #1138562)
+
 * Mon Sep 01 2014 Jitka Plesnikova <jplesnik@xxxxxxxxxx> - 1.14-7
 - Perl 5.20 rebuild
 
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/perl-devel





[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Legacy Announce]     [Fedora PHP Devel]     [Kernel Devel]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Big List of Linux Books]     [Gimp]     [Yosemite Information]
  Powered by Linux