Spec clean-up

- drop -tests subpackage (general lack of interest in this), but include
  them as documentation for the main package
- drop redundant %{?perl_default_filter}
- don't use macros for commands
- can't find any dependency cycle so drop %{?perl_bootstrap} usage
- drop ExtUtils::MakeMaker version requirement to 6.30, actual working minimum

perl-Data-OptList.spec

@@ -1,25 +1,19 @@
 Name:           perl-Data-OptList
 Summary:        Parse and validate simple name/value option pairs
 Version:        0.107
-Release:        3%{?dist}
+Release:        4%{?dist}
 License:        GPL+ or Artistic
 Group:          Development/Libraries
-Source0:        http://search.cpan.org/CPAN/authors/id/R/RJ/RJBS/Data-OptList-%{version}.tar.gz 
 URL:            http://search.cpan.org/dist/Data-OptList/
-Requires:       perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
+Source0:        http://search.cpan.org/CPAN/authors/id/R/RJ/RJBS/Data-OptList-%{version}.tar.gz 
 BuildArch:      noarch
-
+BuildRequires:  perl(ExtUtils::MakeMaker) >= 6.30
-BuildRequires:  perl(ExtUtils::MakeMaker) >= 6.42
 BuildRequires:  perl(List::Util)
 BuildRequires:  perl(Params::Util)
 BuildRequires:  perl(Sub::Install) >= 0.921
 BuildRequires:  perl(Test::More)
-%if !%{defined perl_bootstrap}
 BuildRequires:  perl(Test::Pod)
-%endif
+Requires:       perl(:MODULE_COMPAT_%(eval "`perl -V:version`"; echo $version))
-
-%{?perl_default_filter}
-%{?perl_default_subpackage_tests}
 
 %description
 Hashes are great for storing named data, but if you want more than one entry
@@ -47,25 +41,32 @@ following a name is its value.
 %setup -q -n Data-OptList-%{version}
 
 %build
-%{__perl} Makefile.PL INSTALLDIRS=vendor
+perl Makefile.PL INSTALLDIRS=vendor
 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 \;
+find %{buildroot} -depth -type d -exec rmdir {} \; 2>/dev/null
 %{_fixperms} %{buildroot}
 
 %check
 make test RELEASE_TESTING=1
 
 %files
-%doc Changes LICENSE README
+%doc Changes LICENSE README t/
 %{perl_vendorlib}/Data/
 %{_mandir}/man3/Data::OptList.3pm*
 
 %changelog
+* Fri Jan 20 2012 Paul Howarth <paul@city-fan.org> - 0.107-4
+- drop -tests subpackage (general lack of interest in this), but include
+  them as documentation for the main package
+- drop redundant %%{?perl_default_filter}
+- don't use macros for commands
+- can't find any dependency cycle so drop %%{?perl_bootstrap} usage
+- drop ExtUtils::MakeMaker version requirement to 6.30, actual working minimum
+
 * Wed Jan 11 2012 Paul Howarth <paul@city-fan.org> - 0.107-3
 - package LICENSE file
 - run test suite even when bootstrapping, as it should still pass
@@ -85,7 +86,7 @@ make test RELEASE_TESTING=1
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_15_Mass_Rebuild
 
 * Thu Dec 16 2010 Marcela Maslanova <mmaslano@redhat.com> - 0.106-3
-- 661697 rebuild for fixing problems with vendorach/lib
+- rebuild to fix problems with vendorarch/lib (#661697)
 
 * Fri Apr 30 2010 Marcela Maslanova <mmaslano@redhat.com> - 0.106-2
 - Mass rebuild with perl-5.12.0