Message ID | 1442139982-29309-1-git-send-email-francois.perrad@gadz.org |
---|---|
State | Changes Requested |
Headers | show |
Dear Francois Perrad, Thanks for this fix. See some comments below. On Sun, 13 Sep 2015 12:26:22 +0200, Francois Perrad wrote: > see http://autobuild.buildroot.net/results/502/502e6a6280451acc48e5f500e774b2bd04c7229e/ > > Signed-off-by: Francois Perrad <francois.perrad@gadz.org> > --- > package/perl/0001-hack-EUMM.patch | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > create mode 100644 package/perl/0001-hack-EUMM.patch > > diff --git a/package/perl/0001-hack-EUMM.patch b/package/perl/0001-hack-EUMM.patch > new file mode 100644 > index 0000000..3d54f47 > --- /dev/null > +++ b/package/perl/0001-hack-EUMM.patch > @@ -0,0 +1,20 @@ > +hack EUMM 7.04_01 which comes with Perl 5.22.0 > + > +with static lib build, we cannot call _find_magic_vstring(), > +which does a `require B;` and B is not a pure Perl module. > + You should add your SoB line inside the patch as well. Also, do you think this patch can be accepted upstream? Thanks, Thomas
2015-09-13 12:35 GMT+02:00 Thomas Petazzoni <thomas.petazzoni@free-electrons.com>: > Dear Francois Perrad, > > Thanks for this fix. See some comments below. > > On Sun, 13 Sep 2015 12:26:22 +0200, Francois Perrad wrote: >> see http://autobuild.buildroot.net/results/502/502e6a6280451acc48e5f500e774b2bd04c7229e/ >> >> Signed-off-by: Francois Perrad <francois.perrad@gadz.org> >> --- >> package/perl/0001-hack-EUMM.patch | 20 ++++++++++++++++++++ >> 1 file changed, 20 insertions(+) >> create mode 100644 package/perl/0001-hack-EUMM.patch >> >> diff --git a/package/perl/0001-hack-EUMM.patch b/package/perl/0001-hack-EUMM.patch >> new file mode 100644 >> index 0000000..3d54f47 >> --- /dev/null >> +++ b/package/perl/0001-hack-EUMM.patch >> @@ -0,0 +1,20 @@ >> +hack EUMM 7.04_01 which comes with Perl 5.22.0 >> + >> +with static lib build, we cannot call _find_magic_vstring(), >> +which does a `require B;` and B is not a pure Perl module. >> + > > You should add your SoB line inside the patch as well. > > Also, do you think this patch can be accepted upstream? > I've improved the patch for V2. No, I think that it will be accepted upstream, first in perl-cross and after in perl. François > Thanks, > > Thomas > -- > Thomas Petazzoni, CTO, Free Electrons > Embedded Linux, Kernel and Android engineering > http://free-electrons.com > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
diff --git a/package/perl/0001-hack-EUMM.patch b/package/perl/0001-hack-EUMM.patch new file mode 100644 index 0000000..3d54f47 --- /dev/null +++ b/package/perl/0001-hack-EUMM.patch @@ -0,0 +1,20 @@ +hack EUMM 7.04_01 which comes with Perl 5.22.0 + +with static lib build, we cannot call _find_magic_vstring(), +which does a `require B;` and B is not a pure Perl module. + +--- a/cpan/ExtUtils-MakeMaker/lib/ExtUtils/MakeMaker/version/vpp.pm ++++ b/cpan/ExtUtils-MakeMaker/lib/ExtUtils/MakeMaker/version/vpp.pm +@@ -933,11 +933,7 @@ sub _un_vstring { + if ( length($value) >= 3 && $value !~ /[._]/ + && _is_non_alphanumeric($value)) { + my $tvalue; +- if ( $] ge 5.008_001 ) { +- $tvalue = _find_magic_vstring($value); +- $value = $tvalue if length $tvalue; +- } +- elsif ( $] ge 5.006_000 ) { ++ if ( $] ge 5.006_000 ) { + $tvalue = sprintf("v%vd",$value); + if ( $tvalue =~ /^v\d+(\.\d+){2,}$/ ) { + # must be a v-string
see http://autobuild.buildroot.net/results/502/502e6a6280451acc48e5f500e774b2bd04c7229e/ Signed-off-by: Francois Perrad <francois.perrad@gadz.org> --- package/perl/0001-hack-EUMM.patch | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) create mode 100644 package/perl/0001-hack-EUMM.patch