[kronosnet/kronosnet] 01d70a: build: remove unused DEBUG Automake conditional

Branch: refs/heads/stable1-proposed Home: https://github.com/kronosnet/kronosnet Commit: 01d70a95f989f03b7e78525e36d265b42bf8718f https://github.com/kronosnet/kronosnet/commit/01d70a95f989f03b7e78525e36d26… Author: Ferenc Wágner <wferi(a)debian.org> Date: 2018-01-30 (Tue, 30 Jan 2018) Changed paths: M configure.ac Log Message: ----------- build: remove unused DEBUG Automake conditional Signed-off-by: Ferenc Wágner <wferi(a)debian.org> Commit: 416b3c99b2c9a30e3f2ee81620c0dcb9f161efac https://github.com/kronosnet/kronosnet/commit/416b3c99b2c9a30e3f2ee81620c0d… Author: Ferenc Wágner <wferi(a)debian.org> Date: 2018-01-30 (Tue, 30 Jan 2018) Changed paths: M poc-code/iov-hash/Makefile.am Log Message: ----------- build: LIBS belong to LDADD for binaries Signed-off-by: Ferenc Wágner <wferi(a)debian.org> Commit: 29282cb51a4563845031fc86cf6a2ddb0604a9e5 https://github.com/kronosnet/kronosnet/commit/29282cb51a4563845031fc86cf6a2… Author: Ferenc Wágner <wferi(a)debian.org> Date: 2018-01-30 (Tue, 30 Jan 2018) Changed paths: M configure.ac M libknet/Makefile.am M libtap/Makefile.am Log Message: ----------- build: sanitize LDFLAGS handling The Automake manual states that according to the GNU Coding Standards the so-called "Variables reserved for the user" mustn't be changed by the build system, and they must override the default settings from the build system. The "Flag Variables Ordering" section provides the recipes on which we build here. Using $lt_prog_compiler_pic directly shouldn't be necessary, let's try leaving it out. Signed-off-by: Ferenc Wágner <wferi(a)debian.org> Commit: c6afa80e35115b4bd0cc9e067364209a20ad0fef https://github.com/kronosnet/kronosnet/commit/c6afa80e35115b4bd0cc9e0673642… Author: Ferenc Wágner <wferi(a)debian.org> Date: 2018-01-30 (Tue, 30 Jan 2018) Changed paths: M configure.ac Log Message: ----------- build: we preserve CPPFLAGS, just remove the tempting hooks Signed-off-by: Ferenc Wágner <wferi(a)debian.org> Commit: 57a101d1b11f1ab70a0ac15921d1852af46728e1 https://github.com/kronosnet/kronosnet/commit/57a101d1b11f1ab70a0ac15921d18… Author: Ferenc Wágner <wferi(a)debian.org> Date: 2018-01-30 (Tue, 30 Jan 2018) Changed paths: M configure.ac M kronosnetd/Makefile.am M libknet/Makefile.am M libknet/tests/Makefile.am M libtap/Makefile.am M poc-code/iov-hash/Makefile.am Log Message: ----------- build: sanitize CFLAGS handling As for LDFLAGS previously: leave the CFLAGS "user variable" untouched to enable easy selective overriding of the flags used by the build system. Using $lt_prog_compiler_pic shouldn't be necessary, let's drop it and see if anything breaks. WERROR_CFLAGS was undefined, so dropped as well. Signed-off-by: Ferenc Wágner <wferi(a)debian.org> Commit: 69da7da9cafa18914203c65c45f4f7a4cc712d46 https://github.com/kronosnet/kronosnet/commit/69da7da9cafa18914203c65c45f4f… Author: Ferenc Wágner <wferi(a)debian.org> Date: 2018-01-30 (Tue, 30 Jan 2018) Changed paths: M configure.ac A m4/ax_check_link_flag.m4 Log Message: ----------- build: check for the --enable-new-dtags flag before using it Signed-off-by: Ferenc Wágner <wferi(a)debian.org> Commit: 150e9883604e4ff975c08ea58c8b5662a007a879 https://github.com/kronosnet/kronosnet/commit/150e9883604e4ff975c08ea58c8b5… Author: Ferenc Wágner <wferi(a)debian.org> Date: 2018-01-30 (Tue, 30 Jan 2018) Changed paths: M configure.ac Log Message: ----------- build: use --as-needed to avoid unnecessary dependency on libm I removed --as-needed in 1b96703, but now I found a good reason to use it after all. Signed-off-by: Ferenc Wágner <wferi(a)debian.org> Commit: 7ea95b2ca180e17f084afbf6923a89058478d897 https://github.com/kronosnet/kronosnet/commit/7ea95b2ca180e17f084afbf6923a8… Author: Ferenc Wágner <wferi(a)debian.org> Date: 2018-01-30 (Tue, 30 Jan 2018) Changed paths: M configure.ac Log Message: ----------- build: linking with libm is unnecessary on some systems (like OS X) Inlining of builtin functions can eliminate the libm dependency on other systems as well, but AC_SEARCH_LIBS can't detect this because it tests with a false prototype. Thus we use --as-needed linking already to trim such unnecessary dependencies. Signed-off-by: Ferenc Wágner <wferi(a)debian.org> Commit: 091d8f5eec1d93a79f1031419e466af6939feafb https://github.com/kronosnet/kronosnet/commit/091d8f5eec1d93a79f1031419e466… Author: Ferenc Wágner <wferi(a)debian.org> Date: 2018-01-30 (Tue, 30 Jan 2018) Changed paths: M libknet/Makefile.am M libtap/Makefile.am Log Message: ----------- build: --export-dynamic is needed for dlopened modules only Not that it matters on ELF platforms the least. If Kronosnet is ever ported to Windows, however... Signed-off-by: Ferenc Wágner <wferi(a)debian.org> Compare: https://github.com/kronosnet/kronosnet/compare/9dfd4870afdb...091d8f5eec1d