Skip Menu |

This queue is for tickets about the Perl-MinimumVersion CPAN distribution.

Report information
The Basics
Id: 120578
Status: open
Priority: 0/
Queue: Perl-MinimumVersion

People
Owner: Nobody in particular
Requestors: KENTNL [...] cpan.org
Cc:
AdminCc:

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



Subject: Accidentally finding Perl Versions in XML....


perl -MPerl::MinimumVersion -E 'say Perl::MinimumVersion->new(\q[<link></link><description></description>])->minimum_syntax_version'   # 5.021008

perl -MPerl::MinimumVersion -E 'say Perl::MinimumVersion->new(\q[<link></link><description></descriptioo>])->minimum_syntax_version' # 5.013010

perl -MPerl::MinimumVersion -E 'say Perl::MinimumVersion->new(\q[<link>http://</link>])->minimum_syntax_version' # 5.010

Naturally, you'd say "don't do this, it hurts when I do this"... but it doesn't help when some tool decides to accidentally parse all files in a folder and silently communicate its dependencies.

I can't even explain *why* it produces this output, or _where_ these numbers come from.

 

5.21.8 came out *after* the last Perl::MinimumVersion, so its probably in some mystical place hiding from me.

I don't know what is going on here, I'm just going to avoid the problem by relocating my files into a place the bad code won't touch me :(

 




-- 
- CPAN kentnl@cpan.org
- Gentoo Perl Maintainer kentnl@gentoo.org ( perl@gentoo.org )
 

"doctor, it gives me strange results when I pass it non-perl text and ask it to parse it like it was perl."

Forensics indicates something astounding:

That the bulk of my code that is nonsense in Perl is overlooked, but the '/' compound registers as a regular expression!

and so...

/lettersn

Upgrades to 5.21.8 due to _regex rule via PPI::Regexp because of the '/n' modifier.

Other letters of the alphabet like "/d" of course expose a lower threshold of 5.13.10

Ideally, the nature of this tomfoolery would be more apparent, and Ideally, tools like Dist::Zilla::Plugin::MinimumPerl wouldn't take a carpet bomb to an entire directory and assume they're Perl files ( https://rt.cpan.org/Ticket/Display.html?id=100322 )

I'd also suggest maybe in extreme cases where the code doesn't parse Perl::MinimumVersion should be a bit more vocal about being handed garbage.

 



-- 
- CPAN kentnl@cpan.org
- Gentoo Perl Maintainer kentnl@gentoo.org ( perl@gentoo.org )