]> rtime.felk.cvut.cz Git - coffee/buildroot.git/commitdiff
acl: make sure build picks up TARGET_CFLAGS
authorArnout Vandecappelle <arnout@mind.be>
Fri, 17 Mar 2017 16:06:14 +0000 (17:06 +0100)
committerThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Sat, 18 Mar 2017 13:09:32 +0000 (14:09 +0100)
The acl build system doesn't use automake, therefore it is broken. It
doesn't use the CFLAGS passed by configure. Work around this by passing
CFLAGS in the environment. The makefiles append to CFLAGS, so this
works.

This issue hasn't led to build failures, but it is visible e.g. when
stack protector is enabled: the stack protector options are not applied
to acl. Also debug and optimisation options aren't applied.

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Cc: Yegor Yefremov <yegorslists@googlemail.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
package/acl/acl.mk

index cbe5e9396107649c53a88384e8902df1abf71dd0..9fd5de18a458946f989801d8bd913f013b230192 100644 (file)
@@ -15,7 +15,13 @@ ACL_LICENSE_FILES = doc/COPYING doc/COPYING.LGPL
 
 # While the configuration system uses autoconf, the Makefiles are
 # hand-written and do not use automake. Therefore, we have to hack
-# around their deficiencies by passing installation paths.
+# around their deficiencies by:
+# - explicitly passing CFLAGS (LDFLAGS are passed on from configure,
+#   CFLAGS are not).
+# - explicitly passing the installation prefix, not using DESTDIR.
+
+ACL_MAKE_ENV = CFLAGS="$(TARGET_CFLAGS)"
+
 ACL_INSTALL_STAGING_OPTS = \
        prefix=$(STAGING_DIR)/usr \
        exec_prefix=$(STAGING_DIR)/usr \