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 Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@xxxxxxxxxxxxxxxxx |mtasaka@xxxxxxxxxxxxxxxxxxx Flag| |fedora-review? --- Comment #1 from Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxxxx> 2010-02-22 13:07:24 EST --- Some notes: * License - For this package the license tag should be "GPL+ or Artistic". ref: https://fedoraproject.org/wiki/Packaging/Perl#License_tag * Summary - Please improve Summary (please check the description in "abstract:" item of META.yml file) * URL - We usually use http://search.cpan.org/dist/<module name> . For this package, it is http://search.cpan.org/dist/Ham-Reference-QRZ/ * SourceURL - Recent spec file generated by cpanspec seems to use http://www.cpan.org/modules/by-module/Ham/Ham-Reference-QRZ-%{version}.tar.gz * BuildArch - This package should create noarch binary rpm ("BuildArch: noarch" is needed) * Requires - Usually dependencies related to perl module based rpms are automatically detected by rpmbuild itself. For this package, "Requires: perl(LWP::UserAgent)" is automatically added by rpmbuild and is not needed to be written explicitly. - Please check the following URL: https://fedoraproject.org/wiki/Packaging/Perl#Versioned_MODULE_COMPAT__Requires -- 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 package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review