[riot-notifications] [RIOT-OS/RIOT] make: add LOG_LEVEL to overridable variables (#11592)

Alexandre Abadie notifications at github.com
Tue May 28 17:08:39 CEST 2019


aabadie commented on this pull request.



> @@ -141,9 +141,8 @@ ifeq ($(DEVELHELP),1)
   CFLAGS += -DDEVELHELP
 endif
 
-# If not already set in CFLAGS, set logging level by default to INFO, e.g. 3.
-LOG_LEVEL ?= 3
-ifeq (,$(filter -DLOG_LEVEL=%,$(CFLAGS)))
+# Override LOG_LEVEL if variable is set
+ifdef LOG_LEVEL

That's a choice, if the user already sets the CFLAGS in the Makegile, he is supposed to know what he is doing.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11592#discussion_r288155463
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190528/f5a3af4a/attachment.html>


More information about the notifications mailing list