rpms/perl-URI/devel perl-URI.spec,1.16,1.17

Robin Norwood (rnorwood) fedora-extras-commits at redhat.com
Thu Oct 25 05:02:56 UTC 2007


Author: rnorwood

Update of /cvs/pkgs/rpms/perl-URI/devel
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv1639

Modified Files:
	perl-URI.spec 
Log Message:
fix various package review issues.


Index: perl-URI.spec
===================================================================
RCS file: /cvs/pkgs/rpms/perl-URI/devel/perl-URI.spec,v
retrieving revision 1.16
retrieving revision 1.17
diff -u -r1.16 -r1.17
--- perl-URI.spec	17 Oct 2007 04:31:09 -0000	1.16
+++ perl-URI.spec	25 Oct 2007 05:02:23 -0000	1.17
@@ -1,23 +1,22 @@
 Name:           perl-URI
 Version:        1.35
-Release: 	3.1
+Release:        4%{?dist}
 Summary:        A Perl module implementing URI parsing and manipulation
 
 Group:          Development/Libraries
 License:        GPL+ or Artistic
 URL:            http://search.cpan.org/dist/URI/
 Source0:        http://www.cpan.org/authors/id/G/GA/GAAS/URI-%{version}.tar.gz
-Source1:        filter-requires-perl-URI.sh
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
 BuildArch:      noarch
-BuildRequires:  perl >= 1:5.6.1
 BuildRequires:  perl(MIME::Base64)
 BuildRequires:  perl(ExtUtils::MakeMaker)
+# Uncomment the following line when bug #351761 is fixed and perl-Business-ISBN is included in Fedora
+# Also be sure to remove Requires filter found in prep section for perl(Business::ISBN)
 #BuildRequires:  perl(Business::ISBN)
-Requires:  perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
+Requires:       perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
 
-%define __perl_requires %{SOURCE1}
 
 %description
 This module implements the URI class. Objects of this class represent
@@ -29,10 +28,22 @@
 %setup -q -n URI-%{version}
 chmod 644 uri-test
 
+# Filter unwanted Requires:
+cat << \EOF > %{name}-req
+#!/bin/sh
+%{__perl_requires} $* |\
+  sed -e '/perl(Business::ISBN)/d'
+EOF
+
+%define __perl_requires %{_builddir}/URI-%{version}/%{name}-req
+chmod +x %{__perl_requires}
+
+
 %build
 %{__perl} Makefile.PL INSTALLDIRS=vendor
 make %{?_smp_mflags}
 
+
 %install
 rm -rf $RPM_BUILD_ROOT
 make pure_install PERL_INSTALL_ROOT=$RPM_BUILD_ROOT
@@ -40,9 +51,16 @@
 find $RPM_BUILD_ROOT -type d -depth -exec rmdir {} 2>/dev/null ';'
 chmod -R u+w $RPM_BUILD_ROOT/*
 
-%check || :
+for file in Changes; do
+  iconv -f iso-8859-1 -t utf-8 < "$file" > "${file}_"
+  mv -f "${file}_" "$file"
+done
+
+
+%check
 make test
 
+
 %clean 
 rm -rf $RPM_BUILD_ROOT
 
@@ -55,6 +73,13 @@
 
 
 %changelog
+* Thu Oct 25 2007 Robin Norwood <rnorwood at redhat.com> - 1.35-4
+- Fix various package review issues:
+- Remove redundant BR: perl
+- remove "|| :" from %%check
+- move requires filter into spec file
+- remove tabs and fix spacing
+
 * Wed Oct 17 2007 Tom "spot" Callaway <tcallawa at redhat.com> - 1.35-3.1
 - correct license tag
 - add BR: perl(ExtUtils::MakeMaker)




More information about the fedora-extras-commits mailing list