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



--- Comment #2 from Petr Ĺ abata <psab...@redhat.com> ---
License tag is incorrect.  The code is a mix of Perl, public domain and BSD
without advertising.  I suggest you use "(GPL+ or Artistic) and Public Domain
and BSD".

It's a little unclear whether Carp is really needed.  Tests don't directly
touch lib/Math/UInt64.pm, however, it could still be loaded from the XS code,
hmm.  I guess you can keep it :)

Missing BRs:
 - Exporter
 - Storable (optional)

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to