On Wed, Jun 04, 2008 at 06:52:17PM +0200, Bastian Blank wrote: > On Fri, May 30, 2008 at 08:11:00PM +0300, Niko Tyni wrote: > > Note to self: don't be so sure next time. > > This patch from the upstream ticket is enough to make the whole SVK test > > suite pass for me. There's not much point in working around the bug in > > PerlIO::via::dynamic anymore, so closing the cloned bug. > > Any timeframe for a fix?
The patch isn't applied upstream yet, probably because there were concerns about it creating a memory leak. Meanwhile, PerlIO-via-dynamic 0.13 has been released with this: <http://search.cpan.org/diff?from=PerlIO-via-dynamic-0.12&to=PerlIO-via-dynamic-0.13> + if ($] == 5.010000 && ref($_[-1]) eq 'GLOB') { + # This is to workaround a core bug in perl 5.10.0, see + # http://rt.perl.org/rt3/Public/Bug/Display.html?id=54934 + require Internals; + Internals::SetRefCount($_[-1], Internals::GetRefCount($_[-1])+1); + } which would certainly create a memory leak if used as is with a locally fixed 5.10.0. Possibly the best option is to keep our Perl 5.10.0 unfixed for now and update libperlio-via-dynamic-perl to 0.13. Bastian, please let me know what you think. -- Niko Tyni [EMAIL PROTECTED] -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
