commit 5b0705dd3c1a048c3b3b7db1976957ec109204c5
Author: Paul Howarth <p...@city-fan.org>
Date:   Tue Aug 28 12:26:36 2012 +0100

    Spec clean-up
    
    - BR: perl(base), perl(constant) and perl(lib)
    - Install to vendor directories
    - Drop %defattr, redundant since rpm 4.4
    - Don't need to remove empty directories from the buildroot

 perl-JSON-PP.spec |   18 ++++++++++++------
 1 files changed, 12 insertions(+), 6 deletions(-)
---
diff --git a/perl-JSON-PP.spec b/perl-JSON-PP.spec
index 9e40c6d..496f052 100644
--- a/perl-JSON-PP.spec
+++ b/perl-JSON-PP.spec
@@ -1,6 +1,6 @@
 Name:          perl-JSON-PP
 Version:       2.27200
-Release:       240%{?dist}
+Release:       241%{?dist}
 Summary:       JSON::XS compatible pure-Perl module
 License:       GPL+ or Artistic
 Group:         Development/Libraries
@@ -8,12 +8,14 @@ URL:          http://search.cpan.org/dist/CPAN-Meta-YAML/
 Source0:       
http://search.cpan.org/CPAN/authors/id/M/MA/MAKAMAKA/JSON-PP-%{version}.tar.gz
 BuildRoot:     %{_tmppath}/%{name}-%{version}-%{release}-root-%(id -nu)
 BuildArch:     noarch
-BuildRequires: perl(B)
+BuildRequires: perl(base)
 BuildRequires: perl(Carp)
+BuildRequires: perl(constant)
 BuildRequires: perl(Data::Dumper)
 BuildRequires: perl(Exporter)
 BuildRequires: perl(ExtUtils::MakeMaker)
 BuildRequires: perl(Getopt::Long)
+BuildRequires: perl(lib)
 BuildRequires: perl(Test::More)
 BuildRequires: perl(Tie::IxHash)
 Requires:      perl(:MODULE_COMPAT_%(eval "`perl -V:version`"; echo $version))
@@ -30,14 +32,13 @@ JSON::PP is a pure-Perl module and is compatible with 
JSON::XS.
 %setup -q -n JSON-PP-%{version}
 
 %build
-perl Makefile.PL INSTALLDIRS=perl
+perl Makefile.PL INSTALLDIRS=vendor
 make %{?_smp_mflags}
 
 %install
 rm -rf %{buildroot}
 make 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
@@ -47,15 +48,20 @@ make test
 rm -rf %{buildroot}
 
 %files
-%defattr(-,root,root,-)
 %doc Changes README
 %{_bindir}/json_pp
-%{perl_privlib}/JSON/
+%{perl_vendorlib}/JSON/
 %{_mandir}/man1/json_pp.1*
 %{_mandir}/man3/JSON::PP.3pm*
 %{_mandir}/man3/JSON::PP::Boolean.3pm*
 
 %changelog
+* Tue Aug 28 2012 Paul Howarth <p...@city-fan.org> - 2.27200-241
+- BR: perl(base), perl(constant) and perl(lib)
+- Install to vendor directories
+- Drop %%defattr, redundant since rpm 4.4
+- Don't need to remove empty directories from the buildroot
+
 * Fri Aug 17 2012 Petr Pisar <ppi...@redhat.com> - 2.27200-240
 - Increase release to replace perl sub-package (bug #848961)
 
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/perl-devel

Reply via email to