[perl-Class-Load] Drop explicit dependency on core module perl(Exporter)
Paul Howarth
pghmcfc at fedoraproject.org
Tue Dec 21 09:43:09 UTC 2010
commit 3a84c5683a3b1c176b58004d3e17a20c2afa88fa
Author: Paul Howarth <paul at city-fan.org>
Date: Tue Dec 21 09:36:38 2010 +0000
Drop explicit dependency on core module perl(Exporter)
In the package review, the reviewer noted the explicit dependency on
perl(Exporter), added because of a "use base Exporter" construct in
the code that rpm's automatic dependency finder doesn't find, and asked
if that wasn't a bit redundant given that Exporter is a core module.
The explicit dependency has been removed on the basis that it is
extremely unlikely that perl(Exporter) will ever be split from the main
perl package and hence the explicit dependency is indeed redundant.
Ref: https://bugzilla.redhat.com/show_bug.cgi?id=656408
perl-Class-Load.spec | 7 +++++--
1 files changed, 5 insertions(+), 2 deletions(-)
---
diff --git a/perl-Class-Load.spec b/perl-Class-Load.spec
index 51b42b1..8d87129 100644
--- a/perl-Class-Load.spec
+++ b/perl-Class-Load.spec
@@ -1,6 +1,6 @@
Name: perl-Class-Load
Version: 0.06
-Release: 2%{?dist}
+Release: 3%{?dist}
Summary: A working (require "Class::Name") and more
Group: Development/Libraries
License: GPL+ or Artistic
@@ -12,7 +12,7 @@ BuildRequires: perl(ExtUtils::MakeMaker)
BuildRequires: perl(Test::Fatal)
BuildRequires: perl(Test::More)
Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
-Requires: perl(Exporter)
+# Also requires core module perl(Exporter) via a "use base" construct
%description
require EXPR only accepts Class/Name.pm style module names, not Class::Name.
@@ -54,6 +54,9 @@ rm -rf %{buildroot}
%{_mandir}/man3/Class::Load.3pm*
%changelog
+* Tue Dec 21 2010 Paul Howarth <paul at city-fan.org> - 0.06-3
+- Drop explicit dependency on core module perl(Exporter) (#656408)
+
* Tue Nov 23 2010 Paul Howarth <paul at city-fan.org> - 0.06-2
- Sanitize spec for Fedora submission
More information about the perl-devel
mailing list