]> git.puffer.fish Git - mirror/frr.git/commitdiff
build: fix a whole bunch of *FLAGS
authorDavid Lamparter <equinox@diac24.net>
Wed, 30 Jan 2019 17:11:54 +0000 (18:11 +0100)
committerDavid Lamparter <equinox@diac24.net>
Wed, 30 Jan 2019 18:14:27 +0000 (19:14 +0100)
- some target_CFLAGS that needed to include AM_CFLAGS didn't do so
- libyang/sysrepo/sqlite3/confd CFLAGS + LIBS weren't used at all
- consistently use $(FOO_CFLAGS) instead of @FOO_CFLAGS@
- 2 dependencies were missing for clippy

Signed-off-by: David Lamparter <equinox@diac24.net>
23 files changed:
Makefile.am
babeld/subdir.am
bgpd/subdir.am
configure.ac
eigrpd/subdir.am
isisd/subdir.am
ldpd/subdir.am
lib/subdir.am
nhrpd/subdir.am
ospf6d/subdir.am
ospfclient/subdir.am
ospfd/subdir.am
pbrd/subdir.am
pimd/subdir.am
ripd/subdir.am
ripngd/subdir.am
sharpd/subdir.am
staticd/subdir.am
tests/subdir.am
tools/subdir.am
vtysh/subdir.am
watchfrr/subdir.am
yang/libyang_plugins/subdir.am

index c56a551aa50d7afb24307d9e25eb95f1989271ef..56570cbb903894ea16f130c77f3d8ef358a50aa4 100644 (file)
@@ -4,6 +4,8 @@ AUTOMAKE_OPTIONS = subdir-objects 1.12
 ACLOCAL_AMFLAGS = -I m4
 
 AM_CFLAGS = \
+       $(LIBYANG_CFLAGS) \
+       $(SQLITE3_CFLAGS) \
        $(UNWIND_CFLAGS) \
        $(SAN_FLAGS) \
        $(WERROR) \
@@ -17,7 +19,6 @@ AM_LDFLAGS = \
        $(SAN_FLAGS) \
        # end
 DEFS = @DEFS@ -DSYSCONFDIR=\"$(sysconfdir)/\" -DCONFDATE=$(CONFDATE)
-LIBCAP = @LIBCAP@
 
 AR_FLAGS = @AR_FLAGS@
 ARFLAGS = @ARFLAGS@
index e1f2cb0a00d78b2ecdfeb6b0736419bfc99baf1b..7081c730aa2db7a1613bbd55c6a485d3832d12ec 100644 (file)
@@ -51,4 +51,4 @@ noinst_HEADERS += \
        # end
 
 babeld_babeld_SOURCES = babeld/babel_main.c
-babeld_babeld_LDADD = babeld/libbabel.a lib/libfrr.la @LIBCAP@
+babeld_babeld_LDADD = babeld/libbabel.a lib/libfrr.la $(LIBCAP)
index 7d7d3ca189068c685bd797a5e52b0407068b71a6..2ccf7dae1be322d238141904d619d55780e19476 100644 (file)
@@ -191,15 +191,15 @@ bgpd_bgp_btoa_SOURCES = bgpd/bgp_btoa.c
 
 if ENABLE_BGP_VNC
 bgpd_bgpd_SOURCES += bgpd/rfapi/rfapi_descriptor_rfp_utils.c
-bgpd_bgpd_CFLAGS = -Irfapi -I@top_srcdir@/$(RFPINC)
+bgpd_bgpd_CFLAGS = $(AM_CFLAGS) -Irfapi -I@top_srcdir@/$(RFPINC)
 
 bgpd_bgp_btoa_SOURCES += bgpd/rfapi/rfapi_descriptor_rfp_utils.c
-bgpd_bgp_btoa_CFLAGS = -Irfapi -I@top_srcdir@/$(RFPINC)
+bgpd_bgp_btoa_CFLAGS = $(AM_CFLAGS) -Irfapi -I@top_srcdir@/$(RFPINC)
 endif
 
 # RFPLDADD is set in bgpd/rfp-example/librfp/subdir.am
