Skip Menu |
 

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

Report information
The Basics
Id: 94831
Status: open
Priority: 0/
Queue: Module-Build

People
Owner: Nobody in particular
Requestors: stephen.baynes [...] smoothwall.net
Cc:
AdminCc:

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



Subject: build testcover loses exising values in HARNESS_PERL_SWITCHES
Download (untitled) / with headers
text/plain 659b
If I build testcover any existing value of the HARNESS_PERL_SWITCHES environment value is lost and replaced by "-MDevel::Cover". So for example if I am using HARNESS_PERL_SWITCHES to set -T this is lost if I build testcover, but not if I build test. The "-MDevel::Cover" should be appended to any existing HARNESS_PERL_SWITCHES value, or an alternative way of passing switched be provided. The offending line seems to be local $ENV{HARNESS_PERL_SWITCHES} = "-MDevel::Cover"; in sub ACTION_testcover in Module/Build/Base.pm This line is not changed in the latest release (0.4205) so I presume the issue is still there though I have not actually tested it.
Download (untitled) / with headers
text/plain 830b
On Thu Apr 17 08:22:28 2014, https://www.google.com/accounts/o8/id?id=AItOawk8OBCmUaR8VF4IJkvuFJCdAPwqR_w5xEk wrote: Show quoted text
> If I build testcover any existing value of the HARNESS_PERL_SWITCHES > environment value is lost and replaced by "-MDevel::Cover". So for > example if I am using HARNESS_PERL_SWITCHES to set -T this is lost if > I build testcover, but not if I build test. The "-MDevel::Cover" > should be appended to any existing HARNESS_PERL_SWITCHES value, or an > alternative way of passing switched be provided. > > The offending line seems to be > local $ENV{HARNESS_PERL_SWITCHES} = "-MDevel::Cover"; > in sub ACTION_testcover in Module/Build/Base.pm > > This line is not changed in the latest release (0.4205) so I presume > the issue is still there though I have not actually tested it.
A patch is welcome ;-) Leon
Changing the aforementioned line to: local $ENV{HARNESS_PERL_SWITCHES} = $ENV{HARNESS_PERL_SWITCHES} . " -MDevel::Cover"; seems to do the job. On Wed Jun 04 10:07:57 2014, LEONT wrote: Show quoted text
> On Thu Apr 17 08:22:28 2014, > https://www.google.com/accounts/o8/id?id=AItOawk8OBCmUaR8VF4IJkvuFJCdAPwqR_w5xEk > wrote:
> > If I build testcover any existing value of the HARNESS_PERL_SWITCHES > > environment value is lost and replaced by "-MDevel::Cover". So for > > example if I am using HARNESS_PERL_SWITCHES to set -T this is lost if > > I build testcover, but not if I build test. The "-MDevel::Cover" > > should be appended to any existing HARNESS_PERL_SWITCHES value, or an > > alternative way of passing switched be provided. > > > > The offending line seems to be > > local $ENV{HARNESS_PERL_SWITCHES} = "-MDevel::Cover"; > > in sub ACTION_testcover in Module/Build/Base.pm > > > > This line is not changed in the latest release (0.4205) so I presume > > the issue is still there though I have not actually tested it.
> > A patch is welcome ;-) > > Leon


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.