Skip Menu |

This queue is for tickets about the Mail-Box CPAN distribution.

Report information
The Basics
Id: 70482
Status: resolved
Priority: 0/
Queue: Mail-Box
People
Owner: Nobody in particular
Requestors: TIMB [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Thu Aug 25 09:36:20 2011 TIMB [...] cpan.org - Ticket created
Subject: Tests fail is PERL5LIB is needed to find prereqs due to perl -T
e.g. tests/11field/10cfws.t start with #!/usr/bin/env perl -T so the PERL5LIB env var is ignored, yet in my setup PERL5LIB is needed to find prerequisite modules like Mail::Address. FYI The opinion on the #toolchain irc channel was "any tests with -t on their shebang line are broken".
  Thu Aug 25 09:41:02 2011 TIMB [...] cpan.org - Correspondence added
Adding lib => [ split /:/, $ENV{PERL5LIB} ] to the Tap::Harness->new() args is a reasonable workaround.
  Thu Aug 25 09:41:05 2011 TIMB [...] cpan.org - Status changed from 'new' to 'open'
  Thu Aug 25 09:52:58 2011 Mark [...] Overmeer.net - Correspondence added
Subject: Re: [rt.cpan.org #70482] Tests fail is PERL5LIB is needed to find prereqs due to perl -T
Date: Thu, 25 Aug 2011 15:43:48 +0200
To: Tim_Bunce via RT <bug-Mail-Box [...] rt.cpan.org>
From: Mark Overmeer <mark [...] overmeer.net>
* Tim_Bunce via RT (bug-Mail-Box@rt.cpan.org) [110825 13:36]: Show quoted text
> Thu Aug 25 09:36:20 2011: Request 70482 was acted upon. > Transaction: Ticket created by TIMB > Queue: Mail-Box > Subject: Tests fail is PERL5LIB is needed to find prereqs due to perl -T > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=70482 > > > e.g. tests/11field/10cfws.t start with #!/usr/bin/env perl -T so > the PERL5LIB env var is ignored, > yet in my setup PERL5LIB is needed to find prerequisite modules > like Mail::Address. > > FYI The opinion on the #toolchain irc channel was "any tests with -t > on their shebang line are broken".
I had the tests tainted, because I wanted to know that my module was implemented taint-safe. On many spots I do OS-access so there are many potential traps. However, it clashed with my own test-environment as well. Therefore, I have removed these "-T" in release 2.096 (november 2010) Please upgrade. -- Regards, MarkOv ------------------------------------------------------------------------ Mark Overmeer MSc MARKOV Solutions Mark@Overmeer.net solutions@overmeer.net http://Mark.Overmeer.net http://solutions.overmeer.net
  Thu Aug 25 09:53:41 2011 TIMB [...] cpan.org - Correspondence added
split /$Config{path_sep}/ would be best.
  Tue Nov 22 10:44:36 2011 MARKOV [...] cpan.org - Correspondence added
please upgrade
  Tue Nov 22 10:44:36 2011 MARKOV [...] cpan.org - Status changed from 'open' to 'resolved'