https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20185 --- Comment #9 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to M. Tompsett from comment #6)
Running the following to confirm comment #4: for i in `find . -type f -name "*.p[lm]"`; do perl -wc $i; done
or simply `find . -name "*.pl" -exec perl -wc {} \;` yes. (In reply to M. Tompsett from comment #7)
After getting Catmandu stuff better installed, and hacking softlinks in the path to VerboseWarnings.pm and TmplTokenizer.pm, most problems go away. These two patches are fine. However, what about the horrors of files with multiple packages: C4/Barcodes.pm and OpenILS/QueryParser.pm?
The rewrite of these modules is for somewhere else (and somebody else :)) (In reply to M. Tompsett from comment #8)
If we are already tweaking the line, why not the new syntax? CGI->new?
Feel free to open a bug report to replace all the occurrence of "new CGI" if you think it is useful/major. The variable scoping problem is critical under plack. -- You are receiving this mail because: You are watching all bug changes.