Bug #95431 for Perl-Tags: Testing with the wrong perl

Wed May 07 02:15:07 2014 ANDK [...] cpan.org - Ticket created

Subject:

Testing with the wrong perl

In the report my smoker produced: http://www.cpantesters.org/cpan/report/41758176 in the intro one can see that the testing perl was 5.19.8. But further down where a test complains that it can't find Module::Locate, the INC path is given with 5.18.2 which is the system perl. Somebody has switched to the system perl at that point in time. HTH&&Thanks, Disclaimer: this was discovered by statistical analysis, I'm not speaking as a user)

Wed May 07 04:27:01 2014 hakim.cassimally [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #95431] Testing with the wrong perl
Date:	Wed, 7 May 2014 09:26:51 +0100
To:	bug-Perl-Tags [...] rt.cpan.org
From:	Hakim C <hakim.cassimally [...] gmail.com>

Thanks. Is that likely to be just the shebang lines on the tests? they're cargo culted to #!/usr/bin/perl, which would obviously be wrong -- but then again, other tests have the same line. Will take a look... On 7 May 2014 07:15, Andreas Koenig via RT <bug-Perl-Tags@rt.cpan.org>wrote: Show quoted text

> Wed May 07 02:15:07 2014: Request 95431 was acted upon. > Transaction: Ticket created by ANDK > Queue: Perl-Tags > Subject: Testing with the wrong perl > Broken in: 0.30 > Severity: (no value) > Owner: Nobody > Requestors: ANDK@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=95431 > > > > In the report my smoker produced: > http://www.cpantesters.org/cpan/report/41758176 > > in the intro one can see that the testing perl was 5.19.8. But further > down where a test complains that it can't find Module::Locate, the INC path > is given with 5.18.2 which is the system perl. Somebody has switched to the > system perl at that point in time. > > HTH&&Thanks, > Disclaimer: this was discovered by statistical analysis, I'm not speaking > as a user) >

Wed May 07 04:27:01 2014 The RT System itself - Status changed from 'new' to 'open'

Wed May 07 12:11:09 2014 ether [...] cpan.org - Correspondence added

Shebang lines are unnecessary on tests unless you're setting a flag like -T or -w (and there are other ways of setting those states, too). Note that shebang lines are *not* rewritten in tests like they are for scripts (as they are not installed).

Sun May 11 08:15:30 2014 osfameron [...] cpan.org - Correspondence added

On Wed May 07 12:11:09 2014, ETHER wrote: Show quoted text

> Shebang lines are unnecessary on tests unless you're setting a flag > like -T or -w (and there are other ways of setting those states, too). > Note that shebang lines are *not* rewritten in tests like they are for > scripts (as they are not installed).

Ah, I didn't know about shebang script rewriting even... I've removed shebangs from tests in https://github.com/osfameron/perl-tags/commit/4b60f2499446176b659a14d9773936c9a1fe49f3 will see if that resolves issues at next release.

Sun May 11 08:15:31 2014 osfameron [...] cpan.org - Status changed from 'open' to 'patched'

Sun May 18 17:36:19 2014 osfameron [...] cpan.org - Correspondence added

On Wed May 07 02:15:07 2014, ANDK wrote: Show quoted text

> In the report my smoker produced: > http://www.cpantesters.org/cpan/report/41758176 > > in the intro one can see that the testing perl was 5.19.8. But further > down where a test complains that it can't find Module::Locate, the INC > path is given with 5.18.2 which is the system perl. Somebody has > switched to the system perl at that point in time. > > HTH&&Thanks, > Disclaimer: this was discovered by statistical analysis, I'm not > speaking as a user)

Sun May 18 17:36:20 2014 osfameron [...] cpan.org - Status changed from 'patched' to 'resolved'

Sun May 18 17:36:20 2014 osfameron [...] cpan.org - Fixed in 0.31 added