On 20 February 2013 07:40, demerphq <demer...@gmail.com> wrote: > On 20 February 2013 05:47, Craig A. Berry <craig.a.be...@gmail.com> wrote: >> On Tue, Feb 19, 2013 at 1:00 PM, Nicholas Clark <n...@ccl4.org> wrote: >>> On Tue, Feb 19, 2013 at 05:57:12PM +0100, demerphq wrote: >>> >>>> I want to fix this. >>>> >>>> My current plan is to create a new module ExtUtils::PerlHeaders and >>>> then move the list there. >>>> >>>> EU::MakeMaker can then require this module, and if it exists use it >>>> for the header list. >>>> >>>> I am curious if anyone has any better suggestions or comments? >>>> >>>> The code involved is in lib/ExtUtils/MM_Unix.pm as the sub perldepend(): >>> >>> It's also in ExtUtils::MM_VMS as perldepend(). The intent is the same. The >>> list is different. >> >> Omissions are probably maintenance failures. The one addition I can >> think of at the moment is vmsish.h. >> >>> I noticed this problem a while back. I got some way towards addressing it - >>> I added a routine Config::header_files() to give the list of header files. >>> (So that's your ExtUtils::PerlHeaders already done) >>> >>> However, at the time I didn't have access to VMS, so couldn't fix up the >>> forked code to do it in one place. >>> >>> However, since then, it's been suggested by someone (forget who, sorry) - >>> why do we need a list? Given that the directory of installed header files >>> is under our control, why not just glob it, and depend on everything that >>> is <*.h> ? >>> >>> This seems a simpler suggestion than having to maintain an explicit list. >> >> You might be thinking of: >> >> <http://www.nntp.perl.org/group/perl.perl5.porters/2012/12/msg196426.htmll> > > I have pushed a patch in the yves/hv_h_split branch which should fix > this. I plan to merge this branch to blead once 5.17.9 is out. > > Assuming it passes tests in core on VMS and the various core smoke > environments I will push a patch to my EUMM fork on github and then > file a Pull Request with the master repo (also on github). At which > point we can update EUMM to the latest. > > I consider this patch and getting it or something equivalent merged > into EUMM and perl core a 5.18 release blocker.
Patch ID is: 0b340fe025421621146aac814778d80ad1df28c2 Cheers, Yves -- perl -Mre=debug -e "/just|another|perl|hacker/"
