[gspell/wip/fix-code-coverage] build: use CODE_COVERAGE_LIBS instead of LDFLAGS (WIP, doesn't work)



commit bfaf33396ea2a7d29b54cb01b72dfc96db9ab000
Author: Sébastien Wilmet <swilmet gnome org>
Date:   Sun Oct 9 18:44:02 2016 +0200

    build: use CODE_COVERAGE_LIBS instead of LDFLAGS (WIP, doesn't work)
    
    It requires an up-to-date ax_code_coverage.m4 from autoconf-archive.

 configure.ac       |    2 +-
 gspell/Makefile.am |   18 ++++++++++--------
 2 files changed, 11 insertions(+), 9 deletions(-)
---
diff --git a/configure.ac b/configure.ac
index 6028260..53749c3 100644
--- a/configure.ac
+++ b/configure.ac
@@ -148,7 +148,7 @@ VAPIGEN_CHECK
 
 # Code coverage for unit tests
 AX_REQUIRE_DEFINED([AX_CODE_COVERAGE])
-AX_CODE_COVERAGE()
+AX_CODE_COVERAGE
 
 # Valgrind check
 AX_REQUIRE_DEFINED([AX_VALGRIND_CHECK])
diff --git a/gspell/Makefile.am b/gspell/Makefile.am
index 98b3d11..62a82d8 100644
--- a/gspell/Makefile.am
+++ b/gspell/Makefile.am
@@ -78,13 +78,15 @@ libgspell_core_la_SOURCES =         \
 nodist_libgspell_core_la_SOURCES = \
        $(BUILT_SOURCES)
 
-libgspell_core_la_CFLAGS = \
+libgspell_core_la_LIBADD =     \
+       $(CODE_COVERAGE_LIBS)
+
+libgspell_core_la_CFLAGS =     \
        $(CODE_COVERAGE_CFLAGS)
 
-libgspell_core_la_LDFLAGS =            \
-       -no-undefined                   \
-       $(WARN_LDFLAGS)                 \
-       $(CODE_COVERAGE_LDFLAGS)
+libgspell_core_la_LDFLAGS =    \
+       -no-undefined           \
+       $(WARN_LDFLAGS)
 
 # The real library.
 lib_LTLIBRARIES = libgspell-@GSPELL_API_VERSION@.la
@@ -94,7 +96,8 @@ libgspell_@GSPELL_API_VERSION@_la_SOURCES =
 libgspell_@GSPELL_API_VERSION@_la_LIBADD =     \
        libgspell-core.la                       \
        $(DEP_LIBS)                             \
-       $(GTK_MAC_LIBS)
+       $(GTK_MAC_LIBS)                         \
+       $(CODE_COVERAGE_LIBS)
 
 libgspell_@GSPELL_API_VERSION@_la_CFLAGS =     \
        $(CODE_COVERAGE_CFLAGS)
@@ -103,8 +106,7 @@ libgspell_@GSPELL_API_VERSION@_la_LDFLAGS = \
        -version-info $(GSPELL_LT_VERSION)      \
        -no-undefined                           \
        -export-symbols-regex "^gspell_.*"      \
-       $(WARN_LDFLAGS)                         \
-       $(CODE_COVERAGE_LDFLAGS)
+       $(WARN_LDFLAGS)
 
 libgspell_includedir = $(includedir)/gspell-@GSPELL_API_VERSION@/gspell
 libgspell_include_HEADERS = $(gspell_public_headers)


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]