https://bugzilla.redhat.com/show_bug.cgi?id=839742 Jitka Plesnikova <jplesnik@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #5 from Jitka Plesnikova <jplesnik@xxxxxxxxxx> --- 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@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review