Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: perl-Readonly https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=172677 ------- Additional Comments From rc040203@xxxxxxxxxx 2005-11-10 12:56 EST ------- I am still unable to access your site. Therefore only some comments on the spec file from the attachment: 1. FE's packaging policy recommends to ship one package per perl distribution, and I don't see many compelling reasons to make an exception in this case. I.e. I recommend to split this package into 2 packages: perl-Readonly and perl-Readonly-XS 2. Readonly wants to ship this file: /usr/lib/perl5/vendor_perl/5.8.6/benchmark.pl IMO, shipping a script of this name at this location should be avoided. I recommend to move it elsewhere (eg. %doc) or to remove it entirely. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.