https://bugzilla.redhat.com/show_bug.cgi?id=751119 --- Comment #10 from Petr Šabata <psabata@xxxxxxxxxx> --- This has been rotting for too long... I'll do a fresh re-review. * I suppose the patch makes sense then. All the files it's patching are read-only, though. Consider changing their permissions before running the %patch macro. * You say you're not interested in EPEL5. You can also drop the Buildroot tag and the whole %clean section. * Line 41, where you remove empty directories, is no longer required either. Not even in EPEL. * The same applies to %defattr; it can be removed too. * That perl version constraint isn't really necessary. Simply buildrequire `perl'. * The package still doesn't build; buildrequire Judy-devel to fix this. * There are also some perl buildtime dependencies missing (used by either tests or the tested code and not guaranteed to be in the buildroot): - perl(base) - perl(blib) - perl(constant) - perl(Cwd) - perl(Data::Dumper) - perl(DynaLoader) - perl(File::Basename) - perl(File::Copy) - perl(strict) - perl(vars) - perl(warnings) * You can drop the buildtime dependency on ExtUtils::CBuilder. Leave it to Module::Build. * There's no need to explicitly require the following at runtime, rpmbuild's generators will find them automagically: - perl(Config) - perl(constant) - perl(Sub::Exporter) * Regarding the explicit provides -- well, you were right. I apparently didn't check these files didn't contain any or contained package statements different from the filenames. Therefore, the following need to be provided explicitly to satisfy reflexive dependencies (although providing all these `underscore modules' would be a better option -- feel free to): - perl(Judy::_tie) - perl(Judy::1::_impl) - perl(Judy::1::_obj) - perl(Judy::1::_tie) - perl(Judy::HS::_impl) - perl(Judy::HS::_obj) - perl(Judy::HS::_tie) - perl(Judy::L::_impl) - perl(Judy::L::_obj) - perl(Judy::L::_tie) - perl(Judy::Mem::_impl) - perl(Judy::SL::_dump) - perl(Judy::SL::_impl) - perl(Judy::SL::_obj) - perl(Judy::SL::_tie) * Don't package META.* files in %doc. They are of no use to end users. * The author tests aren't probably all that useful either but this is debatable. * I believe that's all :) -- 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@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review