https://bugzilla.redhat.com/show_bug.cgi?id=839742

Jitka Plesnikova <jples...@redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-review?              |fedora-review+

--- Comment #5 from Jitka Plesnikova <jples...@redhat.com> ---
Changes are ok. 
Package is APPROVED.

For finding the requirement, I am going though the content of the source
tarball. 
I am looking for modules which are loaded by 'use' or 'require' for tests and
lib|scripts|...
I add only the modules which could be packed separately (e.g. lib, Test::More).

The modules which are needed only for tests, I add only to BR. 
If some module can be loaded due to any condition I also don't add it.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to