Merge lp://qastaging/~allanlesage/overlay-scrollbar/TDD into lp://qastaging/overlay-scrollbar

Proposed by Allan LeSage
Status: Needs review
Proposed branch: lp://qastaging/~allanlesage/overlay-scrollbar/TDD
Merge into: lp://qastaging/overlay-scrollbar
Diff against target: 219 lines (+158/-4)
5 files modified
Makefile.am (+2/-0)
Makefile.am.coverage (+48/-0)
build/gcov.m4 (+86/-0)
configure.ac (+11/-0)
os/Makefile.am (+11/-4)
To merge this branch: bzr merge lp://qastaging/~allanlesage/overlay-scrollbar/TDD
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
jenkins (community) continuous-integration Approve
Charles Kerr Pending
Review via email: mp+100427@code.qastaging.launchpad.net

Description of the change

Adding gcov targets to autotools build for code-coverage reporting. For more information, see this blog post: http://qualityhour.wordpress.com/2012/01/29/test-coverage-tutorial-for-cc-autotools-projects/ . To compile with coverage tooling, ./autogen.sh --enable-gcov, then make coverage-html . Note that you'll need lcov to autoconf. Also note that you'll get an error on make if you have no tests, as there are no coverage artifacts to generate the html report. For review, please pay special attention to flags added in the project's makefiles.

To post a comment you must log in.
Revision history for this message
jenkins (martin-mrazik+qa) wrote :

PASSED: Continuous integration, rev:338
http://s-jenkins:8080/job/ayatana-scrollbar-ci/2/

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

Unmerged revisions

338. By Allan LeSage

Added gcov 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