[Bug 517720] Review Request: perl-BZ-Client - A client for the Bugzilla web services API

[bugzilla at redhat.com]

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 at club-internet.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?




--- Comment #2 from Emmanuel Seyman <emmanuel.seyman at 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.