Skip Menu |
 

This queue is for tickets about the Perl-Tags CPAN distribution.

Report information
The Basics
Id: 95431
Status: resolved
Priority: 0/
Queue: Perl-Tags

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

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



Subject: Testing with the wrong perl
Download (untitled) / with headers
text/plain 439b
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)
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) >
Download (untitled) / with headers
text/plain 247b
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).
Download (untitled) / with headers
text/plain 529b
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.
Download (untitled) / with headers
text/plain 505b
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)


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.