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. Yves -- perl -Mre=debug -e "/just|another|perl|hacker/"
