Bug 973476 - Devel::Cover support
Summary: Devel::Cover support
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Bugzilla
Classification: Community
Component: Bugzilla General
Version: 4.4
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Jeff Fearn 🐞
QA Contact: tools-bugs
URL:
Whiteboard:
: 975288 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-06-12 02:32 UTC by Jeff Fearn 🐞
Modified: 2013-10-24 03:53 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-10-24 03:53:16 UTC
Embargoed:


Attachments (Terms of Use)

Description Jeff Fearn 🐞 2013-06-12 02:32:04 UTC
Description of problem:
Add support for Devel::Cover to allow test coverage results to be tracked.

Comment 1 Jeff Fearn 🐞 2013-06-18 03:21:03 UTC
*** Bug 975288 has been marked as a duplicate of this bug. ***

Comment 3 Jason McDonald 2013-08-08 00:12:49 UTC
This change has been abandoned, as desribed in the gerrit review.

Comment 4 Jeff Fearn 🐞 2013-08-08 00:21:10 UTC
Why is this abandoned? There is no reason you can't use Devel::Cover with the solution chosen.

Comment 5 Jason McDonald 2013-08-08 07:00:53 UTC
The commit was abandoned, though it looks like two-thirds of it got into the product via different means.....

[11:30:04] <jmcdonal> simon: what do you want to do about https://code.engineering.redhat.com/gerrit/#/c/9273/ ?
[11:31:16] <simon> If it was my call, I'd abandon it.
[11:31:49] <simon> The change for t/004template.t is incorrect, I submitted the correct one upstream.
[11:32:29] <simon> The change in bugzilla.spec.in means we would never be able to submit a RHEL 5 RPM any time soon, due to missing deps.
[11:33:07] <simon> And the change for extensions/RedHat/template/en/default/pages/component-templates.html.tmpl was submitted in a different bug.

Besides those pieces, what else would need to be done?

Comment 6 Jeff Fearn 🐞 2013-08-08 07:09:12 UTC
Make the spec file pull in the deps and tweak the prove command line to set it up.

This should be close:

HARNESS_PERL_SWITCHES=-MDevel::Cover prove

Comment 7 Jason McDonald 2013-08-08 07:25:24 UTC
It appears that the dependencies aren't available for RHEL5, and there is a significant cost to rectify that.  Therefore I propose to revisit this once the production Bugzilla is on RHEL6.

Comment 8 Jeff Fearn 🐞 2013-08-08 23:47:51 UTC
All the deps are in rhel 6, it is trivial to add those packages to the rhel 5 brew root.


Note You need to log in before you can comment on or make changes to this bug.