Merge lp://staging/~allanlesage/indicator-bluetooth/add-coverage-tooling into lp://staging/indicator-bluetooth/13.10

Proposed by Allan LeSage
Status: Work in progress
Proposed branch: lp://staging/~allanlesage/indicator-bluetooth/add-coverage-tooling
Merge into: lp://staging/indicator-bluetooth/13.10
Diff against target: 217 lines (+158/-3)
5 files modified
Makefile.am (+2/-0)
Makefile.am.coverage (+48/-0)
configure.ac (+13/-0)
m4/gcov.m4 (+86/-0)
src/Makefile.am (+9/-3)
To merge this branch: bzr merge lp://staging/~allanlesage/indicator-bluetooth/add-coverage-tooling
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Charles Kerr (community) Needs Fixing
Review via email: mp+179299@code.staging.launchpad.net

Commit message

Add gcov coverage tooling.

Description of the change

Here comes alesage brandishing the familiar hammer! Add gcov tooling; do ./autogen.sh --enable-gcov and then make coverage-html to see test coverage, more rationale in this old blog post: http://qualityhour.wordpress.com/2012/01/29/test-coverage-tutorial-for-cc-autotools-projects/ .

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) wrote :

alesage, the familiar hammer looks fine but it's hitting the wrong anvil. Could you apply this on top of lp:~charlesk/indicator-bluetooth/gmenuify and have this depend on that merge? Otherwise we're going to get conflicts in configure.ac and src/Makefile.am.

review: Needs Fixing
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Unmerged revisions

53. By Allan LeSage

Add coverage tooling.

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