Skip Menu |
 

This queue is for tickets about the Module-CPANTS-Analyse CPAN distribution.

Report information
The Basics
Id: 97601
Status: resolved
Priority: 0/
Queue: Module-CPANTS-Analyse

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

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Test::Kwalitee incorrectly reports non-use of strict in Inline::CPP
Download (untitled) / with headers
text/plain 642b
After downloading and unpacking the tarball for Inline::CPP, and then setting RELEASE_TESTING=1, running "make test" will cause Test::Kwalitee to incorrectly report that lib/Inline/CPP.pm is not using strict. This became an issue when Inline::CPP restructured its file hierarchy moving CPP.pm from top level into lib/. Oddly, make disttest results in a PASS; the test doesn't incorrectly fail in this case. So, steps to reproduce: Grab Inline::CPP and unpack the tarball. cd into distribution directory. export RELEASE_TESTING=1 perl Makefile.PL && make && make test Steps to have a clean run (ie, problem not detected): make disttest
Download (untitled) / with headers
text/plain 312b
On 2014-07-29 11:03:21, DAVIDO wrote: Show quoted text
> After downloading and unpacking the tarball for Inline::CPP, and then > setting RELEASE_TESTING=1, running "make test" will cause > Test::Kwalitee to incorrectly report that lib/Inline/CPP.pm is not > using strict.
This is a Module::CPANTS::Analyse problem; redirecting.
Hi, could you first tell us 1) the version of Module::CPANTS::Analyse, 2) the version of Module::ExtractUse, 3) the version of Test::Kwalitee, 4) and the version of Inline::CPP (repository or CPAN) ? As far as CPANTS website can tell, it should pass the test if everything is up-to-date. See also http://cpants.cpanauthors.org/dist/Inline-CPP On Wed Jul 30 03:03:21 2014, DAVIDO wrote: Show quoted text
> After downloading and unpacking the tarball for Inline::CPP, and then > setting RELEASE_TESTING=1, running "make test" will cause > Test::Kwalitee to incorrectly report that lib/Inline/CPP.pm is not > using strict. > > This became an issue when Inline::CPP restructured its file hierarchy > moving CPP.pm from top level into lib/. > > Oddly, make disttest results in a PASS; the test doesn't incorrectly > fail in this case. > > So, steps to reproduce: > > Grab Inline::CPP and unpack the tarball. > cd into distribution directory. > export RELEASE_TESTING=1 > perl Makefile.PL && make && make test > > Steps to have a clean run (ie, problem not detected): > > make disttest
Download (untitled) / with headers
text/plain 1.1k
On Tue Jul 29 11:25:01 2014, ISHIGAKI wrote: Show quoted text
> Hi, could you first tell us 1) the version of Module::CPANTS::Analyse, > 2) the version of Module::ExtractUse, 3) the version of > Test::Kwalitee, 4) and the version of Inline::CPP (repository or CPAN)
I guess I'm going to have to withdraw the report. As mysteriously as it appeared, it has heisen-disappeared. The change that triggered its appearance was moving CPP.pm from the base project directory into lib/Inline/CPP.pm. This coincided with Inline::CPP v0.51. By 0.52 it disappeared, with no (seemingly) significant change to CPP.pm that could account for the disappearance. I would love to be able to reproduce it for you, because I feel that the alignment of the celestial bodies that brought it on is probably triggering undesirable behavior (certainly at no point was 'use strict;' ever excluded from CPP.pm, and aside from Test::Kwalitee, all other distribution tests were passing.) But alas, I've stopped being able to produce it. To answer your questions, however: Inline::CPP v0.51 Module::CPANTS::Analyse v0.92 Module::ExtractUse v0.33 Test::Kwalitee v1.19 Perl version: 5.20.0 (Perlbrew). Inline v0.64. Inline::C v0.62
Download (untitled) / with headers
text/plain 1.6k
Thanks. Most probably the error happened because of (temporal) inconsistency of "provides" field in META files. I fetched Inline-CPP-0.51.tar.gz and modified the META files and let "CPP.pm" point to non-existent file, and confirmed the error was reproduced. Fixed in the master. https://github.com/cpants/Module-CPANTS-Analyse/commit/10a1b6e556da78971c5679555fbde58d5132580c On Wed Jul 30 07:38:04 2014, DAVIDO wrote: Show quoted text
> On Tue Jul 29 11:25:01 2014, ISHIGAKI wrote:
> > Hi, could you first tell us 1) the version of > > Module::CPANTS::Analyse, > > 2) the version of Module::ExtractUse, 3) the version of > > Test::Kwalitee, 4) and the version of Inline::CPP (repository or > > CPAN)
> > I guess I'm going to have to withdraw the report. As mysteriously as > it appeared, it has heisen-disappeared. > > The change that triggered its appearance was moving CPP.pm from the > base project directory into lib/Inline/CPP.pm. This coincided with > Inline::CPP v0.51. By 0.52 it disappeared, with no (seemingly) > significant change to CPP.pm that could account for the disappearance. > > I would love to be able to reproduce it for you, because I feel that > the alignment of the celestial bodies that brought it on is probably > triggering undesirable behavior (certainly at no point was 'use > strict;' ever excluded from CPP.pm, and aside from Test::Kwalitee, all > other distribution tests were passing.) > > But alas, I've stopped being able to produce it. > > To answer your questions, however: > > Inline::CPP v0.51 > Module::CPANTS::Analyse v0.92 > Module::ExtractUse v0.33 > Test::Kwalitee v1.19 > Perl version: 5.20.0 (Perlbrew). > Inline v0.64. > Inline::C v0.62
Download (untitled) / with headers
text/plain 1.8k
On Wed Jul 30 12:12:49 2014, ISHIGAKI wrote: Show quoted text
> Thanks. Most probably the error happened because of (temporal) > inconsistency of "provides" field in META files. I fetched Inline-CPP- > 0.51.tar.gz and modified the META files and let "CPP.pm" point to non- > existent file, and confirmed the error was reproduced. Fixed in the > master. > > https://github.com/cpants/Module-CPANTS- > Analyse/commit/10a1b6e556da78971c5679555fbde58d5132580c > > > On Wed Jul 30 07:38:04 2014, DAVIDO wrote:
> > On Tue Jul 29 11:25:01 2014, ISHIGAKI wrote:
> > > Hi, could you first tell us 1) the version of > > > Module::CPANTS::Analyse, > > > 2) the version of Module::ExtractUse, 3) the version of > > > Test::Kwalitee, 4) and the version of Inline::CPP (repository or > > > CPAN)
> > > > I guess I'm going to have to withdraw the report. As mysteriously as > > it appeared, it has heisen-disappeared. > > > > The change that triggered its appearance was moving CPP.pm from the > > base project directory into lib/Inline/CPP.pm. This coincided with > > Inline::CPP v0.51. By 0.52 it disappeared, with no (seemingly) > > significant change to CPP.pm that could account for the > > disappearance. > > > > I would love to be able to reproduce it for you, because I feel that > > the alignment of the celestial bodies that brought it on is probably > > triggering undesirable behavior (certainly at no point was 'use > > strict;' ever excluded from CPP.pm, and aside from Test::Kwalitee, > > all > > other distribution tests were passing.) > > > > But alas, I've stopped being able to produce it. > > > > To answer your questions, however: > > > > Inline::CPP v0.51 > > Module::CPANTS::Analyse v0.92 > > Module::ExtractUse v0.33 > > Test::Kwalitee v1.19 > > Perl version: 5.20.0 (Perlbrew). > > Inline v0.64. > > Inline::C v0.62
Closed as 0.94 is released. Thanks.


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.