https://bugzilla.redhat.com/show_bug.cgi?id=1394267 --- Comment #1 from Athos Ribeiro <athoscribeiro@xxxxxxxxx> --- fedora-review complains about BuildRequires make and findutils, since those are already included in the current minimum build environment. Guidelines are vague on those since "RPM deps may change" and the definition of "basic shell scripts" is not clear. So I believe those are up to the packager. Package name: Is there a reason for this package not being named perl-News-NNTPClient? License: There is no license text other than a note in the README file and in the module itself, saying it can be redistributed or modified under the same terms of Perl itself. It would be nice to ask upstream to distribute a LICENSE file as well (not a blocker). Tests: Some tests on %check are failing, apparently, they rely on a running news server. Would you work on it, so it doesn't run these tests? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx