[riot-notifications] [RIOT-OS/RIOT] boards/native: fix overriden INCLUDES leading to build failure (#11572)
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:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications