Minor spec clean-up

- BR: perl(constant), perl(Data::Dumper) and perl(lib)
- Don't need to remove empty directories from the buildroot
- Drop %defattr, redundant since rpm 4.4

perl-Module-Signature.spec

@@ -1,6 +1,6 @@
 Name:           perl-Module-Signature
 Version:        0.68
-Release:        4%{?dist}
+Release:        5%{?dist}
 Summary:        CPAN signature management utilities and modules
 Group:          Development/Libraries
 License:        CC0
@@ -9,12 +9,15 @@ Source0:        http://search.cpan.org/CPAN/authors/id/F/FL/FLORA/Module-Signatu
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(id -nu)
 BuildArch:      noarch
 BuildRequires:  gnupg
+BuildRequires:  perl(constant)
+BuildRequires:  perl(Data::Dumper)
 BuildRequires:  perl(Digest::SHA)
 BuildRequires:  perl(Digest::SHA1)
 BuildRequires:  perl(Exporter)
 BuildRequires:  perl(ExtUtils::MakeMaker)
 BuildRequires:  perl(ExtUtils::Manifest)
 BuildRequires:  perl(IPC::Run)
+BuildRequires:  perl(lib)
 BuildRequires:  perl(Test::More)
 Requires:       perl(:MODULE_COMPAT_%(eval "`perl -V:version`"; echo $version))
 Requires:       gnupg
@@ -47,7 +50,6 @@ cd -
 rm -rf %{buildroot}
 make -C Module-Signature-%{version} 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
@@ -58,7 +60,6 @@ make -C Module-Signature-%{version} test TEST_SIGNATURE=1
 rm -rf %{buildroot}
 
 %files
-%defattr(-,root,root,-)
 %doc AUTHORS Changes README *.pub
 %{_bindir}/cpansign
 %{perl_vendorlib}/Module/
@@ -66,6 +67,11 @@ rm -rf %{buildroot}
 %{_mandir}/man3/Module::Signature.3pm*
 
 %changelog
+* Tue Jun 12 2012 Paul Howarth <paul@city-fan.org> - 0.68-5
+- BR: perl(constant), perl(Data::Dumper) and perl(lib)
+- Don't need to remove empty directories from the buildroot
+- Drop %%defattr, redundant since rpm 4.4
+
 * Tue Jun 12 2012 Petr Pisar <ppisar@redhat.com> - 0.68-4
 - Perl 5.16 rebuild