[perl-Test-Vars] Fix compatibility with Perl 5.16 (CPAN RT#72133)
Paul Howarth
pghmcfc at fedoraproject.org
Tue Jun 12 23:19:55 UTC 2012
commit 0286f96139d758b73c4967a2fe9cd1718903ec6e
Author: Paul Howarth <paul at city-fan.org>
Date: Wed Jun 13 00:15:19 2012 +0100
Fix compatibility with Perl 5.16 (CPAN RT#72133)
- Fix compatibility with Perl 5.16 (CPAN RT#72133)
- Don't need to remove empty directories from buildroot
Test-Vars-perl516.patch | 21 +++++++++++++++++++++
perl-Test-Vars.spec | 11 +++++++++--
2 files changed, 30 insertions(+), 2 deletions(-)
---
diff --git a/Test-Vars-perl516.patch b/Test-Vars-perl516.patch
new file mode 100644
index 0000000..72ff9f6
--- /dev/null
+++ b/Test-Vars-perl516.patch
@@ -0,0 +1,21 @@
+https://rt.perl.org/rt3/Ticket/Display.html?id=102822
+https://rt.cpan.org/Public/Bug/Display.html?id=72133
+https://rt.cpan.org/Public/Bug/Display.html?id=77438
+
+--- lib/Test/Vars.pm
++++ lib/Test/Vars.pm
+@@ -196,9 +196,13 @@
+ my $op_entereval; # string eval
+ my @op_svusers;
+ BEGIN{
+- foreach my $op(qw(padsv padav padhv aelemfast)){
++ foreach my $op(qw(padsv padav padhv)){
+ $padops[B::opnumber($op)]++;
+ }
++ # blead commit 93bad3fd55489cbd split aelemfast into two ops.
++ # Prior to that, 'aelemfast' handled lexicals too.
++ my $aelemfast = B::opnumber('aelemfast_lex');
++ $padops[$aelemfast == -1 ? B::opnumber('aelemfast') : $aelemfast]++;
+
+ $op_anoncode = B::opnumber('anoncode');
+ $padops[$op_anoncode]++;
diff --git a/perl-Test-Vars.spec b/perl-Test-Vars.spec
index 61a632e..3c6e9f8 100644
--- a/perl-Test-Vars.spec
+++ b/perl-Test-Vars.spec
@@ -1,11 +1,12 @@
Name: perl-Test-Vars
Version: 0.001
-Release: 4%{?dist}
+Release: 5%{?dist}
Summary: Detects unused variables
License: GPL+ or Artistic
Group: Development/Libraries
URL: http://search.cpan.org/dist/Test-Vars/
Source0: http://search.cpan.org/CPAN/authors/id/G/GF/GFUJI/Test-Vars-%{version}.tar.gz
+Patch0: Test-Vars-perl516.patch
BuildArch: noarch
BuildRequires: perl >= 4:5.10.0
BuildRequires: perl(B)
@@ -25,6 +26,9 @@ Test::Vars finds unused variables in order to keep the source code tidy.
%prep
%setup -q -n Test-Vars-%{version}
+# Fix compatibility with Perl 5.16 (CPAN RT#72133)
+%patch0
+
# Placate rpmlint about script interpreters in examples
sed -i -e '1s|^#!perl|#!/usr/bin/perl|' example/*.t
@@ -35,7 +39,6 @@ make %{?_smp_mflags}
%install
make pure_install DESTDIR=%{buildroot}
find %{buildroot} -type f -name .packlist -exec rm -f {} \;
-find %{buildroot} -depth -type d -exec rmdir {} \; 2>/dev/null
%{_fixperms} %{buildroot}
%check
@@ -48,6 +51,10 @@ make test TEST_FILES="xt/*.t"
%{_mandir}/man3/Test::Vars.3pm*
%changelog
+* Tue Jun 12 2012 Paul Howarth <paul at city-fan.org> - 0.001-5
+- Fix compatibility with Perl 5.16 (CPAN RT#72133)
+- Don't need to remove empty directories from buildroot
+
* Tue Jun 12 2012 Petr Pisar <ppisar at redhat.com> - 0.001-4
- Perl 5.16 rebuild
More information about the scm-commits
mailing list