Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Text-Markdown CPAN distribution.

Report information
The Basics
Id: 37964
Status: open
Priority: 0/
Queue: Text-Markdown

People
Owner: Nobody in particular
Requestors: okan [...] demirmen.com
Cc:
AdminCc:

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



Subject: Wide character in print ...
Date: Sun, 27 Jul 2008 16:11:49 -0400
To: bug-Text-Markdown [...] rt.cpan.org
From: Okan Demirmen <okan [...] demirmen.com>
hi! while running the test suite with 1.0.21, i am now seeing these warnings: Wide character in print at /usr/obj/ports/amd64/p5-Text-Markdown-1.0.21/Text-Markdown-1.0.21/t/20fulldocs-text-multimarkdown.t line 26. in t/24fulldocs-pythonmarkdown2-tm and t/29fulldocs-maruku-unittest while the tests are in the "unexpected" category, the following patch is but one way to "fix" the warnings. of course, there are others; and admittedly, i'm not familiar with the test suite anyway. cheers, okan --- t/20fulldocs-text-multimarkdown.t.orig Sun Jul 27 14:54:02 2008 +++ t/20fulldocs-text-multimarkdown.t Sun Jul 27 14:55:11 2008 @@ -4,6 +4,7 @@ use Test::More; use FindBin qw($Bin); use List::MoreUtils qw(uniq); use File::Slurp qw(slurp); +use Encode; our $TIDY = 0; @@ -23,7 +24,7 @@ if (!$@) { return; } print "=" x 80 . "\nDIFFERENCES: + = processed version from .text, - = template from .html\n"; - print Text::Diff::diff(\$expected => \$got, { STYLE => "Unified" }) . "\n"; + print encode("utf-8", Text::Diff::diff(\$expected => \$got, { STYLE => "Unified" }) . "\n"); fail($testname); }; }
Subject: Re: [rt.cpan.org #37964] Wide character in print ...
Date: Sun, 27 Jul 2008 21:28:09 +0100
To: bug-Text-Markdown [...] rt.cpan.org
From: Tomas Doran <bobtfish [...] bobtfish.net>
On 27 Jul 2008, at 21:12, okan@demirmen.com via RT wrote: Show quoted text
> > while the tests are in the "unexpected" category, the following > patch is > but one way to "fix" the warnings. of course, there are others; and > admittedly, i'm not familiar with the test suite anyway. >
Thanks for the patch, applied as r504 :) I'd noticed them, but hadn't got around to fixing the warning as it only happened in the test suite ;) Cheers t0m
Subject: Re: [rt.cpan.org #37964] Wide character in print ...
Date: Sun, 27 Jul 2008 16:55:51 -0400
To: Tomas Doran via RT <bug-Text-Markdown [...] rt.cpan.org>
From: Okan Demirmen <okan [...] demirmen.com>
On Sun 2008.07.27 at 16:28 -0400, Tomas Doran via RT wrote: Show quoted text
> <URL: http://rt.cpan.org/Ticket/Display.html?id=37964 > > > > On 27 Jul 2008, at 21:12, okan@demirmen.com via RT wrote:
> > > > while the tests are in the "unexpected" category, the following > > patch is > > but one way to "fix" the warnings. of course, there are others; and > > admittedly, i'm not familiar with the test suite anyway. > >
> > Thanks for the patch, applied as r504 :) I'd noticed them, but hadn't > got around to fixing the warning as it only happened in the test > suite ;)
great - thanks!