Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=701183
--- Comment #3 from Emmanuel Seyman emmanuel.seyman@club-internet.fr 2011-05-14 17:17:31 EDT --- (In reply to comment #2)
FIX: BuildRequire on perl(Class::Accessor::Fast) instead of perl(Class::Accessor) as Class::Accessor::Fast is used in code (RateLimit.pm:26)
Done
FIX: BuildRequire perl(DBI) because of tests (t/02simple.t:15)
Done
FIX: BuildRequire perl(File::Temp) because of tests (t/02simple.t:16)
Done
TODO: Remove obsolete BuildRoot cleaning in %install section.
Done
TODO: Remove already implicit %defattr in %files section.
Done (first I've heard of this)
TODO: Require perl(CGI::Application) explicitly as this module is extension for that class.
Done
Spec URL: http://people.parinux.org/~seyman/fedora/perl-CGI-Application-Plugin-RateLim... SRPM URL: http://people.parinux.org/~seyman/fedora/perl-CGI-Application-Plugin-RateLim...