On Mon, 2007-03-05 at 16:30 -0600, Tom 'spot' Callaway wrote: > Thoughts? Suggestions are welcomed. MUSTFIX: 1. There is a bug in your spec, which rendering the rpms non-installable: ... Requires: perl(ExtUtils::Embed), perl(ExtUtils-MakeMaker) ... Both these Requires are superfluous, furthermore perl(ExtUtils-MakeMaker) is wrong. The perl(ExtUtils-MakeMaker) is the show-stopper. 2. You missed to move enc2xs to devel: /usr/bin/enc2xs depends on /usr/lib/perl5/5.8.8/Encode/Makefile_PL.e2x which indirectly depends on EU::MakeMaker => move enc2xs to *-devel SHOULDFIX: 3. Several man-pages corresponding to tools having been moved from perl to perl-devel are missing from *-devel REMARK: 4. I don't understand why you moved Test::Harness. Besides that Test::Harness probably belongs into *-devel, because it's a Test:: module and therefore probably is not being used at run-time at all, I don't see any pressing requirement to move it now. I'll try to come up with a patch against cvs fixing issues 1-3, throughout today. Ralf
