neels submitted this change.

View Change


Approvals: neels: Looks good to me, approved Jenkins Builder: Verified
build: drop COVERAGE

There isn't even an --enable-coverage in the configure.ac, so drop the
always empty COVERAGE_CFLAGS and COVERAGE_LDFLAGS.

Change-Id: Ib256ce7d979aec5c368b19ac2c145f75a9dab0fc
---
M src/libosmo-gtlv/Makefile.am
M src/libosmo-pfcp/Makefile.am
2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/src/libosmo-gtlv/Makefile.am b/src/libosmo-gtlv/Makefile.am
index 1c40386..ae77948 100644
--- a/src/libosmo-gtlv/Makefile.am
+++ b/src/libosmo-gtlv/Makefile.am
@@ -8,12 +8,10 @@
AM_CFLAGS = \
-Wall \
$(LIBOSMOCORE_CFLAGS) \
- $(COVERAGE_CFLAGS) \
$(NULL)

AM_LDFLAGS = \
$(LIBOSMOCORE_LIBS) \
- $(COVERAGE_LDFLAGS) \
$(NULL)

lib_LTLIBRARIES = \
diff --git a/src/libosmo-pfcp/Makefile.am b/src/libosmo-pfcp/Makefile.am
index 578ac0b..fdc30e0 100644
--- a/src/libosmo-pfcp/Makefile.am
+++ b/src/libosmo-pfcp/Makefile.am
@@ -11,14 +11,12 @@
$(LIBOSMOCORE_CFLAGS) \
$(LIBOSMOGSM_CFLAGS) \
$(LIBOSMOVTY_CFLAGS) \
- $(COVERAGE_CFLAGS) \
$(NULL)

AM_LDFLAGS = \
$(LIBOSMOCORE_LIBS) \
$(LIBOSMOGSM_LIBS) \
$(LIBOSMOVTY_LIBS) \
- $(COVERAGE_LDFLAGS) \
$(NULL)

lib_LTLIBRARIES = \
@@ -68,5 +66,4 @@
gen__pfcp_ies_auto_LDADD = \
$(top_builddir)/src/libosmo-gtlv/libosmo-gtlv.la \
$(LIBOSMOCORE_LIBS) \
- $(COVERAGE_LDFLAGS) \
$(NULL)

To view, visit change 31761. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmo-pfcp
Gerrit-Branch: master
Gerrit-Change-Id: Ib256ce7d979aec5c368b19ac2c145f75a9dab0fc
Gerrit-Change-Number: 31761
Gerrit-PatchSet: 2
Gerrit-Owner: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged