From 8c000d1a21c71c7acd88aa5ff72ef8be53e646e7 Mon Sep 17 00:00:00 2001
From: Paul Howarth <[email protected]>
Date: Tue, 19 Apr 2016 10:56:02 +0100
Subject: Fix FTBFS due to missing buildreq perl-devel

- Classify buildreqs by usage
- Simplify find commands using -empty and -delete
- Don't need to prune empty directories from the buildroot
- Don't use macros for commands
- Make %files list more explicit
---
 perl-B-Hooks-OP-Check.spec | 63 ++++++++++++++++++++++++++++++++--------------
 1 file changed, 44 insertions(+), 19 deletions(-)

diff --git a/perl-B-Hooks-OP-Check.spec b/perl-B-Hooks-OP-Check.spec
index 29cf4ca..75c2266 100644
--- a/perl-B-Hooks-OP-Check.spec
+++ b/perl-B-Hooks-OP-Check.spec
@@ -1,24 +1,44 @@
 Name:           perl-B-Hooks-OP-Check
 Summary:        Wrap OP check callbacks
 Version:        0.19
-Release:        14%{?dist}
+Release:        15%{?dist}
 License:        GPL+ or Artistic
 Group:          Development/Libraries
 Source0:        
http://search.cpan.org/CPAN/authors/id/Z/ZE/ZEFRAM/B-Hooks-OP-Check-%{version}.tar.gz
 URL:            http://search.cpan.org/dist/B-Hooks-OP-Check
-Requires:   perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo 
$version))
-
+# Module Build
+BuildRequires:  coreutils
+BuildRequires:  findutils
+BuildRequires:  make
+BuildRequires:  perl
+BuildRequires:  perl-devel
 BuildRequires:  perl(ExtUtils::Depends) >= 0.302
+# Dependencies of bundled Module::Install
+BuildRequires:  perl(Config)
+BuildRequires:  perl(Cwd)
+BuildRequires:  perl(ExtUtils::Command)
 BuildRequires:  perl(ExtUtils::MakeMaker) >= 6.42
+BuildRequires:  perl(ExtUtils::Manifest)
+BuildRequires:  perl(ExtUtils::MM_Unix)
+BuildRequires:  perl(Fcntl)
+BuildRequires:  perl(File::Find)
+BuildRequires:  perl(File::Path)
+BuildRequires:  perl(File::Spec)
+BuildRequires:  perl(FileHandle)
+BuildRequires:  perl(FindBin)
+BuildRequires:  perl(JSON)
+BuildRequires:  perl(Parse::CPAN::Meta)
+BuildRequires:  perl(vars)
+BuildRequires:  perl(YAML::Tiny)
+# Module Runtime
+BuildRequires:  perl(DynaLoader)
 BuildRequires:  perl(parent)
+BuildRequires:  perl(strict)
+BuildRequires:  perl(warnings)
+# Test Suite
 BuildRequires:  perl(Test::More)
-
-Requires:       perl(parent)
-
-# obsolete/provide old tests subpackage
-# can be removed during F19 development cycle
-Obsoletes:      %{name}-tests < 0.19-3
-Provides:       %{name}-tests = %{version}-%{release}
+# Dependencies
+Requires:       perl(:MODULE_COMPAT_%(eval "`perl -V:version`"; echo $version))
 
 %{?perl_default_filter}
 
@@ -30,27 +50,32 @@ of 'PL_check'.
 %setup -q -n B-Hooks-OP-Check-%{version}
 
 %build
-%{__perl} Makefile.PL INSTALLDIRS=vendor OPTIMIZE="%{optflags}"
+perl Makefile.PL INSTALLDIRS=vendor OPTIMIZE="%{optflags}"
 make %{?_smp_mflags}
 
 %install
 make pure_install DESTDIR=%{buildroot}
-find %{buildroot} -type f -name .packlist -exec rm -f {} ';'
-find %{buildroot} -type f -name '*.bs' -a -size 0 -exec rm -f {} ';'
-find %{buildroot} -depth -type d -exec rmdir {} 2>/dev/null ';'
-
-%{_fixperms} %{buildroot}/*
+find %{buildroot} -type f -name .packlist -delete
+find %{buildroot} -type f -name '*.bs' -a -empty -delete
+%{_fixperms} %{buildroot}
 
 %check
 make test
 
 %files
 %doc Changes README t/
-%{perl_vendorarch}/*
-%exclude %dir %{perl_vendorarch}/auto
-%{_mandir}/man3/*.3*
+%{perl_vendorarch}/auto/B/
+%{perl_vendorarch}/B/
+%{_mandir}/man3/B::Hooks::OP::Check.3*
 
 %changelog
+* Tue Apr 19 2016 Paul Howarth <[email protected]> - 0.19-15
+- Classify buildreqs by usage
+- Simplify find commands using -empty and -delete
+- Don't need to prune empty directories from the buildroot
+- Don't use macros for commands
+- Make %%files list more explicit
+
 * Thu Feb 04 2016 Fedora Release Engineering <[email protected]> - 
0.19-14
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_24_Mass_Rebuild
 
-- 
cgit v0.12


        
http://pkgs.fedoraproject.org/cgit/perl-B-Hooks-OP-Check.git/commit/?h=master&id=8c000d1a21c71c7acd88aa5ff72ef8be53e646e7
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
[email protected]
http://lists.fedoraproject.org/admin/lists/[email protected]

Reply via email to