Author: blogic
Date: 2014-08-25 08:36:06 +0200 (Mon, 25 Aug 2014)
New Revision: 42282

Modified:
   trunk/config/Config-build.in
   trunk/include/package.mk
Log:
PKG_CHECK_FORMAT_SECURITY: add a menuconfig option, disable by default

The idea is still to enable it by default at some point
I've tested all ar71xx packages (except oldpackages) using CONFIG_ALL=y
Failing packages have been marked with PKG_CHECK_FORMAT_SECURITY:=0 for now
I can test more targets but i have no idea which are the most used

Signed-off-by: Etienne CHAMPETIER <[email protected]>

Modified: trunk/config/Config-build.in
===================================================================
--- trunk/config/Config-build.in        2014-08-25 06:36:01 UTC (rev 42281)
+++ trunk/config/Config-build.in        2014-08-25 06:36:06 UTC (rev 42282)
@@ -93,6 +93,15 @@
 
                  If you are unsure, select N.
 
+       config PKG_CHECK_FORMAT_SECURITY
+               bool
+               prompt "Enable gcc format-security"
+               default n
+               help
+                 Add -Wformat -Werror=format-security to the CFLAGS
+                 You can disable this per package by adding
+                 PKG_CHECK_FORMAT_SECURITY:=0 in the package Makefile
+
        config PKG_BUILD_USE_JOBSERVER
                bool
                prompt "Use top-level make jobserver for packages"

Modified: trunk/include/package.mk
===================================================================
--- trunk/include/package.mk    2014-08-25 06:36:01 UTC (rev 42281)
+++ trunk/include/package.mk    2014-08-25 06:36:06 UTC (rev 42282)
@@ -14,7 +14,7 @@
 PKG_MD5SUM ?= unknown
 PKG_BUILD_PARALLEL ?=
 PKG_USE_MIPS16 ?= 1
-PKG_CHECK_FORMAT_SECURITY ?= 0
+PKG_CHECK_FORMAT_SECURITY ?= 1
 
 ifneq ($(CONFIG_PKG_BUILD_USE_JOBSERVER),)
   MAKE_J:=$(if $(MAKE_JOBSERVER),$(MAKE_JOBSERVER) -j)
@@ -34,8 +34,10 @@
     TARGET_CFLAGS += -mips16 -minterlink-mips16
   endif
 endif
-ifeq ($(strip $(PKG_CHECK_FORMAT_SECURITY)),1)
-  TARGET_CFLAGS += -Wformat -Werror=format-security
+ifdef CONFIG_PKG_CHECK_FORMAT_SECURITY
+  ifeq ($(strip $(PKG_CHECK_FORMAT_SECURITY)),1)
+    TARGET_CFLAGS += -Wformat -Werror=format-security
+  endif
 endif
 
 include $(INCLUDE_DIR)/prereq.mk
_______________________________________________
openwrt-commits mailing list
[email protected]
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-commits

Reply via email to