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)

History Show all quoted text
  Thu Apr 17 08:22:28 2014 https://www.google.com/accounts/o8/id?id=AItOawk8OBCmUaR8VF4IJkvuFJCdAPwqR_w5xEk - Ticket created
Subject: build testcover loses exising values in HARNESS_PERL_SWITCHES
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.
  Wed Jun 04 10:07:57 2014 LEONT [...] cpan.org - Correspondence added
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
  Wed Jun 04 10:07:57 2014 The RT System itself - Status changed from 'new' to 'open'
  Wed Jun 04 12:48:46 2014 LEONT [...] cpan.org - Severity Normal changed to Unimportant
  Thu Jul 17 09:47:10 2014 https://www.google.com/accounts/o8/id?id=AItOawk8OBCmUaR8VF4IJkvuFJCdAPwqR_w5xEk - Correspondence added
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