Skip Menu |

This queue is for tickets about the String-Format CPAN distribution.

Report information
The Basics
Id: 124147
Status: resolved
Priority: 0/
Queue: String-Format
People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 1.17
Fixed in: 1.17_50

History Show all quoted text
  Sat Jan 20 09:38:40 2018 SREZIC [...] cpan.org - Ticket created
Subject: Unescaped left brace in regex is deprecated here
With perl 5.27.8 there are new warnings while running the test suite: ... Unescaped left brace in regex is deprecated here (and will be fatal in Perl 5.32), passed through in regex; marked by <-- HERE in m/ (% # leading '%' (-)? # left-align, rather than right (\d*)? # (optional) minimum field width (?:\.(\d*))? # (optional) maximum field width ({ <-- HERE .*?})? # (optional) stuff inside (\S) # actual format character )/ at /home/eserte/.cpan/build/2018012015/String-Format-1.17-0/blib/lib/String/Format.pm line 77. t/01load.t ........... ok ... (etc.) ... The String-Format test suite still passes, but modules depending on String::Format may fail if using something like Test::NoWarnings.
  Sat Jan 20 10:46:13 2018 SREZIC [...] cpan.org - Dependency by ticket #124150 added
  Thu Feb 22 16:28:40 2018 SREZIC [...] cpan.org - Correspondence added
On 2018-01-20 09:38:40, SREZIC wrote: Show quoted text
> With perl 5.27.8 there are new warnings while running the test suite: > > ... > Unescaped left brace in regex is deprecated here (and will be fatal in > Perl 5.32), passed through in regex; marked by <-- HERE in m/ > (% # leading '%' > (-)? # left-align, rather than right > (\d*)? # (optional) minimum field width > (?:\.(\d*))? # (optional) maximum field width > ({ <-- HERE .*?})? # (optional) stuff inside > (\S) # actual format character > )/ at /home/eserte/.cpan/build/2018012015/String-Format- > 1.17-0/blib/lib/String/Format.pm line 77. > t/01load.t ........... ok > ... (etc.) ... > > > The String-Format test suite still passes, but modules depending on > String::Format may fail if using something like Test::NoWarnings.
A list of modules depending on String::Format and suffering from the problem (i.e. the test suite fails): - THALJEF/Perl-Critic-More-1.003.tar.gz - KRYDE/Perl-Critic-Pulp-96.tar.gz - HOLMLUND/Religion-Bible-Regex-Builder-0.99.tar.gz - AUBERTG/App-GitHooks-Plugin-DetectCommitNoVerify-v1.0.3.tar.gz - AUBERTG/App-GitHooks-Plugin-ForceRegularUpdate-v1.0.5.tar.gz - AUBERTG/App-GitHooks-Plugin-MatchBranchTicketID-v1.0.2.tar.gz - AUBERTG/App-GitHooks-Plugin-RequireCommitMessage-v1.2.0.tar.gz - AUBERTG/App-GitHooks-Plugin-RequireTicketID-v1.1.0.tar.gz
  Mon Feb 26 10:52:39 2018 SREZIC [...] cpan.org - Correspondence added
On 2018-01-20 09:38:40, SREZIC wrote: Show quoted text
> With perl 5.27.8 there are new warnings while running the test suite: > > ... > Unescaped left brace in regex is deprecated here (and will be fatal in > Perl 5.32), passed through in regex; marked by <-- HERE in m/ > (% # leading '%' > (-)? # left-align, rather than right > (\d*)? # (optional) minimum field width > (?:\.(\d*))? # (optional) maximum field width > ({ <-- HERE .*?})? # (optional) stuff inside > (\S) # actual format character > )/ at /home/eserte/.cpan/build/2018012015/String-Format- > 1.17-0/blib/lib/String/Format.pm line 77. > t/01load.t ........... ok > ... (etc.) ... > > > The String-Format test suite still passes, but modules depending on > String::Format may fail if using something like Test::NoWarnings.
A patch is available at: http://cpan.cpantesters.org/authors/id/S/SR/SREZIC/patches/String-Format-1.17-RT124147.patch CPAN.pm users may use the following distroprefs file for automatic patching: https://github.com/eserte/srezic-cpan-distroprefs/blob/master/String-Format.yml
  Sat Mar 03 10:42:27 2018 SREZIC [...] cpan.org - Status changed from 'new' to 'patched'
  Sun Mar 04 04:14:04 2018 SREZIC [...] cpan.org - Status changed from 'patched' to 'resolved'
  Sun Mar 04 04:14:04 2018 SREZIC [...] cpan.org - Fixed in 1.17_50 added