-bgpd_bgpd_LDADD = bgpd/libbgp.a $(RFPLDADD) lib/libfrr.la @LIBCAP@ @LIBM@
-bgpd_bgp_btoa_LDADD = bgpd/libbgp.a $(RFPLDADD) lib/libfrr.la @LIBCAP@ @LIBM@
+bgpd_bgpd_LDADD = bgpd/libbgp.a $(RFPLDADD) lib/libfrr.la $(LIBCAP) $(LIBM)
+bgpd_bgp_btoa_LDADD = bgpd/libbgp.a $(RFPLDADD) lib/libfrr.la $(LIBCAP) $(LIBM)
 
 bgpd_bgpd_snmp_la_SOURCES = bgpd/bgp_snmp.c
 bgpd_bgpd_snmp_la_CFLAGS = $(WERROR) $(SNMP_CFLAGS) -std=gnu99
index d112f8b30fea4b36bf629e2e81cff8dee35d77fb..022900acd08b1c3b5a77786695a51c3fbd10fb3c 100755 (executable)
@@ -1598,18 +1598,21 @@ AC_SUBST([SNMP_CFLAGS])
 dnl ---------------
 dnl libyang
 dnl ---------------
-PKG_CHECK_MODULES([libyang], [libyang >= 0.16.7], , [
+PKG_CHECK_MODULES([LIBYANG], [libyang >= 0.16.7], , [
   AC_MSG_ERROR([libyang (>= 0.16.7) was not found on your system.])
 ])
+ac_cflags_save="$CFLAGS"
+CFLAGS="$CFLAGS $LIBYANG_CFLAGS"
 AC_CHECK_MEMBER([struct lyd_node.priv], [], [
   AC_MSG_ERROR([m4_normalize([
     libyang needs to be compiled with ENABLE_LYD_PRIV=ON.
     See http://docs.frrouting.org/projects/dev-guide/en/latest/building-libyang.html for details.])
   ])
 ], [[#include <libyang/libyang.h>]])
+CFLAGS="$ac_cflags_save"
 
 ac_libs_save="$LIBS"
-LIBS="$LIBS $libyang_LIBS"
+LIBS="$LIBS $LIBYANG_LIBS"
 AC_CHECK_FUNC([ly_register_types], [
   libyang_ext_builtin=true
   AC_DEFINE([LIBYANG_EXT_BUILTIN], [1], [have ly_register_types()])
@@ -1631,7 +1634,7 @@ dnl configuration rollbacks
 dnl ---------------
 SQLITE3=false
 if test "$enable_config_rollbacks" = "yes"; then
-  PKG_CHECK_MODULES([sqlite3], [sqlite3], [
+  PKG_CHECK_MODULES([SQLITE3], [sqlite3], [
     AC_DEFINE([HAVE_CONFIG_ROLLBACKS], [1], [Enable configuration rollbacks])
     AC_DEFINE([HAVE_SQLITE3], [1], [Enable sqlite3 database])
     SQLITE3=true
@@ -1659,7 +1662,7 @@ dnl ---------------
 dnl sysrepo
 dnl ---------------
 if test "$enable_sysrepo" = "yes"; then
-  PKG_CHECK_MODULES([sysrepo], [libsysrepo],
+  PKG_CHECK_MODULES([SYSREPO], [libsysrepo],
       [AC_DEFINE([HAVE_SYSREPO], [1], [Enable sysrepo integration])
       SYSREPO=true],
       [SYSREPO=false
index d532afbbe632de5abe746de981923d40dc6aaa04..4503030fdfe854762802a85bb54fd2d9c8d8e779 100644 (file)
@@ -65,4 +65,4 @@ noinst_HEADERS += \
        # end
 
 eigrpd_eigrpd_SOURCES = eigrpd/eigrp_main.c
-eigrpd_eigrpd_LDADD = eigrpd/libeigrp.a lib/libfrr.la @LIBCAP@
+eigrpd_eigrpd_LDADD = eigrpd/libeigrp.a lib/libfrr.la $(LIBCAP)
index c5b9b31a480735bc52f50bb54b98d775e6970647..4371d5993af4e4224bf87632e0215565173dfa33 100644 (file)
@@ -98,7 +98,7 @@ ISIS_SOURCES = \
        isisd/isis_pfpacket.c \
        # end
 
-ISIS_LDADD_COMMON = lib/libfrr.la @LIBCAP@
+ISIS_LDADD_COMMON = lib/libfrr.la $(LIBCAP)
 
 # Building isisd
 
index 24e738d6227a0805c5861f40d98e3f65bf3ab7b0..42c5ad024b2988197a9ef348cf501d1c9550f860 100644 (file)
@@ -55,4 +55,4 @@ noinst_HEADERS += \
        # end
 
 ldpd_ldpd_SOURCES = ldpd/ldpd.c
-ldpd_ldpd_LDADD = ldpd/libldp.a lib/libfrr.la @LIBCAP@
+ldpd_ldpd_LDADD = ldpd/libldp.a lib/libfrr.la $(LIBCAP)
index 43b39100cb3a701675bf540109f9d08db75fee8f..41da1adddbfde97286bd1415803d09a33d576880 100644 (file)
@@ -3,7 +3,7 @@
 #
 lib_LTLIBRARIES += lib/libfrr.la
 lib_libfrr_la_LDFLAGS = -version-info 0:0:0 -Xlinker -e_libfrr_version
-lib_libfrr_la_LIBADD = @LIBCAP@ $(UNWIND_LIBS) -lyang
+lib_libfrr_la_LIBADD = $(LIBCAP) $(UNWIND_LIBS) $(LIBYANG_LIBS)
 
 lib_libfrr_la_SOURCES = \
        lib/agg_table.c \
@@ -115,7 +115,7 @@ vtysh_scan += \
 vtysh_scan += $(top_srcdir)/lib/agentx.c
 
 if SQLITE3
-lib_libfrr_la_LIBADD += -lsqlite3
+lib_libfrr_la_LIBADD += $(SQLITE3_LIBS)
 lib_libfrr_la_SOURCES += lib/db.c
 endif
 
@@ -287,7 +287,7 @@ endif
 
 lib_confd_la_CFLAGS = $(WERROR) $(CONFD_CFLAGS)
 lib_confd_la_LDFLAGS = -avoid-version -module -shared -export-dynamic
-lib_confd_la_LIBADD = lib/libfrr.la -lconfd
+lib_confd_la_LIBADD = lib/libfrr.la $(CONFD_LIBS)
 lib_confd_la_SOURCES = lib/northbound_confd.c
 
 #
@@ -297,9 +297,9 @@ if SYSREPO
 module_LTLIBRARIES += lib/sysrepo.la
 endif
 
-lib_sysrepo_la_CFLAGS = $(WERROR)
+lib_sysrepo_la_CFLAGS = $(WERROR) $(SYSREPO_CFLAGS)
 lib_sysrepo_la_LDFLAGS = -avoid-version -module -shared -export-dynamic
-lib_sysrepo_la_LIBADD = lib/libfrr.la -lsysrepo
+lib_sysrepo_la_LIBADD = lib/libfrr.la $(SYSREPO_LIBS)
 lib_sysrepo_la_SOURCES = lib/northbound_sysrepo.c
 
 #
@@ -385,11 +385,13 @@ lib/command_lex.h: lib/command_lex.c
        @if test ! -f $@; then $(MAKE) $(AM_MAKEFLAGS) "lib/command_lex.c"; else :; fi
 lib/command_lex.lo: lib/command_parse.h
 lib/command_parse.lo: lib/command_lex.h
+lib/clippy-command_lex.$(OBJEXT): lib/command_parse.h
+lib/clippy-command_parse.$(OBJEXT): lib/command_lex.h
 lib/lib_clippy-command_lex.$(OBJEXT): lib/command_parse.h
 lib/lib_clippy-command_parse.$(OBJEXT): lib/command_lex.h
 
 lib/route_types.h: $(top_srcdir)/lib/route_types.txt $(top_srcdir)/lib/route_types.pl
-       @PERL@ $(top_srcdir)/lib/route_types.pl < $(top_srcdir)/lib/route_types.txt > $@
+       $(PERL) $(top_srcdir)/lib/route_types.pl < $(top_srcdir)/lib/route_types.txt > $@
 DISTCLEANFILES += lib/route_types.h
 
 if GIT_VERSION
@@ -403,7 +405,7 @@ PHONY_GITVERSION=lib/gitversion.h.tmp
 .SILENT: lib/gitversion.h lib/gitversion.h.tmp
 GITH=lib/gitversion.h
 lib/gitversion.h.tmp: $(top_srcdir)/.git
-       @PERL@ $(top_srcdir)/lib/gitversion.pl $(top_srcdir) > ${GITH}.tmp
+       $(PERL) $(top_srcdir)/lib/gitversion.pl $(top_srcdir) > ${GITH}.tmp
 lib/gitversion.h: lib/gitversion.h.tmp
        { test -f ${GITH} && diff -s -q ${GITH}.tmp ${GITH}; } || cp ${GITH}.tmp ${GITH}
 
index 758c22e2be87d62a406082d1688416df8fb4ab07..6e2b91780f764625f109b0261b0c1460c717f4f0 100644 (file)
@@ -8,8 +8,8 @@ vtysh_scan += $(top_srcdir)/nhrpd/nhrp_vty.c
 man8 += $(MANBUILD)/nhrpd.8
 endif
 
-nhrpd_nhrpd_LDADD = lib/libfrr.la @LIBCAP@ @CARES_LIBS@
-nhrpd_nhrpd_CFLAGS = $(AM_CFLAGS) @CARES_CFLAGS@
+nhrpd_nhrpd_LDADD = lib/libfrr.la $(LIBCAP) $(CARES_LIBS)
+nhrpd_nhrpd_CFLAGS = $(AM_CFLAGS) $(CARES_CFLAGS)
 nhrpd_nhrpd_SOURCES = \
        nhrpd/linux.c \
        nhrpd/netlink_arp.c \
index d9c29f2651b9e028996f72367ea2c1ff23386ef3..eac0eee45f849894b1b6dd7be1ab9916114387bf 100644 (file)
@@ -73,7 +73,7 @@ noinst_HEADERS += \
        ospf6d/ospf6d.h \
        # end
 
-ospf6d_ospf6d_LDADD = ospf6d/libospf6.a lib/libfrr.la @LIBCAP@
+ospf6d_ospf6d_LDADD = ospf6d/libospf6.a lib/libfrr.la $(LIBCAP)
 ospf6d_ospf6d_SOURCES = \
        ospf6d/ospf6_main.c \
        # end
index c05d920d5fb4d9eabd949e887bc5de8a90f3ae54..94d489358ce7824b363cc3c1fccb39439bc3f337 100644 (file)
@@ -23,7 +23,7 @@ endif
 
 ospfclient_ospfclient_LDADD = \
        ospfclient/libfrrospfapiclient.la \
-       @LIBCAP@ \
+       $(LIBCAP) \
        # end
 
 if STATIC_BIN
index 3ad1b870b4b7b01f8ac4b24fe4e52aaa180ad47d..48dd741b24639d6121179ea9e9997bec0339857c 100644 (file)
@@ -98,7 +98,7 @@ noinst_HEADERS += \
        ospfd/ospf_zebra.h \
        # end
 
-ospfd_ospfd_LDADD = ospfd/libfrrospf.a lib/libfrr.la @LIBCAP@ @LIBM@
+ospfd_ospfd_LDADD = ospfd/libfrrospf.a lib/libfrr.la $(LIBCAP) $(LIBM)
 ospfd_ospfd_SOURCES = ospfd/ospf_main.c
 
 ospfd_ospfd_snmp_la_SOURCES = ospfd/ospf_snmp.c
index 79475590345dd8ac9d7e8476fb47e7e5eadca393..0f2e7ad8bdb6f7e27e15e95fa351b9c6b0541568 100644 (file)
@@ -38,5 +38,5 @@ pbrd/pbr_debug_clippy.c: $(CLIPPY_DEPS)
 pbrd/pbr_debug.$(OBJEXT): pbrd/pbr_debug_clippy.c
 
 pbrd_pbrd_SOURCES = pbrd/pbr_main.c
-pbrd_pbrd_LDADD = pbrd/libpbr.a lib/libfrr.la @LIBCAP@
+pbrd_pbrd_LDADD = pbrd/libpbr.a lib/libfrr.la $(LIBCAP)
 
index fef8e36577fe9f5116cafe43222d2d3b871a785f..99701430c59588be5d92c7b20ebe0e799ffbc779 100644 (file)
@@ -115,7 +115,7 @@ noinst_HEADERS += \
        pimd/mtracebis_routeget.h \
        # end
 
-pimd_pimd_LDADD = pimd/libpim.a lib/libfrr.la @LIBCAP@
+pimd_pimd_LDADD = pimd/libpim.a lib/libfrr.la $(LIBCAP)
 pimd_pimd_SOURCES = pimd/pim_main.c
 
 pimd_test_igmpv3_join_LDADD = lib/libfrr.la
index 1c2f8d64c8fda43a736645a6969e80280aaeefdf..2a63cc52296a2afaaf100b6b7f7fd4d0c845e0c8 100644 (file)
@@ -44,7 +44,7 @@ noinst_HEADERS += \
        ripd/ripd.h \
        # end
 
-ripd_ripd_LDADD = ripd/librip.a lib/libfrr.la @LIBCAP@
+ripd_ripd_LDADD = ripd/librip.a lib/libfrr.la $(LIBCAP)
 ripd_ripd_SOURCES = \
        ripd/rip_main.c \
        # end
index d401e9bbf6cfa586b1d32ee174f95b93df2254a8..ea0ccf14827c52698bd5adbd7a1c14343ce243c8 100644 (file)
@@ -40,7 +40,7 @@ noinst_HEADERS += \
        ripngd/ripngd.h \
        # end
 
-ripngd_ripngd_LDADD = ripngd/libripng.a lib/libfrr.la @LIBCAP@
+ripngd_ripngd_LDADD = ripngd/libripng.a lib/libfrr.la $(LIBCAP)
 ripngd_ripngd_SOURCES = \
        ripngd/ripng_main.c \
        # end
index 2a34aecfb3eff05f495b85f1222d594cb5cba3dc..c00d90ea0a148a9cd54cffc27f1f9a90bf1f0213 100644 (file)
@@ -24,5 +24,5 @@ sharpd/sharp_vty_clippy.c: $(CLIPPY_DEPS)
 sharpd/sharp_vty.$(OBJEXT): sharpd/sharp_vty_clippy.c
 
 sharpd_sharpd_SOURCES = sharpd/sharp_main.c
-sharpd_sharpd_LDADD = sharpd/libsharp.a lib/libfrr.la @LIBCAP@
+sharpd_sharpd_LDADD = sharpd/libsharp.a lib/libfrr.la $(LIBCAP)
 
index 33cc0e20505ea1d37b149595a3bcd034b294f27b..17c4536fe9fc2b393f63f38d6fc8f7354845aac4 100644 (file)
@@ -32,4 +32,4 @@ staticd/static_vty_clippy.c: $(CLIPPY_DEPS)
 staticd/static_vty.$(OBJEXT): staticd/static_vty_clippy.c
 
 staticd_staticd_SOURCES = staticd/static_main.c
-staticd_staticd_LDADD = staticd/libstatic.a lib/libfrr.la @LIBCAP@
+staticd_staticd_LDADD = staticd/libstatic.a lib/libfrr.la $(LIBCAP)
index 7d2800a3a2a8e81d6e8c5548a01a20080edf649c..42730cb83a009385393ad65d38ac7ddf220cd741 100644 (file)
@@ -112,10 +112,13 @@ TESTS_CPPFLAGS = $(AM_CPPFLAGS) \
        -I$(top_srcdir)/tests/helpers/c \
        -I$(top_builddir)/tests/helpers/c \
        # end
-TESTS_CFLAGS = $(SAN_FLAGS)
+TESTS_CFLAGS = \
+       $(LIBYANG_CFLAGS) \
+       $(SAN_FLAGS) \
+       # end
 # note no -Werror
 
-ALL_TESTS_LDADD = lib/libfrr.la @LIBCAP@
+ALL_TESTS_LDADD = lib/libfrr.la $(LIBCAP)
 BGP_TEST_LDADD = bgpd/libbgp.a $(RFPLDADD) $(ALL_TESTS_LDADD) -lm
 ISISD_TEST_LDADD = isisd/libisis.a $(ALL_TESTS_LDADD)
 OSPF6_TEST_LDADD = ospf6d/libospf6.a $(ALL_TESTS_LDADD)
index ff41fe2c63ce684deb5c68df47d467e2fbce2ab3..7713bb1ade5baa8aad9f4515e8f3091f4c54d0f9 100644 (file)
@@ -23,10 +23,10 @@ tools_permutations_SOURCES = tools/permutations.c
 tools_permutations_LDADD = lib/libfrr.la
 
 tools_gen_northbound_callbacks_SOURCES = tools/gen_northbound_callbacks.c
-tools_gen_northbound_callbacks_LDADD = lib/libfrr.la -lyang
+tools_gen_northbound_callbacks_LDADD = lib/libfrr.la $(LIBYANG_LIBS)
 
 tools_gen_yang_deviations_SOURCES = tools/gen_yang_deviations.c
-tools_gen_yang_deviations_LDADD = lib/libfrr.la -lyang
+tools_gen_yang_deviations_LDADD = lib/libfrr.la $(LIBYANG_LIBS)
 
 tools_ssd_SOURCES = tools/start-stop-daemon.c
 
index c9cdb5854383695dd3c949d2e678131585820bba..74595788b0c744a9ecd260ae1b28e811721b9d97 100644 (file)
@@ -24,7 +24,7 @@ noinst_HEADERS += \
        vtysh/vtysh_user.h \
        # end
 
-vtysh_vtysh_LDADD = lib/libfrr.la @LIBCAP@ @LIBREADLINE@ @LIBS@ @LIBPAM@
+vtysh_vtysh_LDADD = lib/libfrr.la $(LIBCAP) $(LIBREADLINE) $(LIBS) $(LIBPAM)
 
 AM_V_EXTRACT = $(am__v_EXTRACT_$(V))
 am__v_EXTRACT_ = $(am__v_EXTRACT_$(AM_DEFAULT_VERBOSITY))
index f0b49c9a8482101ea6386329abf9ce756b122541..c27491e55c7709a0e9e17614696e7a0d9d761c11 100644 (file)
@@ -13,7 +13,7 @@ noinst_HEADERS += \
        watchfrr/watchfrr_errors.h \
        # end
 
-watchfrr_watchfrr_LDADD = lib/libfrr.la @LIBCAP@
+watchfrr_watchfrr_LDADD = lib/libfrr.la $(LIBCAP)
 watchfrr_watchfrr_SOURCES = \
        watchfrr/watchfrr.c \
        watchfrr/watchfrr_errors.c \
index 716478908342d2ede059e0b376c65f0bc44b3825..fe5f34a28a034dfc57d398e15f82ff7b51bcd48e 100644 (file)
@@ -8,7 +8,7 @@ else
 libyang_plugins_LTLIBRARIES += yang/libyang_plugins/frr_user_types.la
 endif
 
-yang_libyang_plugins_frr_user_types_la_CFLAGS = $(WERROR)
+yang_libyang_plugins_frr_user_types_la_CFLAGS = $(WERROR) $(LIBYANG_CFLAGS)
 yang_libyang_plugins_frr_user_types_la_LDFLAGS = -avoid-version -module -shared -export-dynamic
 yang_libyang_plugins_frr_user_types_la_LIBADD =
 yang_libyang_plugins_frr_user_types_la_SOURCES = yang/libyang_plugins/frr_user_types.c