pdl icon indicating copy to clipboard operation
pdl copied to clipboard

Change in perl blead causes test failures in PDL

Open jkeenan opened this issue 1 year ago • 4 comments

BBC report: https://github.com/Perl/perl5/issues/22354

Failure has been bisected to https://github.com/Perl/perl5/commit/42cf75238c3687c1b1fad9263614036f7059c2dc, which was committed to blead on Jun 24 2024.

Sample CPANtesters failure report: http://www.cpantesters.org/cpan/report/f2acc314-35cb-11ef-989b-e9500d8dafec

Failure is in t/01-pptest.t, line 592.

jkeenan avatar Jun 30 '24 19:06 jkeenan

So in https://github.com/Perl/perl5/commit/2f4409bf707ec51ce63fd97c981a2c9e08b97241 I aligned the behavior of input maps in ExtUtils::ParseXS with that of output maps. I had no idea PDL would contain a copy of the old input behavior (for both types of maps, which was always wrong for output ones). This patch would bring everything in line again

diff --git Basic/Gen/PP.pm Basic/Gen/PP.pm
index 98f2d5d6..b32a1569 100644
--- Basic/Gen/PP.pm
+++ Basic/Gen/PP.pm
@@ -1032,7 +1032,7 @@ sub typemap_eval { # lifted from ExtUtils::ParseXS::Eval, ignoring eg $ALIAS
   my ($var, $type, $num, $init, $pname, $arg, $ntype, $argoff, $subtype)
     = @$varhash{qw(var type num init pname arg ntype argoff subtype)};
   my $ALIAS;
-  my $rv = eval qq("$code");
+  my $rv = eval qq(qq\a$code\a);
   die $@ if $@;
   $rv;
 }

Leont avatar Jun 30 '24 20:06 Leont

So what does this mean for PDL? Are you asking me to change the code? Will it work on older EU:PXS? Whose life is better as a result of this "alignment"? When is the new EU:PXS going to be released on CPAN?

mohawk2 avatar Jun 30 '24 21:06 mohawk2

On 6/30/24 17:04, mohawk2 wrote:

So what does this mean for PDL? Are you asking me to change the code? Will it work on older EU:PXS? Whose life is better as a result of this "alignment"? When is the new EU:PXS going to be released on CPAN?

At this point I'm just the messenger here; I don't have definitive answers to all your questions. At this point I'd simply advise following the discussion in the perl GH issue to which I linked.

jkeenan avatar Jun 30 '24 22:06 jkeenan

At this point I'm just the messenger here; I don't have definitive answers to all your questions. At this point I'd simply advise following the discussion in the perl GH issue to which I linked.

Fair enough! I will do so.

mohawk2 avatar Jun 30 '24 23:06 mohawk2