[riot-notifications] [RIOT-OS/RIOT] cpu/esp: treat undefined reference as errors (#11246)
notifications at github.com
Sat Mar 23 18:42:23 CET 2019
### Contribution description
For the esp cpus, treat undefined reference as errors.
It was silently ignoring compilation errors.
### Cleanup needed
This pull request should show compilation issues that are currently present and should be fixed to have this one merged.
I noticed that currently `stdio_uart` needs `isrpipe` that needs `xtimer` but without saying the dependency. This should show this issue when building everything and need to be handled in separate pull requests.
### Testing procedure
Just define an `extern void lala(void);` function in an application and call it.
The compilation works in master and ignores the issue.
With this pull request it will correctly complain about it.
### Issues/PRs references
Found while working on running tests with the `esp32` no open pull request yet.
You can view, comment on, or merge this pull request online at:
-- Commit Summary --
* esp32: treat undefined reference as errors
* esp8266: treat undefined reference as errors
-- File Changes --
M cpu/esp32/Makefile.include (1)
M cpu/esp8266/Makefile.include (1)
-- 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