Bug #98019 for Pod-Perldoc: Pod::Perldoc supports utf8 strings partially

Thu Aug 14 16:20:13 2014 k [...] obuk.org - Ticket created

Subject:	Pod::Perldoc supports utf8 strings partially
Date:	Thu, 14 Aug 2014 23:42:50 +0900 (JST)
To:	bug-Pod-Perldoc [...] rt.cpan.org
From:	KUBO Koichi <k [...] obuk.org>

Hi, Perldoc is very convenient, and I use every day. But it may return incorrect result recently. I've tried the following commands with Pod-Perldoc-3.23. perldoc -L JA -u -f 'scalar' # ok perldoc -L JA -u -a 'SvPV' # ok perldoc -L JA -u -v '$_' # not ok perldoc -L JA -u -f 'qq' # not ok perldoc -L JA -u -q 'perl' # not ok It looks to just like not calling the binmode simply. Could you add calling binmode on the file handles as follows. binmode(PVAR, ":utf8"); # search_perlvar binmode(PERLOP, ":utf8"); # search_perlop binmode(INFAQ, ":utf8"); # search_perlfaqs POD2::JA - https://github.com/obuk/POD2-JA thanks.

Thu Aug 14 16:25:01 2014 mrallen1 [...] yahoo.com - Correspondence added

On Thu Aug 14 16:20:13 2014, k@obuk.org wrote: Show quoted text

> Could you add calling binmode on the file handles as follows. > > binmode(PVAR, ":utf8"); # search_perlvar > binmode(PERLOP, ":utf8"); # search_perlop > binmode(INFAQ, ":utf8"); # search_perlfaqs

Hi! Thanks for the very helpful bug report. Yes, I will get this fixed in a future release. (Hopefully sometime this weekend.) Thanks again, Mark

Thu Aug 14 16:25:01 2014 The RT System itself - Status changed from 'new' to 'open'

Thu Aug 14 21:24:52 2014 k [...] obuk.org - Correspondence added

Subject:	Re: [rt.cpan.org #98019] Pod::Perldoc supports utf8 strings partially
Date:	Fri, 15 Aug 2014 10:24:37 +0900 (JST)
To:	bug-Pod-Perldoc [...] rt.cpan.org
From:	KUBO Koichi <k [...] obuk.org>

Hi, I added the test script. Please use it if necessary. * https://github.com/obuk/POD2-JA/blob/master/t/pod-perldoc-98019.t thanks, Koichi

Sat Aug 16 12:52:04 2014 mrallen1 [...] yahoo.com - Correspondence added

On Thu Aug 14 21:24:52 2014, k@obuk.org wrote: Show quoted text

> Hi, > > I added the test script. Please use it if necessary. > > * https://github.com/obuk/POD2-JA/blob/master/t/pod-perldoc-98019.t

Thanks - I just put 3.23_01 on CPAN, so please give that a try and let me know how it goes. I think it should work.

Sat Aug 16 18:00:47 2014 k [...] obuk.org - Correspondence added

Subject:	Re: [rt.cpan.org #98019] Pod::Perldoc supports utf8 strings partially
Date:	Sun, 17 Aug 2014 07:00:22 +0900 (JST)
To:	bug-Pod-Perldoc [...] rt.cpan.org
From:	KUBO Koichi <k [...] obuk.org>

Show quoted text

>>>>> "Mark Allen via RT" <bug-Pod-Perldoc@rt.cpan.org> wrote:

> <URL: https://rt.cpan.org/Ticket/Display.html?id=98019 > > > Thanks - I just put 3.23_01 on CPAN, so please give that a try and > let me know how it goes.

Prove the test, and the output is as follows: $ prove -v pod-perldoc-98019.t pod-perldoc-98019.t .. ok 1 - use POD2::JA; ok 2 - perldoc -L ja -u -f scalar ok 3 - perldoc -L ja -u -a SvPV ok 4 - perldoc -L ja -u -v $_ ok 5 - perldoc -L ja -u -f qq ok 6 - perldoc -L ja -u -q perl 1..6 ok Thanks for the quick turnaround.

Mon Aug 18 23:48:18 2014 mrallen1 [...] yahoo.com - Status changed from 'open' to 'patched'

Mon Aug 18 23:48:18 2014 mrallen1 [...] yahoo.com - Fixed in 3.23_01 added

Mon Aug 18 23:48:19 2014 mrallen1 [...] yahoo.com - Broken in 3.22_02 added

Mon Aug 18 23:49:52 2014 mrallen1 [...] yahoo.com - Status changed from 'patched' to 'resolved'

Tue Aug 19 09:13:52 2014 k [...] obuk.org - Correspondence added

Subject:	Re: [rt.cpan.org #98019] Resolved: Pod::Perldoc supports utf8 strings partially
Date:	Tue, 19 Aug 2014 22:13:26 +0900 (JST)
To:	bug-Pod-Perldoc [...] rt.cpan.org
From:	KUBO Koichi <k [...] obuk.org>

Show quoted text

>>>>> "Mark Allen via RT" <bug-Pod-Perldoc@rt.cpan.org> wrote:

> <URL: https://rt.cpan.org/Ticket/Display.html?id=98019 > > > According to our records, your request has been resolved. If you have any > further questions or concerns, please respond to this message.

Hi Mark, I tested some pod files that has not been converted to utf8. And I've found the 3.23_01 that not to parse the line =encoding other than utf8 yet. Please check the patch uuencoded follows. thanks, koichi

Download Pod-Perldoc-3.23.patch.gz
application/octet-stream 1.4k

Message body not shown because it is not plain text.

Tue Aug 19 09:58:31 2014 k [...] obuk.org - Status changed from 'resolved' to 'open'

Wed Aug 20 09:16:20 2014 k [...] obuk.org - Correspondence added

Subject:	Re: [rt.cpan.org #98019] Resolved: Pod::Perldoc supports utf8 strings partially
Date:	Wed, 20 Aug 2014 22:16:03 +0900 (JST)
To:	bug-Pod-Perldoc [...] rt.cpan.org
From:	KUBO Koichi <k [...] obuk.org>

Show quoted text

>>>>>> "Mark Allen via RT" <bug-Pod-Perldoc@rt.cpan.org> wrote:

>> <URL: https://rt.cpan.org/Ticket/Display.html?id=98019 > >> >> According to our records, your request has been resolved. If you have any >> further questions or concerns, please respond to this message.

Hi, I made the commit in github instead of the patch for Pod-Perldoc-3.24. * https://github.com/obuk/Pod-Perldoc/commit/819d1cd Please discard the patch uuencoded before. thanks, Koichi

Tue Sep 09 23:37:20 2014 mrallen1 [...] yahoo.com - Correspondence added

On Wed Aug 20 09:16:20 2014, k@obuk.org wrote: Show quoted text

> I made the commit in github instead of the patch for Pod-Perldoc-3.24. > > * https://github.com/obuk/Pod-Perldoc/commit/819d1cd

Thanks for the patch. I took your idea and used it to make some functions. It's on CPAN as 3.24_01

Tue Sep 09 23:37:23 2014 mrallen1 [...] yahoo.com - Status changed from 'open' to 'patched'

Thu Jul 28 12:33:33 2016 mrallen1 [...] yahoo.com - Status changed from 'patched' to 'resolved'