| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106 |
- From 68f8393bd356a3d0598cf77e1044b7e8b98aa4d8 Mon Sep 17 00:00:00 2001
- Message-Id: <68f8393bd356a3d0598cf77e1044b7e8b98aa4d8.1493920165.git.baruch@tkos.co.il>
- From: Baruch Siach <baruch@tkos.co.il>
- Date: Thu, 4 May 2017 15:56:14 +0300
- Subject: [PATCH] Revert "build: enable building cli during tests"
- This reverts commit 3cb28534d34392ceec4adead0cfa97039796ccb7.
- Contrary to what 3cb28534d commit log claims, the cli programs depend on
- dynamic libraries support of the toolchain. Enabling cli programs
- unconditionally breaks static build as follows:
- In file included from lib/cli/cls/basic.c:12:0:
- ./include/netlink/cli/utils.h:25:19: fatal error: dlfcn.h: No such file or directory
- compilation terminated.
- Makefile:3666: recipe for target 'lib/cli/cls/lib_cli_cls_basic_la-basic.lo' failed
- make[1]: *** [lib/cli/cls/lib_cli_cls_basic_la-basic.lo] Error 1
- Revert that commit to restore the ability of static only build of libnl, and
- its dependencies.
- Signed-off-by: Baruch Siach <baruch@tkos.co.il>
- ---
- Upstream status: https://github.com/thom311/libnl/pull/141
- Makefile.am | 21 ++++++---------------
- 1 file changed, 6 insertions(+), 15 deletions(-)
- --- a/Makefile.am
- +++ b/Makefile.am
- @@ -3,8 +3,6 @@
- ACLOCAL_AMFLAGS = -I m4
-
- lib_LTLIBRARIES =
- -noinst_LTLIBRARIES =
- -check_LTLIBRARIES =
-
- check_PROGRAMS =
- check_programs =
- @@ -511,6 +509,8 @@ EXTRA_lib_libnl_xfrm_3_la_DEPENDENCIES =
- lib_libnl_xfrm_3_la_LIBADD = \
- lib/libnl-3.la
-
- +if ENABLE_CLI
- +
- lib_cli_ltlibraries_cls = \
- lib/cli/cls/basic.la \
- lib/cli/cls/cgroup.la
- @@ -524,15 +524,11 @@ lib_cli_ltlibraries_qdisc = \
- lib/cli/qdisc/pfifo.la \
- lib/cli/qdisc/plug.la
-
- -if ENABLE_CLI
- pkglib_clsdir = $(pkglibdir)/cli/cls
- pkglib_qdiscdir = $(pkglibdir)/cli/qdisc
- pkglib_cls_LTLIBRARIES = $(lib_cli_ltlibraries_cls)
- pkglib_qdisc_LTLIBRARIES = $(lib_cli_ltlibraries_qdisc)
- -else
- -check_LTLIBRARIES += \
- - $(lib_cli_ltlibraries_cls) \
- - $(lib_cli_ltlibraries_qdisc)
- +
- endif
-
- lib_cli_ldflags = \
- @@ -565,9 +561,6 @@ src_lib_ldflags =
-
- if ENABLE_CLI
- lib_LTLIBRARIES += src/lib/libnl-cli-3.la
- -src_lib_ldflags += -version-info $(LT_CURRENT):$(LT_REVISION):$(LT_AGE)
- -else
- -check_LTLIBRARIES += src/lib/libnl-cli-3.la
- endif
-
- src_lib_libnl_cli_3_la_SOURCES = \
- @@ -594,7 +587,7 @@ src_lib_libnl_cli_3_la_CPPFLAGS = \
- -I$(srcdir)/include \
- -I$(builddir)/include
- src_lib_libnl_cli_3_la_LDFLAGS = \
- - $(src_lib_ldflags) \
- + -version-info $(LT_CURRENT):$(LT_REVISION):$(LT_AGE) \
- -Wl,--version-script=$(srcdir)/libnl-cli-3.sym
- src_lib_libnl_cli_3_la_LIBADD = \
- lib/libnl-3.la \
- @@ -679,8 +672,6 @@ else
- noinst_PROGRAMS += $(cli_programs)
- endif
- endif
- -else
- -check_PROGRAMS += $(cli_programs)
- endif
-
- src_genl_ctrl_list_CPPFLAGS = $(src_cppflags)
- @@ -858,10 +849,12 @@ tests_test_complex_HTB_with_hash_filters
- tests_test_u32_filter_with_actions_CPPFLAGS = $(tests_cppflags)
- tests_test_u32_filter_with_actions_LDADD = $(tests_ldadd)
-
- +if ENABLE_CLI
- check_PROGRAMS += \
- tests/test-cache-mngr \
- tests/test-genl \
- tests/test-nf-cache-mngr
- +endif
-
- tests_cli_ldadd = \
- $(tests_ldadd) \
|
