[perl-POE-Component-Server-HTTP] Fix the dependencies and clean the spec a bit

Petr Šabata psabata at fedoraproject.org
Mon Feb 18 16:12:40 UTC 2013


commit 367969b8a6828beb5d302c58d244fc0cead9abfb
Author: Petr Šabata <contyk at redhat.com>
Date:   Mon Feb 18 17:12:35 2013 +0100

    Fix the dependencies and clean the spec a bit

 perl-POE-Component-Server-HTTP.spec |   62 +++++++++++++---------------------
 1 files changed, 24 insertions(+), 38 deletions(-)
---
diff --git a/perl-POE-Component-Server-HTTP.spec b/perl-POE-Component-Server-HTTP.spec
index 48b4c9f..37cb169 100644
--- a/perl-POE-Component-Server-HTTP.spec
+++ b/perl-POE-Component-Server-HTTP.spec
@@ -6,68 +6,54 @@ License:        GPL+ or Artistic
 Group:          Development/Libraries
 URL:            http://search.cpan.org/dist/POE-Component-Server-HTTP/
 Source0:        http://www.cpan.org/authors/id/R/RC/RCLAMP/POE-Component-Server-HTTP-%{version}.tar.gz
-BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
-
 BuildArch:      noarch
-Requires:       perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
-
-BuildRequires:  perl(LWP)
+Requires:       perl(:MODULE_COMPAT_%(eval "`perl -V:version`"; echo $version))
+BuildRequires:  perl
+BuildRequires:  perl(Carp)
+BuildRequires:  perl(constant)
+BuildRequires:  perl(Exporter)
+BuildRequires:  perl(ExtUtils::MakeMaker)
+BuildRequires:  perl(File::Basename)
+BuildRequires:  perl(File::Spec)
 BuildRequires:  perl(HTTP::Date)
+BuildRequires:  perl(HTTP::Request)
+BuildRequires:  perl(HTTP::Response)
 BuildRequires:  perl(HTTP::Status)
-# >= 0.3007     vvvvvvvv
-BuildRequires:  perl(POE) 
-BuildRequires:  perl(YAML)
+BuildRequires:  perl(IO::Socket::INET)
+BuildRequires:  perl(LWP::ConnCache)
+BuildRequires:  perl(LWP::UserAgent)
+BuildRequires:  perl(POE)
+BuildRequires:  perl(POE::API::Peek)
+BuildRequires:  perl(POE::Component::Server::TCP)
+BuildRequires:  perl(POE::Kernel)
+BuildRequires:  perl(Socket)
+BuildRequires:  perl(strict)
+BuildRequires:  perl(Sys::Hostname)
 BuildRequires:  perl(Test::More)
-
-# Note that this _is_ listed as a BR in Makefile.PL, but 1) is only used in
-# t/30_error.t and 2) depends on Devel::Size, which is broken on x86_64 (see
-# cpan bug at, e.g., http://rt.cpan.org/Public/Bug/Display.html?id=21037)
-# When this is addressed, I'll gladly package those two as well, for proper
-# build-time testing.
-#BuildRequires:  perl(POE::API::Peek)
-
+BuildRequires:  perl(vars)
+BuildRequires:  perl(YAML)
 
 %description
 POE::Component::Server::HTTP (PoCo::HTTPD) is a framework for building
 custom HTTP servers based on POE. It is loosely modeled on the ideas of
 apache and the mod_perl/Apache module.
 
-
 %prep
 %setup -q -n POE-Component-Server-HTTP-%{version}
 
-
 %build
-%{__perl} Makefile.PL INSTALLDIRS=vendor
+perl Makefile.PL INSTALLDIRS=vendor
 make %{?_smp_mflags}
 
-
 %install
-rm -rf %{buildroot}
-
-make pure_install PERL_INSTALL_ROOT=%{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
-# see rant in BR's, above
-rm t/30_error.t
-
-# I'm unsure as to wrapping this with _with_network_tests logic.  Should be
-# kosher, it's only doing tests on localhost.
 make test
 
-
-%clean
-rm -rf %{buildroot}
-
-
 %files
-%defattr(-,root,root,-)
 %doc Changes README test.perl
 %{perl_vendorlib}/*
 %{_mandir}/man3/*



More information about the perl-devel mailing list