[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Bug 230608] missing config.h in latest -14



On Wed, 2007-03-07 at 08:14 +0100, Ralf Corsepius wrote:
> 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)

Yep. Robin caught that one today too. Sorry bout that. :/

> 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

OK.

> SHOULDFIX:
> 3. Several man-pages corresponding to tools having been moved from 
> perl to perl-devel are missing from *-devel

OK.

> 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 moved it because it seemed an obvious candidate for -devel. Also, when
I was separating packages into their own packages, -CPAN needed a much
newer Test::Harness.

What about the other Test:: items? Do you think they are safe to move to
-devel? Or should we just put all of Test:: in base for now? 

~spot


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]