Skip Menu |

This queue is for tickets about the Devel-MAT CPAN distribution.

Report information
The Basics
Id: 125818
Status: resolved
Priority: 0/
Queue: Devel-MAT
People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.35
Fixed in: 0.36

History Show all quoted text
  Wed Jul 11 12:59:10 2018 SREZIC [...] cpan.org - Ticket created
Subject: s///r does not work with older perls
The test suite fails with older perls (5.12.5 and older): ... Bareword found where operator expected at /home/slaven.rezic/.cpan/build/2018071117/Devel-MAT-0.35-3/blib/lib/Devel/MAT/Tool/Count.pm line 93, near "s/^Devel::MAT::SV:://r" # Failed test 'use Devel::MAT::Tool::Count;' # at t/00use.t line 20. # Tried to use 'Devel::MAT::Tool::Count'. # Error: syntax error at /home/slaven.rezic/.cpan/build/2018071117/Devel-MAT-0.35-3/blib/lib/Devel/MAT/Tool/Count.pm line 93, near "s/^Devel::MAT::SV:://r" # Compilation failed in require at t/00use.t line 20. # BEGIN failed--compilation aborted at t/00use.t line 20. # Looks like you failed 1 test of 17. t/00use.t ................ Dubious, test returned 1 (wstat 256, 0x100) Failed 1/17 subtests ...
  Mon Jul 16 12:50:55 2018 leonerd-cpan [...] leonerd.org.uk - Correspondence added
On Wed Jul 11 12:59:10 2018, SREZIC wrote: Show quoted text
> The test suite fails with older perls (5.12.5 and older): > > ... > Bareword found where operator expected at > /home/slaven.rezic/.cpan/build/2018071117/Devel-MAT-0.35- > 3/blib/lib/Devel/MAT/Tool/Count.pm line 93, near > "s/^Devel::MAT::SV:://r"
Ahyes, oops. I'd like to start targeting more recent versions of perl in the analysis tooling, so I can use newer perl features, but right now the dumper is in the same distribution, so that would limit what versions of perl can be analysed. I think what I might do is split the dumper part out into its own distribution, so it can be more conservative and continue to target 5.8 and above, meanwhile the analysis tools can then start to target newer versions only, to be able to make use of newer features. -- Paul Evans
  Mon Jul 16 12:50:56 2018 The RT System itself - Status changed from 'new' to 'open'
  Tue Jul 17 10:23:13 2018 leonerd-cpan [...] leonerd.org.uk - Correspondence added
On Mon Jul 16 12:50:55 2018, PEVANS wrote: Show quoted text
> I think what I might do is split the dumper part out into its own > distribution, so it can be more conservative and continue to target > 5.8 and above, meanwhile the analysis tools can then start to target > newer versions only, to be able to make use of newer features.
Done, to a degree, in version 0.36 - now split between Devel-MAT and newly-independent Devel-MAT-Dumper. The latter is still 5.10+ currently but hopefully I can extend it to 5.8 -- Paul Evans
  Tue Jul 17 10:23:41 2018 leonerd-cpan [...] leonerd.org.uk - Status changed from 'open' to 'resolved'
  Tue Jul 17 10:23:41 2018 leonerd-cpan [...] leonerd.org.uk - Fixed in 0.36 added