fix typo in %%summary and tidy up %%description remove redundant dependency

on perl >= 1:5.6.1 fix argument order for find with -depth drop "|| :"
    from %%check (only required for ancient rpmbuild versions) add buildreq
    perl(ExtUtils::MakeMaker)

perl-String-CRC32.spec

@@ -1,32 +1,29 @@
 Name:           perl-String-CRC32
 Version:        1.4
-Release:        2%{?dist}
+Release:        3%{?dist}
-
+Summary:        Perl interface for cyclic redundancy check generation
-Summary:        Perl interface for cyclic redundency check generation
-
 Group:          Development/Libraries
 License:        Public Domain
 URL:            http://search.cpan.org/dist/String-CRC32/
 Source0:        http://search.cpan.org/CPAN/authors/id/S/SO/SOENKE/String-CRC32-%{version}.tar.gz
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
-
+BuildRequires:  perl(ExtUtils::MakeMaker)
-BuildRequires:  perl >= 1:5.6.1
 Requires:       perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
 
 %description
-This packages provides a perl module to generate checksums from strings
+This packages provides a perl module to generate checksums from strings and
-and from files.
+from files.
 
-The checksums are the same as those calculated by ZMODEM, PKZIP,
+The checksums are the same as those calculated by ZMODEM, PKZIP, PICCHECK and
-PICCHECK and many others.
+many others.
-There's another perl module called String::CRC which allows to calculate
+
-not only 32 bit CRC numbers, but the generated sums differ from those of
+There's another perl module called String::CRC, which supports calculation of
-the programs mentioned above.
+CRC values of various widths (i.e. not just 32 bits), but the generated sums
+differ from those of the programs mentioned above.
 
 %prep
 %setup -q -n String-CRC32-%{version}
 
-
 %build
 %{__perl} Makefile.PL INSTALLDIRS=vendor
 make %{?_smp_mflags}
@@ -36,10 +33,10 @@ rm -rf $RPM_BUILD_ROOT
 make pure_install PERL_INSTALL_ROOT=$RPM_BUILD_ROOT
 find $RPM_BUILD_ROOT -type f -name .packlist -exec rm -f {} ';'
 find $RPM_BUILD_ROOT -type f -name '*.bs' -empty -exec rm -f {} ';'
-find $RPM_BUILD_ROOT -type d -depth -exec rmdir {} 2>/dev/null ';'
+find $RPM_BUILD_ROOT -depth -type d -exec rmdir {} 2>/dev/null ';'
 chmod -R u+w $RPM_BUILD_ROOT/*
 
-%check || :
+%check
 make test
 
 %clean
@@ -49,10 +46,17 @@ rm -rf $RPM_BUILD_ROOT
 %defattr(-,root,root,-)
 %doc README
 %{perl_vendorarch}/String/
-%{perl_vendorarch}/auto/String
+%{perl_vendorarch}/auto/String/
 %{_mandir}/man3/*.3*
 
 %changelog
+* Mon Aug 13 2007 Paul Howarth <paul@city-fan.org> 1.4-3
+- fix typo in %%summary and tidy up %%description
+- remove redundant dependency on perl >= 1:5.6.1
+- fix argument order for find with -depth
+- drop "|| :" from %%check (only required for ancient rpmbuild versions)
+- add buildreq perl(ExtUtils::MakeMaker)
+
 * Wed Aug 02 2006 Warren Togami <wtogami@redhat.com> 1.4-2
 - bump