[perl-Config-IniFiles/el5] add explicit Requires for perl(List::MoreUtils)

Tom Callaway spot at fedoraproject.org
Tue Jun 26 01:07:42 UTC 2012


commit 3a3f04a037befd5eb07150d32ac0cea33b83e107
Author: Tom Callaway <spot at fedoraproject.org>
Date:   Mon Jun 25 21:08:01 2012 -0400

    add explicit Requires for perl(List::MoreUtils)

 perl-Config-IniFiles-2.72-el6.patch  |   52 ++++++++++++++++++++++++++++++++++
 perl-Config-IniFiles-2.72-warn.patch |   25 ++++++++++++++++
 perl-Config-IniFiles.spec            |   18 ++++++++++--
 3 files changed, 92 insertions(+), 3 deletions(-)
---
diff --git a/perl-Config-IniFiles-2.72-el6.patch b/perl-Config-IniFiles-2.72-el6.patch
new file mode 100644
index 0000000..c1be389
--- /dev/null
+++ b/perl-Config-IniFiles-2.72-el6.patch
@@ -0,0 +1,52 @@
+From 53d46071471820e0f18b5f5cd439e24cb7f94678 Mon Sep 17 00:00:00 2001
+From: Lubomir Rintel <lubo.rintel at gooddata.com>
+Date: Fri, 1 Jun 2012 13:55:39 +0200
+Subject: [PATCH 1/2] Older List::MoreUtils provide no 'none' function
+
+---
+ lib/Config/IniFiles.pm |    8 ++++----
+ 1 files changed, 4 insertions(+), 4 deletions(-)
+
+diff --git a/lib/Config/IniFiles.pm b/lib/Config/IniFiles.pm
+index 32a496e..1ababa2 100644
+--- a/lib/Config/IniFiles.pm
++++ b/lib/Config/IniFiles.pm
+@@ -11,7 +11,7 @@ use Carp;
+ use Symbol 'gensym','qualify_to_ref';   # For the 'any data type' hack
+ use Fcntl qw( SEEK_SET SEEK_CUR );
+ 
+-use List::MoreUtils qw(any none);
++use List::MoreUtils qw(any);
+ 
+ use File::Basename qw( dirname );
+ use File::Temp qw/ tempfile /;
+@@ -687,7 +687,7 @@ sub newval {
+ 
+     $self->AddSection($sect);
+ 
+-    if (none { $_ eq $parm } @{$self->{parms}{$sect}})
++    if (not grep { $_ eq $parm } @{$self->{parms}{$sect}})
+     {
+         CORE::push(@{$self->{parms}{$sect}}, $parm)
+     }
+@@ -1394,7 +1394,7 @@ sub _touch_section {
+ 
+     $self->{mysects} ||= [];
+ 
+-    if (none { $_ eq $sect } @{$self->{mysects}})
++    if (not grep { $_ eq $sect } @{$self->{mysects}})
+     {
+         CORE::push @{$self->{mysects}}, $sect;
+     }
+@@ -1535,7 +1535,7 @@ sub _SetGroupMember_helper
+         $self->{group}{$group} = [];
+     }
+ 
+-    if (none {$_ eq $sect} @{$self->{group}{$group}}) {
++    if (not grep {$_ eq $sect} @{$self->{group}{$group}}) {
+         CORE::push @{$self->{group}{$group}}, $sect;
+     }
+ 
+-- 
+1.7.1
+
diff --git a/perl-Config-IniFiles-2.72-warn.patch b/perl-Config-IniFiles-2.72-warn.patch
new file mode 100644
index 0000000..98bd8d0
--- /dev/null
+++ b/perl-Config-IniFiles-2.72-warn.patch
@@ -0,0 +1,25 @@
+From 688ed6edbc7d94181657de0de2ac9dc1ea5a20da Mon Sep 17 00:00:00 2001
+From: Lubomir Rintel <lubo.rintel at gooddata.com>
+Date: Fri, 1 Jun 2012 13:52:38 +0200
+Subject: [PATCH 2/2] Get rid of a warning
+
+---
+ lib/Config/IniFiles.pm |    2 +-
+ 1 files changed, 1 insertions(+), 1 deletions(-)
+
+diff --git a/lib/Config/IniFiles.pm b/lib/Config/IniFiles.pm
+index 1ababa2..281c78c 100644
+--- a/lib/Config/IniFiles.pm
++++ b/lib/Config/IniFiles.pm
+@@ -823,7 +823,7 @@ sub _no_filename
+ {
+     my $self = shift;
+ 
+-    return not length $self->{cf};
++    return not length ($self->{cf} || '');
+ }
+ 
+ sub _read_line_num
+-- 
+1.7.1
+
diff --git a/perl-Config-IniFiles.spec b/perl-Config-IniFiles.spec
index 24b491c..ab885d5 100644
--- a/perl-Config-IniFiles.spec
+++ b/perl-Config-IniFiles.spec
@@ -1,6 +1,6 @@
 Name:           perl-Config-IniFiles
 Version:        2.72
-Release:        1%{?dist}
+Release:        2%{?dist}.1
 Summary:        A module for reading .ini-style configuration files
 
 Group:          Development/Libraries
@@ -9,6 +9,9 @@ URL:            http://search.cpan.org/dist/Config-IniFiles/
 Source0:        http://www.cpan.org/authors/id/S/SH/SHLOMIF/Config-IniFiles-%{version}.tar.gz
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
+Patch0:         perl-Config-IniFiles-2.72-el6.patch
+Patch1:         perl-Config-IniFiles-2.72-warn.patch
+
 BuildRequires:  perl(Module::Build::Compat)
 BuildRequires:  perl(ExtUtils::MakeMaker)
 BuildRequires:  perl(Test::More)
@@ -17,6 +20,8 @@ BuildArch:      noarch
 Requires:       perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
 # Not autodetected. Found in lib/Config/IniFiles.pm:2265
 Requires:       perl(IO::Scalar) >= 2.109
+# Not autodetected on el5
+Requires:	perl(List::MoreUtils)
 
 %description
 Config::IniFiles provides a way to have readable configuration files
@@ -27,6 +32,8 @@ from a tied hash.
 
 %prep
 %setup -q -n Config-IniFiles-%{version}
+%patch0 -p1
+%patch1 -p1
 
 %build
 %{__perl} Makefile.PL INSTALLDIRS=vendor
@@ -42,9 +49,7 @@ chmod -R u+w $RPM_BUILD_ROOT/*
 
 
 %check
-%if 0%{?fedora}
 make test
-%endif
 
 
 %clean
@@ -59,6 +64,13 @@ rm -rf $RPM_BUILD_ROOT
 
 
 %changelog
+* Mon Jun 25 2012 Tom Callaway <spot at fedoraproject.org> - 2.72-2.1
+- add explicit Requires: perl(List::MoreUtils) for el5 (bz827198)
+
+* Fri Jun 01 2012 Lubomir Rintel (GoodData) <lubo.rintel at gooddata.com> - 2.72-2
+- Fix compatibility with el6
+- Enable test suite
+
 * Fri May 11 2012 Tom Callaway <spot at fedoraproject.org> - 2.72-1
 - update to 2.72
 - notable fix: SECURITY BUG FIX: Config::IniFiles used to write 


More information about the scm-commits mailing list