[riot-notifications] [RIOT-OS/RIOT] make: newlib: check if newlib nano folder was found (#11145)

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

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

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

-- Commit Summary --

  * make: newlib: check if newlib nano folder was found

-- File Changes --

    M makefiles/libc/newlib.mk (5)

-- Patch Links --

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


More information about the notifications mailing list