Skip Menu |
 

This queue is for tickets about the IO-Pager CPAN distribution.

Report information
The Basics
Id: 82351
Status: resolved
Priority: 0/
Queue: IO-Pager

People
Owner: Nobody in particular
Requestors: dwheeler [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: 0.30
Fixed in: 0.31



Download (untitled) / with headers
text/plain 579b
The switch to IO::Handle in v0.30 is great, except that the handle returned does not have a file associated with it! This patch fixes it: --- a/lib/IO/Pager/Unbuffered.pm +++ b/lib/IO/Pager/Unbuffered.pm @@ -16,7 +16,7 @@ sub new(;$) { # [FH], procedural return $_[1] if defined($_[2]) && $_[2] eq 'procedural'; #...fall back to IO::Handle for transparent OO programming - eval "require IO::Handle" && return new IO::Handle; + eval 'require IO::Handle' && return IO::Handle->new_from_fd(fileno $_[1], 'w'); } $!=$@, return 0 if $@ =~ 'pipe';
Subject: Re: [rt.cpan.org #82351]
Date: Mon, 31 Dec 2012 15:36:57 -0500
To: bug-IO-Pager [...] rt.cpan.org
From: Jerrad Pierce <belg4mit [...] pthbb.org>
Download (untitled) / with headers
text/plain 146b
Thanks, I'd actually played around with that, but did not use it for some reason that I have since forgotten. Perhaps it failed one of the tests?
Subject: Re: [rt.cpan.org #82351]
Date: Mon, 31 Dec 2012 13:00:46 -0800
To: bug-IO-Pager [...] rt.cpan.org
From: "David E. Wheeler" <dwheeler [...] cpan.org>
Download (untitled) / with headers
text/plain 105b
All tests pass for me with that change. It could probably use a regression test before applying. David
Heya, any chance of a release soon with a fix for this? Thanks! David


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.