[riot-notifications] [RIOT-OS/RIOT] Introduce module vars (#10928)

Kaspar Schleiser 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
endif
```

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

none
You can view, comment on, or merge this pull request online at:

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

-- 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 --

https://github.com/RIOT-OS/RIOT/pull/10928.patch
https://github.com/RIOT-OS/RIOT/pull/10928.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/10928
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190201/b21e897e/attachment.html>


More information about the notifications mailing list