Re^2: Very Odd Issue When Using pp to Create an .exe File Including Date::Calc

Wow, well done swl, I'm glad someone dug in to sniff out where/what the issue was (I seriously doubt I'm up to the task at my capability level, I'm just one of those Perl "lusers" :-) ). I'd give you all my ++ for the day on this if I could.

Just another Perl hooker - But the purist monks keep telling me I should do it for love, not money.

Comment on Re^2: Very Odd Issue When Using pp to Create an .exe File Including Date::Calc Download Code

Replies are listed 'Best First'.
Re^3: Very Odd Issue When Using pp to Create an .exe File Including Date::Calc by swl (Parson) on Jun 28, 2017 at 23:50 UTC
No worries perldigious. As haukex noted, Module::ScanDeps is a regexp based parser so it's perhaps not surprising that it misses some things when parsing perl code. That said, I had a quick look at the Module::ScanDeps code and it's probably in scan_line where it does an early return while processing semicolon separated chunks and it finds a chunk where a perl version >= 5.9.5 is being used (currently line 812). When it finds a matching chunk it returns feature.pm and thus stops processing the rest of the line. It should probably add feature.pm to `%found` and call `next` instead. There looks to be a similar issue in the following block where it handles pragmas. If I get a chance I'll submit a pull request to the github repo, but others are welcome to beat me to it as it won't be today.	[reply] [d/l] [select]
Re^4: Very Odd Issue When Using pp to Create an .exe File Including Date::Calc by swl (Parson) on Jul 03, 2017 at 21:54 UTC
GitHub pull request has been submitted and merged. https://github.com/rschupp/Module-ScanDeps/pull/2	[reply]


The stupid question is the question not asked
	PerlMonks