Measure code coverage only when gcov option is given

Change-Id: I9b4e2abfbacfc96e51bdf4d4cd48dd32c1c699bf
Signed-off-by: Yunmi Ha <yunmi.ha@samsung.com>

diff --git a/Makefile.am b/Makefile.am
index d69295b..0fabd6e 100644 (file)
--- a/Makefile.am
+++ b/Makefile.am
@@ -290,7 +290,7 @@ check_PROGRAMS = \
        src/tests/salvage_pipe_entry \
        src/tests/filters
 
-check_CFLAGS = $(AM_CFLAGS) -O0 -fprofile-arcs -ftest-coverage -DUNIT_TEST \
+check_CFLAGS = $(AM_CFLAGS) -O0 -fprofile-arcs -DUNIT_TEST \
        -DEXIT_SKIP=77 -DEXIT_HARD_ERROR=99 # autotools convention exit codes
 
 TESTS = $(check_PROGRAMS) \

diff --git a/packaging/dlog.spec b/packaging/dlog.spec
index ff8d9b3..8031281 100644 (file)
--- a/packaging/dlog.spec
+++ b/packaging/dlog.spec
@@ -24,8 +24,8 @@ BuildRequires: libcap-devel
 BuildRequires: libtool
 BuildRequires: pkgconfig(capi-base-common)
 BuildRequires: pkgconfig(libtzplatform-config)
-BuildRequires: lcov
 %if 0%{?gcov:1}
+BuildRequires: lcov
 BuildRequires: zip
 %endif
 
@@ -158,7 +158,11 @@ make %{?jobs:-j%jobs} CFLAGS+='-D__DLOG_VERSION=\"%{version}\" -D__DLOG_RELEASE=
 
 # autoconf adds a bunch of flags, including -O2.
 # Clear those so that test coverage can be measured properly
+%if 0%{?gcov:1}
+make check CFLAGS='-Wno-gnu-variable-sized-type-not-at-end -Wno-logical-op-parentheses -ftest-coverage -D__DLOG_VERSION=\"%{version}\" -D__DLOG_RELEASE=\"%{release}\"'
+%else
 make check CFLAGS='-Wno-gnu-variable-sized-type-not-at-end -Wno-logical-op-parentheses -D__DLOG_VERSION=\"%{version}\" -D__DLOG_RELEASE=\"%{release}\"'
+%endif
 
 make dox