[riot-notifications] [RIOT-OS/RIOT] cpu/esp: treat undefined reference as errors (#11246)

Gaƫtan Harter 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:

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

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

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


More information about the notifications mailing list