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

Ga√ętan Harter notifications at github.com
Tue May 28 09:51:28 CEST 2019


@jcarrano currently it is using APIs that are not available for `boards/native/board.c` as it uses restrictive includes, so the placement is wrong.
As `native/boards` is not using any system `posix` headers it should be possible to use the normal `INCLUDES` there. Doing a rationalization of the `NATIVEINCLUDES` handling in general could be a good thing.

However, in practice, why is there even a `native_motor_driver_qdec_simulation` in `boards/native/board.c`?
If there is a need for a fake test device, why not put it in its own module ?
It is always compiled even if `motor_driver` features is not used.

-- 
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#issuecomment-496405388
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190528/a1d1996a/attachment-0001.html>


More information about the notifications mailing list