[openwrt/openwrt] verbose.mk: print ERROR messages in non-verbose

LEDE Commits lede-commits at lists.infradead.org
Mon Feb 28 14:52:16 PST 2022


aparcar pushed a commit to openwrt/openwrt.git, branch master:
https://git.openwrt.org/aee3594ffcb72ae3e18c3719012d52519ee2d160

commit aee3594ffcb72ae3e18c3719012d52519ee2d160
Author: Paul Spooren <mail at aparcar.org>
AuthorDate: Mon Feb 28 10:14:25 2022 +0100

    verbose.mk: print ERROR messages in non-verbose
    
    Using `make -j9` only prints a subset of messages to follow the build
    process progressing. However this silently skips over errors which might
    be of interested. Using `make V=s` easily floods the terminal making it
    hard to find error messages between the lines.
    
    A compromise is the usage of `$(call ERROR_MESSAGE,...)` which prints a
    message in red. This function is silenced in the non-verbose mode, even
    if only used at a single place in `package/Makefile` where it notifies
    about a OPKG corner case.
    
    This commit moves the `ERROR_MESSAGE` definition outside of the
    `OPENWRT_VERBOSE` condition and print error messages in every mode.
    
    With this in place further error messages are possible.
    
    Signed-off-by: Paul Spooren <mail at aparcar.org>
---
 include/verbose.mk | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/include/verbose.mk b/include/verbose.mk
index 3ecf842d11..4487a207e8 100644
--- a/include/verbose.mk
+++ b/include/verbose.mk
@@ -29,15 +29,15 @@ ifeq ($(IS_TTY),1)
   endif
 endif
 
+define ERROR_MESSAGE
+  printf "$(_R)%s$(_N)\n" "$(1)" >&8
+endef
+
 ifeq ($(findstring s,$(OPENWRT_VERBOSE)),)
   define MESSAGE
 	printf "$(_Y)%s$(_N)\n" "$(1)" >&8
   endef
 
-  define ERROR_MESSAGE
-	printf "$(_R)%s$(_N)\n" "$(1)" >&8
-  endef
-
   ifeq ($(QUIET),1)
     ifneq ($(CURDIR),$(TOPDIR))
       _DIR:=$(patsubst $(TOPDIR)/%,%,${CURDIR})
@@ -60,5 +60,4 @@ else
   define MESSAGE
     printf "%s\n" "$(1)"
   endef
-  ERROR_MESSAGE=$(MESSAGE)
 endif




More information about the lede-commits mailing list