Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Devel-Cover CPAN distribution.

Report information
The Basics
Id: 12410
Status: resolved
Priority: 0/
Queue: Devel-Cover

People
Owner: Nobody in particular
Requestors: jeffrey.wren [...] paradigm-works.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.52
Fixed in: (no value)



Subject: perl module referenced from multiple paths not merged.
Distribution Name: Devel-Cover-0.53.tar.gz Perl Version: 5.6.1 OS: Linux I am not using the "t" test environment. I have a home grown test environment that accesses a perl module via a couple different paths. For example: /my_path/tests/../lib/Utils.pm /my_path/bin/../Utils.pm After running my suite of tests I run the "cover" script and combine all of the coverage databases. During this process the following message is displayed: Devel::Cover: merging data for /my_path/tests/../lib/Utils.pm into /my_path/bin/../lib/Utils.pm However, once I view the html results it is clear that the data did not merge. I believe I found the problem. The following line in the DB.pm module: Db.pm::line 639 my $sc = $st->$get($file); Should be changed to
Subject: perl module referenced from multiple paths not merged.
Distribution Name: Devel-Cover-0.53.tar.gz Perl Version: 5.6.1 OS: Linux I am not using the "t" test environment. I have a home grown test environment that accesses a perl module via a couple different paths. For example: /my_path/tests/../lib/Utils.pm /my_path/bin/../Utils.pm After running my suite of tests I run the "cover" script and combine all of the coverage databases. During this process the following message is displayed: Devel::Cover: merging data for /my_path/tests/../lib/Utils.pm into /my_path/bin/../lib/Utils.pm However, once I view the html results it is clear that the data did not merge. I believe I found the problem. The following line in the DB.pm module: Db.pm::line 639 my $sc = $st->$get($file); Should be changed to
From: jeffrey.wren [...] paradigm-works.com
[guest - Mon Apr 25 16:15:33 2005]: Show quoted text
> Distribution Name: Devel-Cover-0.53.tar.gz > Perl Version: 5.6.1 > OS: Linux > > I am not using the "t" test environment. I have a home grown test > environment that accesses a perl module via a couple different paths. > For example: > /my_path/tests/../lib/Utils.pm > /my_path/bin/../Utils.pm > > After running my suite of tests I run the "cover" script and combine > all of the coverage databases. During this process the following > message is displayed: > > Devel::Cover: merging data for /my_path/tests/../lib/Utils.pm into > /my_path/bin/../lib/Utils.pm > > However, once I view the html results it is clear that the data did > not merge. I believe I found the problem. The following line in the > DB.pm module: > > Db.pm::line 639 > my $sc = $st->$get($file); > > Should be changed to
Sorry: Somehow hit the send button: The line should be changed to my $sc = $st->$get($digests{$digest}); This appeared to fix my problem. Thanks, Jeff
Date: Fri, 23 Sep 2005 00:57:06 +0200
From: Paul Johnson <paul [...] pjcj.net>
To: Guest via RT <bug-Devel-Cover [...] rt.cpan.org>
Subject: Re: [cpan #12410] perl module referenced from multiple paths not merged.
RT-Send-Cc:
On Mon, Apr 25, 2005 at 04:21:25PM -0400, Guest via RT wrote: Show quoted text
> > However, once I view the html results it is clear that the data did > > not merge. I believe I found the problem. The following line in the > > DB.pm module: > > > > Db.pm::line 639 > > my $sc = $st->$get($file); > > > > Should be changed to
> > > Sorry: Somehow hit the send button: The line should be changed to > my $sc = $st->$get($digests{$digest}); > > This appeared to fix my problem.
Thanks very much. This fix will be in the next release. -- Paul Johnson - paul@pjcj.net http://www.pjcj.net