Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=607873 --- Comment #6 from Jos Vos <jos@xxxxxx> 2010-10-12 08:39:21 EDT --- (In reply to comment #2) > 1) The perl libraries should be in the OTRS* namespace, not the Kernel* > namespace. > 2) Since the perl libraries have APIs, they should probably live in > %perl_vendorlib Agreed, but for this we need OTRS to change it (I assume you don't want to apply these major changes yourself as a patch?). > 3) A separate -doc subpackage is required for the many example scripts > 4) The dummy perl requires and perl provides macros should go, and the > Kernel/cpan-lib/* directory should be removed entirely. Ideally, yes, but as long as we can "hide" the internal cpan-lib with a find-provides/requires trick, I can live with it. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- 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