[riot-notifications] [RIOT-OS/RIOT] TAKEOVER: makefiles/color: Add color functions, new attempt. #12156 (#16434)

Francisco notifications at github.com
Tue May 4 11:04:26 CEST 2021


### Contribution description

This PR split out the color functions from #12156 without attempting to replace all usage but allowing to use built-ini make functions instead of shell calls.

### Testing procedure

```
diff --git a/Makefile.include b/Makefile.include
index 5ddce7f8dd..1c50e6881d 100644
--- a/Makefile.include
+++ b/Makefile.include
@@ -583,7 +583,8 @@ flashfile: $(FLASHFILE)
 ifeq (,$(FLASHFILE))
   $(error FLASHFILE is not defined for this board: $(FLASHFILE))
 endif
-
+$(info $(call c_green,Color function test))
+$(info no color)
 # By default always build ELFFILE, BINFILE and FLASHFILE
 ifeq ($(RIOT_CI_BUILD),1)
   # Don't build BINFILE on the CI to save some computation time
```

Compile any application.

### Issues/PRs references

Split out from #12156
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/16434

-- Commit Summary --

  * makefiles/color: preprocess color strings.
  * makefiles/utils/ansi: Refactor ansi codes into their own file.
  * makefiles/color: Add color functions.

-- File Changes --

    M Makefile.include (1)
    M makefiles/color.inc.mk (19)
    A makefiles/utils/ansi.mk (11)
    A makefiles/utils/ansi_special.mk (16)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/16434.patch
https://github.com/RIOT-OS/RIOT/pull/16434.diff

-- 
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/16434
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210504/fc92218b/attachment.htm>


More information about the notifications mailing list