Bug #54121 for Module-ExtractUse: Yields empty elements

Fri Jan 29 09:38:28 2010 leonerd-cpan [...] leonerd.org.uk - Ticket created

Subject:

Yields empty elements

A fair number of my modules are failing one of the CPANTS tests; prereq_matches_use. It is suspected this is because of a bug in Module::ExtractUse. From #perl / Freenode: (14:10) <bigpresh> LeoNerd: Module::ExtractUse is used to extract a list of modules used; it does attempt to strip out POD first (14:11) <LeoNerd> bigpresh: It's just that there's a page giving details, and it doesn't list any.. eg.: http://cpants.perl.org/dist/kwalitee/IO-Async-Loop-Ppoll#prereq_matches_use (14:11) <LeoNerd> Notice the list is empty (14:12) <bigpresh> LeoNerd: Oh yeah - I didn't notice that ... (14:16) <bigpresh> Although, actually, it returned a list of 4 modules being used, the first of which is an empty string... (14:16) <bigpresh> That could be confusing the rest of the CPANTS code ... (14:21) <bigpresh> LeoNerd: Just for reference, http://scsys.co.uk:8002/38693 this paste: DB<1> use Class::ByOS; DB<2> use Module::ExtractUse; DB<3> $meu = Module::ExtractUse->new; DB<4> $meu->extract_use($INC{'Class/ByOS.pm'}); DB<5> x $meu->array; 0 '' 1 'warnings' 2 'strict' 3 'Exporter' Perhaps this initial empty value is confusing the CPANTS testing code? -- Paul Evans

Fri Jan 29 09:51:56 2010 davidp [...] preshweb.co.uk - Correspondence added

On Fri Jan 29 09:38:28 2010, PEVANS wrote: [...] Show quoted text

> DB<4> $meu->extract_use($INC{'Class/ByOS.pm'}); > > DB<5> x $meu->array; > 0 '' > 1 'warnings' > 2 'strict' > 3 'Exporter' > > > Perhaps this initial empty value is confusing the CPANTS testing code?

Just to add a little extra info, I tested with Class::ByOS and Net::Async::IRC, both modules which fail the CPANTS prereq_matches_use test, and both returned an empty element; I also tested with String::Expand, which passes the prereq_matches_use test, and there was no empty element for that, so it would seem that this empty element returned by Module::ExtractUse is causing those prereq_matches_use failures.

Fri Jan 29 09:51:57 2010 The RT System itself - Status changed from 'new' to 'open'

Tue Feb 02 07:34:22 2010 davidp [...] preshweb.co.uk - Cc BIGPRESH added

Fri May 31 01:23:47 2013 ishigaki [...] cpan.org - Correspondence added

Hi, this issue should have been fixed in Module::ExtractUse 0.30 (I tested Module::ExtractUse 0.30 with both Class::ByOS and Net::Async::IRC, and got no empty elements). If you still encounter the same problem with another .pm file, please let us know. On Fri Jan 29 23:51:56 2010, BIGPRESH wrote: Show quoted text

> On Fri Jan 29 09:38:28 2010, PEVANS wrote: > [...]

> > DB<4> $meu->extract_use($INC{'Class/ByOS.pm'}); > > > > DB<5> x $meu->array; > > 0 '' > > 1 'warnings' > > 2 'strict' > > 3 'Exporter' > > > > > > Perhaps this initial empty value is confusing the CPANTS testing code?

> > Just to add a little extra info, I tested with Class::ByOS and > Net::Async::IRC, both modules which fail the CPANTS prereq_matches_use > test, and both returned an empty element; I also tested with > String::Expand, which passes the prereq_matches_use test, and there was > no empty element for that, so it would seem that this empty element > returned by Module::ExtractUse is causing those prereq_matches_use failures. >

Fri May 31 01:23:48 2013 ishigaki [...] cpan.org - Status changed from 'open' to 'resolved'