[riot-notifications] [RIOT-OS/RIOT] boards/native: fix overriden INCLUDES leading to build failure (#11572)

Gaƫtan Harter notifications at github.com
Mon May 27 14:00:28 CEST 2019


cladmi requested changes on this pull request.

`NATIVEINCLUDES` is currently necessary here. At least, changing it is not trivial. And `+=` should not be used at all.

The goal to use `NATIVEINCLUDES` is to not use the modules include directories and currently use the system `posix` headers for example.

It make sense for module giving a `RIOT` interface around `system` functionalities.

If `boards` needs both `RIOT` and `system` headers it means the `system` specific part may need to be split out.



-- 
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/11572#pullrequestreview-242241197
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190527/0b531596/attachment-0001.html>


More information about the notifications mailing list