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=517720 Emmanuel Seyman <emmanuel.seyman@xxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs? --- Comment #2 from Emmanuel Seyman <emmanuel.seyman@xxxxxxxxxxxxxxxx> 2009-08-25 09:19:35 EDT --- (In reply to comment #1) > > FAIL rpmlint is silent for SRPM and for RPM. > - summary ends with dot; please delete it > - OTOH, the %description sentence should end with a dot Done. > FAIL License text is included in package > - please add LICENSE to %doc Done. > FAIL BuildRequires are proper > - please add perl(HTTP::Cookies) Done. > A suggestion: > in %files, you might replace > %{perl_vendorlib}/* > by the more specific > %{perl_vendorlib}/BZ Done. Thanks for the review. Asking for CVS. New Package CVS Request ======================= Package Name: perl-BZ-Client Short Description: A client for the Bugzilla web services API Owners: eseyman Branches: F-10 F-11 InitialCC: perl-sig -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review