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



--- Comment #5 from Xavier Bachelot <xav...@bachelot.org> ---
The package looks good, there are only a couple mostly cosmetic issues :
- blank line at top of spec.
- it would be clearer to not mix Requires and BuildRequires. Move the only
Requires line below the last BuildRequires line.
- the %defattr line in the %files section is not needed for anything but EL5.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=iPXU3DegH9&a=cc_unsubscribe
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/perl-devel

Reply via email to