libappindicator

Merge lp://staging/~allanlesage/libappindicator/TDD into lp://staging/libappindicator/0.5

TDD
Merge into trunk.0.5

Proposed by Allan LeSage on 2012-04-02

Status:	Merged
Approved by:	Charles Kerr on 2012-04-12
Approved revision:	233
Merged at revision:	239
Proposed branch:	lp://staging/~allanlesage/libappindicator/TDD
Merge into:	lp://staging/libappindicator/0.5
Diff against target:	201 lines (+152/-0) 5 files modified Makefile.am (+2/-0) Makefile.am.coverage (+48/-0) configure.ac (+14/-0) m4/gcov.m4 (+86/-0) src/Makefile.am (+2/-0)
To merge this branch:	bzr merge lp://staging/~allanlesage/libappindicator/TDD
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Charles Kerr (community)		2012-04-02	Approve on 2012-04-12
Review via email: mp+100429@code.staging.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.