dee

Merge lp://qastaging/~fginther/dee/add-code-coverage into lp://qastaging/dee

  1. add-code-coverage
  2. Merge into trunk
Proposed by Francis Ginther
Status: Merged
Approved by: Michal Hruby
Approved revision: 396
Merged at revision: 395
Proposed branch: lp://qastaging/~fginther/dee/add-code-coverage
Merge into: lp://qastaging/dee
Diff against target: 211 lines (+156/-2)
5 files modified
Makefile.am (+4/-0)
Makefile.am.coverage (+48/-0)
build/autotools/gcov.m4 (+89/-0)
configure.ac (+11/-0)
src/Makefile.am (+4/-2)
To merge this branch: bzr merge lp://qastaging/~fginther/dee/add-code-coverage
Reviewer Review Type Date Requested Status
Michal Hruby (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+139586@code.qastaging.launchpad.net

Commit message

Add code coverage reporting with coverage-html and coverage-gcovr targets.

Coverage reporting can be enabled with --enable-gcov.

Description of the change

Add code coverage reporting with coverage-html and coverage-gcovr targets.

Coverage reporting can be enabled with --enable-gcov.

Testing:
 - Built coverage targets and verified results.
 - Built in pbuilder chroot with hooks to enable gcovr report.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:395
http://jenkins.qa.ubuntu.com/job/dee-mbs-ci/3/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-mbs-ci/./build=pbuilder,distribution=quantal,flavor=amd64/3/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-mbs-ci/./build=pbuilder,distribution=quantal,flavor=i386/3/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/dee-mbs-ci/3//rebuild/?

review: Approve (continuous-integration)
lp://qastaging/~fginther/dee/add-code-coverage updated
396. By Francis Ginther

Fixed building for non-srcdir builds and cleaned up a warning message.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:396
http://jenkins.qa.ubuntu.com/job/dee-mbs-ci/4/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-mbs-ci/./build=pbuilder,distribution=quantal,flavor=amd64/4/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dee-mbs-ci/./build=pbuilder,distribution=quantal,flavor=i386/4/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/dee-mbs-ci/4//rebuild/?

review: Approve (continuous-integration)
Revision history for this message
Francis Ginther (fginther) wrote :

I've fixed the issues with non-srcdir builds:

# mkdir builddir;cd builddir; ../autogen.sh; make

Retested with local builds, pbuilder and jenkins (the -ci run above includes coverage data).

Revision history for this message
Michal Hruby (mhr3) wrote :

Seems to work fine now (minus some warnings about double PHONY declaration)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: