Specify all dependencies

perl-PPI.spec

@@ -1,6 +1,13 @@
+# PPI::XSAccessor is experimental
+%if 0%{?rhel} >= 7
+%bcond_with XSAccessor
+%else
+%bcond_without XSAccessor
+%endif
+
 Name:           perl-PPI
 Version:        1.215
-Release:        7%{?dist}
+Release:        8%{?dist}
 Summary:        Parse, Analyze and Manipulate Perl
 Group:          Development/Libraries
 License:        GPL+ or Artistic
@@ -8,21 +15,29 @@ URL:            http://search.cpan.org/dist/PPI/
 Source0:        http://www.cpan.org/authors/id/A/AD/ADAMK/PPI-%{version}.tar.gz
 Patch0:         PPI-1.215-UTF8.patch
 BuildArch:      noarch
-BuildRequires:  perl(ExtUtils::MakeMaker)
-BuildRequires:  perl(List::Util) >= 1.20
-BuildRequires:  perl(Storable) >= 2.17
+BuildRequires:  perl(Carp)
 BuildRequires:  perl(Class::Inspector) => 1.22
 BuildRequires:  perl(Clone) >= 0.30
+BuildRequires:  perl(constant)
+BuildRequires:  perl(Cwd)
 BuildRequires:  perl(Digest::MD5) >= 2.35
-BuildRequires:  perl(File::Remove) >= 1.42
+BuildRequires:  perl(Exporter)
+BuildRequires:  perl(ExtUtils::MakeMaker)
+BuildRequires:  perl(File::Path)
+BuildRequires:  perl(File::Spec) >= 0.84
+BuildRequires:  perl(File::Spec::Functions)
 BuildRequires:  perl(IO::String) >= 1.07
+BuildRequires:  perl(File::Remove) >= 1.42
+BuildRequires:  perl(List::Util) >= 1.20
 BuildRequires:  perl(List::MoreUtils) >= 0.16
 BuildRequires:  perl(Params::Util) >= 1.00
-BuildRequires:  perl(Task::Weaken)
+BuildRequires:  perl(Scalar::Util)
+BuildRequires:  perl(Storable) >= 2.17
 BuildRequires:  perl(Test::More) >= 0.86
 BuildRequires:  perl(Test::NoWarnings) >= 0.084
 BuildRequires:  perl(Test::Object) >= 0.07
 BuildRequires:  perl(Test::SubCalls) >= 1.07
+BuildRequires:  perl(Task::Weaken)
 # Circular dependencies in release tests, so don't do them when bootstrapping:
 # Perl::MinimumVersion -> PPI
 %if 0%{!?perl_bootstrap:1}
@@ -35,6 +50,11 @@ BuildRequires:  perl(Test::MinimumVersion) >= 0.101080
 BuildRequires:  perl(Test::Pod) >= 1.00
 %endif
 Requires:       perl(:MODULE_COMPAT_%(eval "`perl -V:version`"; echo $version))
+%if %{with XSAccessor}
+Requires:       perl(Class::XSAccessor)
+%endif
+# Run-require Task::Weaken, see Changes for more details.
+Requires:       perl(Task::Weaken)
 
 # Filter out redundant unversioned provides
 %global __provides_exclude ^perl\\(PPI::.+\\)$
@@ -49,6 +69,10 @@ Parse, analyze and manipulate Perl (without perl).
 # Re-code docs as UTF-8
 %patch0 -p1
 
+%if %{without XSAccessor}
+rm lib/PPI/XSAccessor.pm
+sed -i '/^lib\/PPI\/XSAccessor\.pm$/d' MANIFEST
+%endif
 
 %build
 perl Makefile.PL INSTALLDIRS=vendor
@@ -77,6 +101,9 @@ make test TEST_FILES="xt/*.t" RELEASE_TESTING=1
 
 
 %changelog
+* Thu Aug 16 2012 Petr Pisar <ppisar@redhat.com> - 1.215-8
+- Specify all dependencies
+
 * Fri Jul 20 2012 Fedora Release Engineering <rel-eng@lists.fedoraproject.org> - 1.215-7
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_18_Mass_Rebuild