Skip Menu |

This queue is for tickets about the HTML-Strip CPAN distribution.

Report information
The Basics
Id: 104379
Status: resolved
Priority: 0/
Queue: HTML-Strip
People
Owner: Nobody in particular
Requestors: ntyni [...] iki.fi
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 2.09
Fixed in: 2.10

History Show all quoted text
  Tue May 12 14:37:51 2015 ntyni [...] iki.fi - Ticket created
Subject: Subroutine HTML::Strip::DESTROY redefined
On recent perls (probably since 5.15.6), using HTML::Strip generates a warning: % perl -we 'use HTML::Strip' Subroutine HTML::Strip::DESTROY redefined at /usr/lib/x86_64-linux-gnu/perl/5.20/DynaLoader.pm line 210. Apparently the diagnostics have improved, as there are indeed DESTROY methods in both Strip.pm and Strip.xs. It looks like the one Strip.pm never gets called. (This is also tracked as https://bugs.debian.org/785032 ) Thanks for your work on free software, -- Niko Tyni (Debian Perl team) ntyni@debian.org
  Sun May 24 08:28:56 2015 ntyni [...] iki.fi - Correspondence added
From: ntyni [...] iki.fi
On Tue May 12 14:37:51 2015, ntyni@iki.fi wrote: Show quoted text
> On recent perls (probably since 5.15.6), using HTML::Strip generates a > warning: > > % perl -we 'use HTML::Strip' > Subroutine HTML::Strip::DESTROY redefined at /usr/lib/x86_64-linux- > gnu/perl/5.20/DynaLoader.pm line 210. > > Apparently the diagnostics have improved, as there are indeed DESTROY > methods in both Strip.pm and Strip.xs. It looks like the one Strip.pm > never gets called.
Update: I've just patched our Debian package to disable the pure Perl version of the DESTROY method to fix the warning. This is not quite optimal as it probably leaves a small memory leak, but that's been the case earlier anyway. Please let me know if you have a better fix. Thanks for your work, -- Niko Tyni (Debian Perl group) ntyni@debian.org
  Thu Jul 16 09:18:57 2015 TIMB [...] cpan.org - Correspondence added
I just hit this too. Any chance of a new release?
  Thu Jul 16 09:18:58 2015 The RT System itself - Status changed from 'new' to 'open'
  Fri Apr 22 07:24:42 2016 KILINRAX [...] cpan.org - Correspondence added
My rt.cpan.org emails seem to be going into a black hole. Apologies for the ludicrously long delay. I've fixed the warning and just uploaded v2.10 to cpan.
  Mon Apr 25 11:37:02 2016 KILINRAX [...] cpan.org - Correspondence added
2.10 now available on CPAN.
  Mon Apr 25 11:37:15 2016 KILINRAX [...] cpan.org - Status changed from 'open' to 'resolved'
  Mon Apr 25 11:37:16 2016 KILINRAX [...] cpan.org - Fixed in 2.10 added