[Bug 566379] Review Request: perl-Ham-Reference-QRZ - Ham::Reference::QRZ perl modul

[bugzilla at redhat.com]

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 at ioa.s.u-tokyo.ac.jp> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|nobody at fedoraproject.org    |mtasaka at ioa.s.u-tokyo.ac.jp
               Flag|                            |fedora-review?

--- Comment #1 from Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp> 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.