Skip Menu |

This queue is for tickets about the Lingua-LO-Romanize CPAN distribution.

Report information
The Basics
Id: 66952
Status: resolved
Priority: 0/
Queue: Lingua-LO-Romanize
People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.08
Fixed in: (no value)
Attachments
diff.out

History Show all quoted text
  Sat Mar 26 08:54:11 2011 SREZIC [...] cpan.org - Ticket created
Subject: Test fails with perl 5.13.11
The test suite fails on my freebsd box with perl 5.13.11. It works on the same system with perl 5.12.3. Here's the test output: Output from '/usr/bin/make test': PERL_DL_NONLAZY=1 /usr/home/cpansand/var/ctps/2pre5140/install/perl-167630b6ab7e291cbd4f89943a3aec8d6a1ecbfc/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t # Failed test 'use Lingua::LO::Romanize;' # at t/00-load.t line 6. # Tried to use 'Lingua::LO::Romanize'. # Error: Unrecognized character \xE0; marked by <-- HERE after <-- HERE near column 9 at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize/Syllable.pm line 34. # Compilation failed in require at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize/Types.pm line 8. # BEGIN failed--compilation aborted at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize/Types.pm line 8. # Compilation failed in require at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize.pm line 10. # BEGIN failed--compilation aborted at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize.pm line 10. # Compilation failed in require at (eval 4) line 2. # BEGIN failed--compilation aborted at (eval 4) line 2. # Failed test 'use Lingua::LO::Romanize::Types;' # at t/00-load.t line 7. # Tried to use 'Lingua::LO::Romanize::Types'. # Error: Attempt to reload Lingua/LO/Romanize/Types.pm aborted. # Compilation failed in require at (eval 288) line 2. # BEGIN failed--compilation aborted at (eval 288) line 2. # Failed test 'use Lingua::LO::Romanize::Word;' # at t/00-load.t line 8. # Tried to use 'Lingua::LO::Romanize::Word'. # Error: Attempt to reload Lingua/LO/Romanize/Types.pm aborted. # Compilation failed in require at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize/Word.pm line 9. # BEGIN failed--compilation aborted at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize/Word.pm line 9. # Compilation failed in require at (eval 289) line 2. # BEGIN failed--compilation aborted at (eval 289) line 2. # Failed test 'use Lingua::LO::Romanize::Syllable;' # at t/00-load.t line 9. # Tried to use 'Lingua::LO::Romanize::Syllable'. # Error: Attempt to reload Lingua/LO/Romanize/Syllable.pm aborted. # Compilation failed in require at (eval 290) line 2. # BEGIN failed--compilation aborted at (eval 290) line 2. Use of uninitialized value $Lingua::LO::Romanize::VERSION in concatenation (.) or string at t/00-load.t line 12. # Testing Lingua::LO::Romanize , Perl 5.013011, /usr/home/cpansand/var/ctps/2pre5140/install/perl-167630b6ab7e291cbd4f89943a3aec8d6a1ecbfc/bin/perl # Looks like you failed 4 tests of 4. t/00-load.t .......... Dubious, test returned 4 (wstat 1024, 0x400) Failed 4/4 subtests Unrecognized character \xE0; marked by <-- HERE after <-- HERE near column 9 at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize/Syllable.pm line 34. Compilation failed in require at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize/Types.pm line 8. BEGIN failed--compilation aborted at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize/Types.pm line 8. Compilation failed in require at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize.pm line 10. BEGIN failed--compilation aborted at /var/tmp/CPAN-build/Lingua-LO-Romanize-0.08-Umiqmd/blib/lib/Lingua/LO/Romanize.pm line 10. Compilation failed in require at t/01-romanization.t line 7. BEGIN failed--compilation aborted at t/01-romanization.t line 7. t/01-romanization.t .. Dubious, test returned 255 (wstat 65280, 0xff00) No subtests run t/pod-coverage.t ..... ok t/pod.t .............. ok Regards, Slaven
  Sun Mar 27 02:05:29 2011 ANDK [...] cpan.org - Correspondence added
Now also reported with bisect to rt.perl http://rt.perl.org/rt3/Ticket/Display.html?id=87110
  Sun Mar 27 02:05:29 2011 The RT System itself - Status changed from 'new' to 'open'
  Thu Mar 31 15:29:17 2011 khw [...] cpan.org - Correspondence added
From: corporate [...] khwilliamson.com
On Sun Mar 27 02:05:29 2011, ANDK wrote: Show quoted text
> Now also reported with bisect to rt.perl > > http://rt.perl.org/rt3/Ticket/Display.html?id=87110
This bug stems from Perl's starting to enforce Unicode restrictions that it didn't before. In Syllable.pm, the BGN_PCGN hash is initialized with the => operator. Perl says that the left-hand side of that operator need not be quoted, but only when that LHS is syntactically the same as an identifier. Several of the items in the initialization are not legal identifiers according to Unicode rules, the first being the LAO SEMIVOWEL SIGN LO at line 34. That character may not legally begin an identifier. Previously, Perl didn't care, but as part of our general cleanup and trying to conform more with Unicode standards, we started to. We haven't decided, as a result of this failure, if we will back out our new enforcement for Perl 5.14, or ask that this module be changed. It is never pleasant to be confronted with having to change something that used to work in the past, and then breaks as a result of somebody else's doing something; I know--I've been there more times than I care to. Therefore, I have taken the liberty of generating and attaching a patch for your module, which simply quotes all the LH sides of this initialization. With this patch applied, all tests pass. Even if Perl doesn't enforce the standard in 5.14, it seems to me to be a good idea to do something like this patch in your module. I don't believe that the LAO SEMIVOWEL SIGN LO is able to stand-alone in Lao, so it makes sense for it to be quoted. We'd be interested in your thoughts on this. Is it no big deal for you to make a change, or do you think we're off base for even asking you to consider it, or something else? Thanks, Karl Williamson
Subject: diff.out
Download diff.out
application/octet-stream 4.3k

Message body not shown because it is not plain text.

  Sun May 22 01:57:57 2011 jokke [...] cpan.org - Correspondence added
RT-Send-CC: corporate [...] khwilliamson.com
Thanks for the feedback and diff. The bug has been resolved in version 0.09.
  Sun May 22 01:57:58 2011 jokke [...] cpan.org - Status changed from 'open' to 'resolved'