Merge lp:~allanlesage/unity-chromium-extension/lcov-versioning into lp:unity-chromium-extension

Proposed by Allan LeSage
Status: Needs review
Proposed branch: lp:~allanlesage/unity-chromium-extension/lcov-versioning
Merge into: lp:unity-chromium-extension
Diff against target: 41 lines (+4/-25)
1 file modified
m4/gcov.m4 (+4/-25)
To merge this branch: bzr merge lp:~allanlesage/unity-chromium-extension/lcov-versioning
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
WebApps Pending
Review via email: mp+234730@code.launchpad.net

Description of the change

Chop out that pesky lcov versioning code from m4/gcov--it was interfering with our coverage build.

Need this for lp:coverage-builder testing, as this is one of very few remaining autotools projects, want to demonstrate we can support.

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

Unmerged revisions

248. By Allan LeSage

Correct lcov versioning problem, was breaking coverage build on trusty.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'm4/gcov.m4'
2--- m4/gcov.m4 2013-12-23 01:17:40 +0000
3+++ m4/gcov.m4 2014-09-15 18:46:02 +0000
4@@ -33,33 +33,12 @@
5 AC_MSG_ERROR([ccache must be disabled when --enable-gcov option is used. You can disable ccache by setting environment variable CCACHE_DISABLE=1.])
6 fi
7
8- lcov_version_list="1.6 1.7 1.8 1.9 1.10"
9 AC_CHECK_PROG(LCOV, lcov, lcov)
10+ if test -z "$LCOV"; then
11+ AC_MSG_ERROR([Could not find lcov])
12+ fi
13+
14 AC_CHECK_PROG(GENHTML, genhtml, genhtml)
15-
16- if test "$LCOV"; then
17- AC_CACHE_CHECK([for lcov version], glib_cv_lcov_version, [
18- glib_cv_lcov_version=invalid
19- lcov_version=`$LCOV -v 2>/dev/null | $SED -e 's/^.* //'`
20- for lcov_check_version in $lcov_version_list; do
21- if test "$lcov_version" = "$lcov_check_version"; then
22- glib_cv_lcov_version="$lcov_check_version (ok)"
23- fi
24- done
25- ])
26- else
27- lcov_msg="To enable code coverage reporting you must have one of the following lcov versions installed: $lcov_version_list"
28- AC_MSG_ERROR([$lcov_msg])
29- fi
30-
31- case $glib_cv_lcov_version in
32- ""|invalid[)]
33- lcov_msg="You must have one of the following versions of lcov: $lcov_version_list (found: $lcov_version)."
34- AC_MSG_ERROR([$lcov_msg])
35- LCOV="exit 0;"
36- ;;
37- esac
38-
39 if test -z "$GENHTML"; then
40 AC_MSG_ERROR([Could not find genhtml from the lcov package])
41 fi

Subscribers

People subscribed via source and target branches

to all changes: