Skip Menu |

This queue is for tickets about the ExtUtils-MakeMaker CPAN distribution.

Report information
The Basics
Id: 83448
Status: resolved
Worked: 10 min
Priority: 0/
Queue: ExtUtils-MakeMaker

People
Owner: BINGOS [...] cpan.org
Requestors: demerphq [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: (no value)



Subject: Missing headers in ExtUtils-MakeMaker
ExtUtils::MakeMaker maintains a list of header files from core. This list is required to be up to date, yet it appears to be rarely updated. Attached patch adds some missing headers, including one which will only exist once I merge yves/hv_h_split (hv_func.h) It is the _01 build which my other patch from today is applied on top of. See https://rt.cpan.org/Public/Bug/Display.html?id=83441 In general this is a most unsatisfactory situation. What should actually happen is that Perl should distribute a file which contains this list, and EU::MM should use that. I haven't attempted to implement this idea as of yet. I am marking this as important, as I plan to merge yves/hv_h_split right after 5.17.10 is released, at which point it will be a blocker for 5.18
Subject: 0001-include-missing-headers-so-XS-modules-rebuild-when-t.patch
From d8012ff49000b391b64b3862ce174177a126e345 Mon Sep 17 00:00:00 2001 From: Yves Orton <demerphq@gmail.com> Date: Mon, 10 Dec 2012 09:43:59 +0100 Subject: [PATCH] include missing headers so XS modules rebuild when they change --- cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm | 5 ++++- cpan/ExtUtils-MakeMaker/lib/ExtUtils/MakeMaker.pm | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm b/cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm index 388192e..68d19fa 100644 --- a/cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm +++ b/cpan/ExtUtils-MakeMaker/lib/ExtUtils/MM_Unix.pm @@ -15,7 +15,7 @@ use ExtUtils::MakeMaker qw($Verbose neatvalue); # If we make $VERSION an our variable parse_version() breaks use vars qw($VERSION); -$VERSION = '6.64'; +$VERSION = '6.64_01'; $VERSION = eval $VERSION; require ExtUtils::MM_Any; @@ -2771,6 +2771,7 @@ PERL_HDRS = \ $(PERL_INC)/gv.h \ $(PERL_INC)/handy.h \ $(PERL_INC)/hv.h \ + $(PERL_INC)/hv_func.h \ $(PERL_INC)/intrpvar.h \ $(PERL_INC)/iperlsys.h \ $(PERL_INC)/keywords.h \ @@ -2789,12 +2790,14 @@ PERL_HDRS = \ $(PERL_INC)/pp_proto.h \ $(PERL_INC)/proto.h \ $(PERL_INC)/regcomp.h \ + $(PERL_INC)/regcharclass.h \ $(PERL_INC)/regexp.h \ $(PERL_INC)/regnodes.h \ $(PERL_INC)/scope.h \ $(PERL_INC)/sv.h \ $(PERL_INC)/thread.h \ $(PERL_INC)/unixish.h \ + $(PERL_INC)/utf8.h \ $(PERL_INC)/util.h $(OBJECT) : $(PERL_HDRS) diff --git a/cpan/ExtUtils-MakeMaker/lib/ExtUtils/MakeMaker.pm b/cpan/ExtUtils-MakeMaker/lib/ExtUtils/MakeMaker.pm index d040891..fbe854b 100644 --- a/cpan/ExtUtils-MakeMaker/lib/ExtUtils/MakeMaker.pm +++ b/cpan/ExtUtils-MakeMaker/lib/ExtUtils/MakeMaker.pm @@ -18,7 +18,7 @@ our @Overridable; my @Prepend_parent; my %Recognized_Att_Keys; -our $VERSION = '6.64'; +our $VERSION = '6.64_01'; $VERSION = eval $VERSION; # Emulate something resembling CVS $Revision$ -- 1.7.5.4
From: demerphq [...] gmail.com
On Tue Feb 19 11:06:58 2013, demerphq@gmail.com wrote: Show quoted text
> I am marking this as important, as I plan to merge yves/hv_h_split
right Show quoted text
> after 5.17.10 is released, at which point it will be a blocker for 5.18
See also: https://rt.perl.org/rt3/Ticket/Display.html?id=116861
I am guessing this has been resolved as the headers aren't explicitly listed in EUMM any longer.