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

Emmanuel Seyman <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |fedora-cvs?

--- Comment #2 from Emmanuel Seyman <[email protected]> ---
(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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to