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@xxxxxxxxxxxxxxxx> 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-RateLimit/perl-CGI-Application-Plugin-RateLimit.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-CGI-Application-Plugin-RateLimit/perl-CGI-Application-Plugin-RateLimit-1.0-2.fc14.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review