Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Locale-Maketext CPAN distribution.

Report information
The Basics
Id: 47316
Status: resolved
Priority: 0/
Queue: Locale-Maketext

People
Owner: TODDR [...] cpan.org
Requestors: dmuey [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 1.15_01



Subject: [1.14 target - item #2] Lexicon object support
Here is a patch that: - replaces the old Guts[Loader] logic cludge with a much simpler and efficient - version bump to 1.13_84 - change log entry As you can see it is very unobtrusive and should work fine with existing uses of Maketext. 1) debug when utf8 is available multivac:Locale-Maketext-1.13 dmuey$ perl -Ilib -e 'sub Locale::Maketext::DEBUG {1};require Locale::Maketext;' utf8 on for _compile() multivac:Locale-Maketext-1.13 dmuey$ 2) make utf8 return 0 instead of 1 3) debug when utf8 is available multivac:Locale-Maketext-1.13 dmuey$ perl -Ilib -e 'sub Locale::Maketext::DEBUG {1};require Locale::Maketext;' utf8 not available for _compile() () utf8.pm did not return a true value at lib/Locale/Maketext.pm line 495. multivac:Locale-Maketext-1.13 dmuey$
Subject: guts_perfomance.patch

Message body is not shown because it is too large.

the patch was missing oneimportant element, this line: use Locale::Maketext::GutsLoader; needs removed :)
Also missing in patch: The files removed from MANIFEST: -lib/Locale/Maketext/Guts.pm -lib/Locale/Maketext/GutsLoader.pm need removed from tree/tarball
Subject: [1.14 target - item #2] Guts.pm behavior improvement
To make it easier to review and implement I've put all changes into a single small patch. This is a complete w/ Changelog, POD, tests, and version 1.13_83 -> 1.13_84

Message body is not shown because it is too large.

Subject: [1.14 target - item #2] Guts.pm behavior improvement
On Thu Jul 30 14:36:39 2009, DMUEY wrote: Show quoted text
> To make it easier to review and implement I've put all changes into a > single small patch. > > This is a complete w/ Changelog, POD, tests, and version 1.13_83 ->
1.13_84 Show quoted text
>
A ready-to-upload-via-pause tarball of said patch
Download Locale-Maketext-1.13_84.tar.gz
application/x-gzip 47.9k

Message body not shown because it is not plain text.

From: toddr [...] null.net
forked perl blead on github - http://github.com/toddr/perl/tree/toddr/maketext-locale will be submitting to P5P post 5.12.0 freeze per advice from jesse Commit: http://github.com/toddr/perl/commit/ad3e49d61724630e34390e30c3fa 92c0f21d26fa
Commit 87d86da5f7830848364c3d8df2f53c8ddee4414a has been accepted by upstream (blead). This will be made available in the next CPAN release.
p5p commit message commit 87d86da5f7830848364c3d8df2f53c8ddee4414a Author: Todd Rinaldo <toddr@cpanel.net> Date: Thu Jul 15 14:24:07 2010 -0400 Clean up older Locale::Maketext internals to no longer do crazy string eval tricks.
1.15_02 is the RC for 1.16. No issues so far. closing ticket.