[riot-notifications] [RIOT-OS/RIOT] Introduce module vars (#10928)
notifications at github.com
Sat Feb 2 00:17:28 CET 2019
### Contribution description
Currently, if a module wants to add to the global CFLAGS et al, it needs this clumsy construct:
ifneq (,$(filter foo,$(USEMODULE)))
CFLAGS += fooflag
Apart from being ugly, it currently doesn't work reliably in cpu/foobar/Makefile.include, as the dependencies are not yet processed.
This PR allows used modules to append to some selected (through addition to MODULE_VARS) variables by setting e.g., 'CFLAGS.modulename += flag'.
A second PR adapts sys/Makefile.include to this scheme.
### Testing procedure
Code should compile identically to before.
### Issues/PRs references
You can view, comment on, or merge this pull request online at:
-- Commit Summary --
* make: introduce module variables
* make: sys/Makefile.include: adapt to module vars
-- File Changes --
M Makefile.include (14)
M sys/Makefile.include (51)
-- Patch Links --
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