From 23ba3c45aff7c6ffe0909b19d3f708c0e94d2fae Mon Sep 17 00:00:00 2001
From: Paul Howarth <paul@city-fan.org>
Date: Wed, 16 Oct 2019 15:18:57 +0100
Subject: [PATCH] Spec tidy-up

- Use author-independent source URL
- Classify builds by usage
- Drop redundant buildroot cleaning in %install section
- Fix permissions verbosely
---
 perl-Test-Distribution.spec | 33 +++++++++++++++++++++++++--------
 1 file changed, 25 insertions(+), 8 deletions(-)

diff --git a/perl-Test-Distribution.spec b/perl-Test-Distribution.spec
index 9489d92..7fed598 100644
--- a/perl-Test-Distribution.spec
+++ b/perl-Test-Distribution.spec
@@ -1,21 +1,32 @@
 Name:		perl-Test-Distribution
 Version:	2.00
-Release:	31%{?dist}
+Release:	32%{?dist}
 Summary:	Perform tests on all modules of a distribution
 License:	GPL+ or Artistic
 URL:		https://metacpan.org/release/Test-Distribution
-Source0:	https://cpan.metacpan.org/authors/id/S/SR/SRSHAH/Test-Distribution-%{version}.tar.gz
+Source0:	https://cpan.metacpan.org/modules/by-module/Test/Test-Distribution-%{version}.tar.gz
 Patch0:		Test-Distribution-2.00-utf8.patch
 BuildArch:	noarch
+# Build
+BuildRequires:	coreutils
 BuildRequires:	perl-generators
+BuildRequires:	perl-interpreter
+BuildRequires:	perl(Module::Build)
+# Module
 BuildRequires:	perl(ExtUtils::Manifest)
 BuildRequires:	perl(File::Find::Rule) >= 0.03
-BuildRequires:	perl(Module::Build)
 BuildRequires:	perl(Module::CoreList) >= 1.93
+BuildRequires:	perl(Module::Signature)
 BuildRequires:	perl(Pod::Coverage) >= 0.17
+BuildRequires:	perl(strict)
 BuildRequires:	perl(Test::More)
 BuildRequires:	perl(Test::Pod) >= 0.95
 BuildRequires:	perl(Test::Pod::Coverage)
+BuildRequires:	perl(vars)
+BuildRequires:	perl(warnings)
+# Test Suite
+# (no additional dependencies)
+# Dependencies
 Requires:	perl(:MODULE_COMPAT_%(eval "`perl -V:version`"; echo $version))
 # these are considered "optional"; autoreq doesn't pick them up
 Requires:	perl(File::Find::Rule) >= 0.03
@@ -37,13 +48,12 @@ that they all define a $VERSION.
 %patch0
 
 %build
-perl Build.PL installdirs=vendor
+perl Build.PL --installdirs=vendor
 ./Build
 
 %install
-rm -rf %{buildroot}
-./Build install destdir=%{buildroot} create_packlist=0
-%{_fixperms} %{buildroot}
+./Build install --destdir=%{buildroot} --create_packlist=0
+%{_fixperms} -c %{buildroot}
 
 %check
 ./Build test
@@ -51,9 +61,16 @@ rm -rf %{buildroot}
 %files
 %doc Changes.pod README
 %{perl_vendorlib}/Test/
-%{_mandir}/man3/Test::Distribution.3pm*
+%{_mandir}/man3/Test::Distribution.3*
 
 %changelog
+* Wed Oct 16 2019 Paul Howarth <paul@city-fan.org> - 2.00-32
+- Spec tidy-up
+  - Use author-independent source URL
+  - Classify builds by usage
+  - Drop redundant buildroot cleaning in %%install section
+  - Fix permissions verbosely
+
 * Fri Jul 26 2019 Fedora Release Engineering <releng@fedoraproject.org> - 2.00-31
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_31_Mass_Rebuild