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=566379

--- Comment #6 from Mamoru Tasaka <[email protected]> 2010-02-24 
13:07:03 EST ---
For -2:

* Module compat Requires for perl modules
  - You don't seem to have addressed:
   
https://fedoraproject.org/wiki/Packaging/Perl#Versioned_MODULE_COMPAT__Requires

    Please add MODULE_COMPAT Requires

* BuildRequires for tests
  - build.log shows:
--------------------------------------------------------
    82  t/00-load.t ....... ok
    83  t/pod-coverage.t .. skipped: Test::Pod::Coverage 1.08 required for
testing POD coverage
    84  t/pod.t ........... skipped: Test::Pod 1.22 required for testing POD
--------------------------------------------------------
    Please add "perl(Test::Pod::Coverage)" "perl(Test::Pod)" to
    BuildRequires to enable the latter 2 tests.

-- 
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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to