[riot-notifications] [RIOT-OS/RIOT] make: add LOG_LEVEL to overridable variables (#11592)
notifications at github.com
Tue May 28 17:36:16 CEST 2019
aabadie commented on this pull request.
> @@ -141,9 +141,8 @@ ifeq ($(DEVELHELP),1)
CFLAGS += -DDEVELHELP
-# 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
Thinking of it a second time, not checking if CFLAGS is already set would lead to build issue because of LOG_LEVEL macro already defined. I updated this another time.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications