Bug #101043 for Perl-MinimumVersion: Not detecting XS functions added in recent releases

Sat Dec 20 15:35:21 2014 ether [...] cpan.org - Ticket created

Subject:

Not detecting XS functions added in recent releases

Cond::Expr uses the perl API parse_fullexpr(), which was added in 5.14.0 -- see L<perl5140delta/New-Parsing-Functions> -- but this is not detected.

Sat Nov 18 12:59:38 2017 DROLSKY [...] cpan.org - Correspondence added

Does this module even handle XS? AFAICT it just uses PPI, which apparently tries to parse the XS as Perl and comes up with some strange result. It seems like this module should be explicitly ignoring XS code.

Sat Nov 18 12:59:39 2017 The RT System itself - Status changed from 'new' to 'open'

Wed Sep 05 11:11:17 2018 TODDR [...] cpan.org - Correspondence added

On Sat Nov 18 12:59:38 2017, DROLSKY wrote: Show quoted text

> Does this module even handle XS? AFAICT it just uses PPI, which > apparently tries to parse the XS as Perl and comes up with some > strange result. It seems like this module should be explicitly > ignoring XS code.

Couldn't you just have a list of strings to search for in the XS code and if you see them, report back 5.14.0 (or whatever) as the minimum version? I realize this leaves things open to corner cases but I bet it gets 99.x % of the cases.