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(a)club-internet.fr> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag| |fedora-cvs?
--- Comment #2 from Emmanuel Seyman <emmanuel.seyman(a)club-internet.fr> 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.