Bug #67930 for IO-Pager: Cannot print UTF8 data through IO::Pager.

Wed May 04 01:51:03 2011 nomad [...] null.net - Ticket created

Subject:	Cannot print UTF8 data through IO::Pager.
Date:	Wed, 4 May 2011 17:50:39 +1200
To:	bug-IO-Pager [...] rt.cpan.org
From:	Mark Lawrence <nomad [...] null.net>

I cannot see that the IO::Pager::* filehandle is exposed anywhere to calling programs. This would be necessary for functions such as binmode. I have tried implementing a BINMODE method, faking open::import calls to work around this but I can't even seem to manage that. Any chance of a patch? -- Mark Lawrence

Wed May 04 09:18:49 2011 belg4mit [...] pthbb.org - Correspondence added

Subject:	Re: [rt.cpan.org #67930] Cannot print UTF8 data through IO::Pager.
Date:	Wed, 04 May 2011 09:18:35 -0400
To:	bug-IO-Pager [...] rt.cpan.org
From:	belg4mit [...] pthbb.org

Show quoted text

>I cannot see that the IO::Pager::* filehandle is exposed anywhere to >calling programs.

As shown in the synopsis, the filehandle is returned by new. Show quoted text

>I have tried implementing a BINMODE method, faking open::import calls >to work around this but I can't even seem to manage that. Any chance of >a patch?

Yes, there is no binmode method at this time, I will look into it.

Wed May 04 09:18:50 2011 The RT System itself - Status changed from 'new' to 'open'

Wed May 04 19:22:34 2011 nomad [...] null.net - Correspondence added

Subject:	Re: [rt.cpan.org #67930] Cannot print UTF8 data through IO::Pager.
Date:	Thu, 5 May 2011 11:22:03 +1200
To:	"belg4mit [...] pthbb.org via RT" <bug-IO-Pager [...] rt.cpan.org>
From:	Mark Lawrence <nomad [...] null.net>

On Wed May 04, 2011 at 09:18:50AM -0400, belg4mit@pthbb.org via RT wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=67930 > >

> >I cannot see that the IO::Pager::* filehandle is exposed anywhere to > >calling programs.

> > As shown in the synopsis, the filehandle is returned by new.

I don't think that statement is quite right, or you may have misunderstood my point. In the absence of a BINMODE method I was looking for a way to get to the *real* filehandle that is opened to the pager program so I could call binmode() myself on it. But perhaps that is really the wrong way to look at it, and my bug report should really just be about a missing BINMODE method. For the moment I found out I could run "binmode($pager->[1], ...)" as a work-around, but I do hate breaking the encapsulation. Show quoted text

> >I have tried implementing a BINMODE method, faking open::import > >calls to work around this but I can't even seem to manage that. Any > >chance of a patch?

> > Yes, there is no binmode method at this time, I will look into it.

Thanks. -- Mark Lawrence

Sun Jan 29 17:53:41 2012 jpierce [...] cpan.org - Correspondence added

Hello, Sorry for the long delay in response. This was actually fixed shortly after you filed the bug, but never got off my hard drive. There should be a release with the fix out shortly. It's simply a matter of adding support for binmode() to the desired class with: sub BINMODE{ binmode(shift()->[0], @_); } Since 2-arg binmode lets you set the encoding of a handle. Cheers!

Sun Jan 29 17:53:42 2012 jpierce [...] cpan.org - Status changed from 'open' to 'resolved'

Thu Feb 02 17:59:23 2012 nomad [...] null.net - Correspondence added

Subject:	Re: [rt.cpan.org #67930] Cannot print UTF8 data through IO::Pager.
Date:	Fri, 3 Feb 2012 09:59:02 +1100
To:	Jerrad Pierce via RT <bug-IO-Pager [...] rt.cpan.org>
From:	Mark Lawrence <nomad [...] null.net>

Hi Jerrad, Show quoted text

> Sorry for the long delay in response. This was actually fixed shortly > after you filed the bug, but never got off my hard drive. There should > be a release with the fix out shortly. It's simply a matter of adding

Great to hear it was a simple fix. Looking forward to seeing the new release. By the way, your previous release (distribution 0.06) still had only version 0.05 and 0.03 modules inside it which can lead to some confusion. Perhaps worth updating all values for the next release. Cheers, Mark. -- Mark Lawrence

Thu Feb 02 17:59:24 2012 The RT System itself - Status changed from 'resolved' to 'open'

Thu Feb 02 20:51:16 2012 belg4mit [...] pthbb.org - Correspondence added

Subject:	Re: [rt.cpan.org #67930] Cannot print UTF8 data through IO::Pager.
Date:	Thu, 02 Feb 2012 20:50:25 -0500
To:	bug-IO-Pager [...] rt.cpan.org
From:	Jerrad Pierce <belg4mit [...] pthbb.org>

Yes, IO::Pager.pm was 0.06, other files had versions matching when they were last updated.

Fri Feb 03 04:15:44 2012 nomad [...] null.net - Correspondence added

Subject:	Re: [rt.cpan.org #67930] Cannot print UTF8 data through IO::Pager.
Date:	Fri, 3 Feb 2012 20:15:14 +1100
To:	"belg4mit [...] pthbb.org via RT" <bug-IO-Pager [...] rt.cpan.org>
From:	Mark Lawrence <nomad [...] null.net>

On Thu Feb 02, 2012 at 08:51:17PM -0500, belg4mit@pthbb.org via RT wrote: Show quoted text

> > Yes, IO::Pager.pm was 0.06, other files had > versions matching when they were last updated.

Actually this link: http://search.cpan.org/~jpierce/IO-Pager-0.06/lib/IO/Pager.pm still shows the version as being 0.05, as does installing/upgrading with cpanm: mark@lifebook:$ cpanm IO::Pager IO::Pager is up to date. (0.05) Mark. -- Mark Lawrence

Fri Feb 03 21:08:31 2012 jpierce [...] cpan.org - Correspondence added

0.16 is mirroring now

Fri Feb 03 21:08:32 2012 jpierce [...] cpan.org - Status changed from 'open' to 'resolved'

Sat Feb 04 21:58:26 2012 jpierce [...] cpan.org - Severity Important added

Sat Feb 04 21:58:26 2012 jpierce [...] cpan.org - Fixed in 0.16 added

Sat Feb 04 21:58:26 2012 jpierce [...] cpan.org - Broken in 0.06 added