Skip Menu |

This queue is for tickets about the Mojolicious-Plugin-ConsoleLogger CPAN distribution.

Report information
The Basics
Id: 117228
Status: open
Priority: 0/
Queue: Mojolicious-Plugin-ConsoleLogger
People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.06
Fixed in: (no value)

History Show all quoted text
  Fri Aug 26 01:34:23 2016 SREZIC [...] cpan.org - Ticket created
Subject: Unescaped left brace in regex is illegal here in regex
With perl 5.25.1 and later t/logger.t fails: Unescaped left brace in regex is illegal here in regex; marked by <-- HERE in m/console\.group\("fatal"\);\s*console\.log\({ <-- HERE "json":"structure"}\);\s*console\.groupEnd\("fatal"\);/ at t/logger.t line 46. t/logger.t .. Dubious, test returned 255 (wstat 65280, 0xff00) No subtests run See also http://matrix.cpantesters.org/?dist=Mojolicious-Plugin-ConsoleLogger%200.06 for an overview of test reports.
  Tue Dec 20 12:58:00 2016 jkeenan [...] cpan.org - Correspondence added
On Fri Aug 26 01:34:23 2016, SREZIC wrote: Show quoted text
> With perl 5.25.1 and later t/logger.t fails: > > Unescaped left brace in regex is illegal here in regex; marked by <-- > HERE in m/console\.group\("fatal"\);\s*console\.log\({ <-- HERE > "json":"structure"}\);\s*console\.groupEnd\("fatal"\);/ at t/logger.t > line 46. > t/logger.t .. > Dubious, test returned 255 (wstat 65280, 0xff00) > No subtests run > > > See also http://matrix.cpantesters.org/?dist=Mojolicious-Plugin- > ConsoleLogger%200.06 for an overview of test reports.
Patch attached should address the problem. Thank you very much. Jim Keenan
Subject: Mojolicious-Plugin-ConsoleLogger-unescaped-left-brace.diff
diff -u -r t/logger.t corrected_t/logger.t --- t/logger.t 2016-12-20 12:54:47.918141831 -0500 +++ corrected_t/logger.t 2016-12-20 12:55:12.160740475 -0500 @@ -43,14 +43,14 @@ qr/console\.group\("error"\);\s*console\.log\("error"\);\s*console\.groupEnd\("error"\);/ ) ->content_like( - qr/console\.group\("fatal"\);\s*console\.log\({"json":"structure"}\);\s*console\.groupEnd\("fatal"\);/ + qr/console\.group\("fatal"\);\s*console\.log\(\{"json":"structure"}\);\s*console\.groupEnd\("fatal"\);/ ) ->content_like( qr/console\.group\("session"\)/ ) ->content_like(qr/"session":"value"/) ->content_like( - qr/console\.group\("config"\);\s*console\.log\({"config":"value"}\);\s*console\.groupEnd\("config"\);/ + qr/console\.group\("config"\);\s*console\.log\(\{"config":"value"}\);\s*console\.groupEnd\("config"\);/ ) ->content_like( qr/console\.group\("stash"\)/
  Tue Dec 20 12:58:00 2016 The RT System itself - Status changed from 'new' to 'open'
  Mon Jan 09 09:05:49 2017 jkeenan [...] cpan.org - Correspondence added
RT-Send-CC: XSAWYERX [...] cpan.org
This is a bug report for: Mojolicious-Plugin-ConsoleLogger-0.06 Due to unescaped left braces in some regular expression patterns in your code, your module won't work on 5.26.0. The CPAN Tester automated testing service cannot test your distribution on new versions of Perl until it is fixed and any modules that depend on yours will not work either. Please see: http://matrix.cpantesters.org/?dist=Mojolicious-Plugin-ConsoleLogger Here is a link to all modules that depend on yours: http://deps.cpantesters.org/depended-on-by.pl?dist=Mojolicious-Plugin-ConsoleLogger We would appreciate help in sorting this out before the new version of Perl is released in May of this year. These errors are quite easy to correct and I would be happy to lend a hand. I'm including a log file of my failure to build and install using "cpanm". Thanks! Jim Keenan
Subject: mojolicious-plugin-consolelogger.log

Message body is not shown because it is too large.