Skip Menu |

This queue is for tickets about the Language-Farnsworth CPAN distribution.

Report information
The Basics
Id: 118125
Status: open
Priority: 0/
Queue: Language-Farnsworth

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

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



Subject: Unescaped left brace in regex is illegal here in regex
Module compilation fails with bleadperl (e.g. perl 5.25.5): # Failed test 'use Language::Farnsworth;' # at t/00-sanity.t line 4. # Tried to use 'Language::Farnsworth'. # Error: Unescaped left brace in regex is illegal here in regex; marked by <-- HERE in m/(?<!\\)(\$\w+|\${ <-- HERE [^}]+})/ at /tmpfs/.cpan-build-cpansand/2016092806/Language-Farnsworth-0.7.7-0/blib/lib/Language/Farnsworth/Evaluate.pm line 866. # Compilation failed in require at /tmpfs/.cpan-build-cpansand/2016092806/Language-Farnsworth-0.7.7-0/blib/lib/Language/Farnsworth.pm line 10. # BEGIN failed--compilation aborted at /tmpfs/.cpan-build-cpansand/2016092806/Language-Farnsworth-0.7.7-0/blib/lib/Language/Farnsworth.pm line 10. # Compilation failed in require at t/00-sanity.t line 4. # BEGIN failed--compilation aborted at t/00-sanity.t line 4.
On Wed Sep 28 17:55:52 2016, SREZIC wrote: Show quoted text
> Module compilation fails with bleadperl (e.g. perl 5.25.5): > > # Failed test 'use Language::Farnsworth;' > # at t/00-sanity.t line 4. > # Tried to use 'Language::Farnsworth'. > # Error: Unescaped left brace in regex is illegal here in regex; > marked by <-- HERE in m/(?<!\\)(\$\w+|\${ <-- HERE [^}]+})/ at > /tmpfs/.cpan-build-cpansand/2016092806/Language-Farnsworth-0.7.7- > 0/blib/lib/Language/Farnsworth/Evaluate.pm line 866. > # Compilation failed in require at /tmpfs/.cpan-build- > cpansand/2016092806/Language-Farnsworth-0.7.7- > 0/blib/lib/Language/Farnsworth.pm line 10. > # BEGIN failed--compilation aborted at /tmpfs/.cpan-build- > cpansand/2016092806/Language-Farnsworth-0.7.7- > 0/blib/lib/Language/Farnsworth.pm line 10. > # Compilation failed in require at t/00-sanity.t line 4. > # BEGIN failed--compilation aborted at t/00-sanity.t line 4.
Something like the patch attached should address the problem. Thank you very much. Jim Keenan
Subject: Language-Farnsworth-unrecognized-left-brace.diff
Only in blib/lib: auto Only in blib/lib/Language: .exists diff -u -r lib/Language/Farnsworth/Evaluate.pm blib/lib/Language/Farnsworth/Evaluate.pm --- lib/Language/Farnsworth/Evaluate.pm 2010-08-28 15:01:27.000000000 -0400 +++ blib/lib/Language/Farnsworth/Evaluate.pm 2016-12-20 12:39:37.728873773 -0500 @@ -863,7 +863,7 @@ "".$output; }; - $value =~ s/(?<!\\)(\$\w+|\${[^}]+})/$ss->($1)/eg; + $value =~ s/(?<!\\)(\$\w+|\$\{[^}]+})/$ss->($1)/eg; my $val = new Language::Farnsworth::Value::String($value); return $val; } @@ -914,4 +914,4 @@ Copyright (C) 2010 by Ryan Voots -This library is free software; It is licensed exclusively under the Artistic License version 2.0 only. \ No newline at end of file +This library is free software; It is licensed exclusively under the Artistic License version 2.0 only.