https://bugzilla.redhat.com/show_bug.cgi?id=855665 Emmanuel Seyman <emmanuel@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-cvs? --- Comment #2 from Emmanuel Seyman <emmanuel@xxxxxxxxx> --- (In reply to comment #1) > > TODO: You can remove pruning empty directories from %install section. > > TODO: Do not package ignore.txt as documentation. > TODO: Do not package META.json as documentation. > > TODO: Build-require `perl(Exporter)' (lib/Data/Validate/Type.pm:6). > TODO: Build-require `perl(base)' (lib/Data/Validate/Type.pm:6). > TODO: Build-require `perl(lib)' (t/20-hashref.t:8). I've implemented all of this. Thank you for your review and spotting all of these. Spec URL: http://people.parinux.org/~seyman/fedora/perl-Data-Validate-Type/perl-Data-Validate-Type.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-Data-Validate-Type/perl-Data-Validate-Type-1.3.0-2.fc17.src.rpm New Package SCM Request ======================= Package Name: perl-Data-Validate-Type Short Description: Public interface to Params::Util offering data type validation functions Owners: eseyman Branches: f17 f18 InitialCC: perl-sig -- 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