[riot-notifications] [RIOT-OS/RIOT] make: newlib: check if newlib nano folder was found (#11145)
notifications at github.com
Fri Mar 8 18:53:56 CET 2019
### Contribution description
The logic for finding the newlib nano include dir can, if no folder is found, end up with an empty "-I".
This leads to weird error messages, like:
```cc1: error: -I/home/kaspar/src/riot/core/include: No such file or directory [-Werror=missing-include-dirs]```
followed by many headers that cannot be found.
This is caused by "-I -Icpu/include". I lost a while debugging this, and just yesterday another user on IRC popped up with the same problem.
Underlying is a fedora newlib packaging problem, which seems to not build with nanospecs.
This PR adds a simple check whether NEWLIB_NANO_INCLUDE_DIR is empty despite nanospecs enabled, and outputs an error.
### Testing procedure
Move the newlib nano folder (/usr/arm-none-eabi/include/newlib-nano on my arch) out of the way. Compile on master and with this PR.
### Issues/PRs references
You can view, comment on, or merge this pull request online at:
-- Commit Summary --
* make: newlib: check if newlib nano folder was found
-- File Changes --
M makefiles/libc/newlib.mk (5)
-- 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