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

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



"Tom 'spot' Callaway" <tcallawa redhat com> writes:

> On Mon, 2007-03-05 at 10:42 -0600, Tom 'spot' Callaway wrote:
>
>> Well, as Joe pointed out (when he wasn't name-calling), CPAN does depend
>> on ExtUtils::MakeMaker.
>> 
>> So, we can do the following:
>> 
>> * Move ExtUtils::MakeMaker to its own package. Move CPAN to its own
>> package. Have the CPAN package depend on ExtUtils::MakeMaker, have the
>> ExtUtils::MakeMaker package depend on perl-devel.
>> 
>> In functionality, this brings us back to where we began, except that
>> now, default installs (just perl) will not get CPAN.
>> 
>> * Move ExtUtils::MakeMaker and CPAN to perl-devel. Again, default
>> installs (just perl) won't get CPAN.
>> 
>> * The third option is to move config.h back into perl, and document this
>> as an exception case.
>
> CPAN isn't the only thing:
>
> [spot localhost perl-5.8.8]$ grep -r "require ExtUtils::MakeMaker" *
> lib/CPAN.pm:        require ExtUtils::MakeMaker;
> lib/CPAN.pm:    require ExtUtils::MakeMaker;
> lib/ExtUtils/Embed.pm:#require ExtUtils::MakeMaker;
> lib/ExtUtils/Embed.pm:    require ExtUtils::MakeMaker;
> lib/ExtUtils/t/backwards.t:require ExtUtils::MakeMaker;
> lib/ExtUtils/MM.pm:require ExtUtils::MakeMaker;
> utils/perlbug.PL:        require ExtUtils::MakeMaker;
>
> so, we'd need to handle ExtUtils::Embed and perlbug too. :/
>
> perlbug is really the hardest one. I think we really want that to be in
> the base perl package.
>
> The "third option" above, is looking more and more like the cleanest fix
> to me. Thoughts?

It might be.  I can see putting MakeMaker and CPAN into perl-devel, but
perlbug really seems to me to belong in the main perl package.

-RN

-- 
Robin Norwood
Red Hat, Inc.

"The Sage does nothing, yet nothing remains undone."
-Lao Tzu, Te Tao Ching


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