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

Gaƫtan Harter notifications at github.com
Tue Mar 26 11:33:12 CET 2019


cladmi commented on this pull request.



> @@ -131,7 +131,6 @@ endif
 
 LINKFLAGS += -T$(RIOTCPU)/$(CPU)/ld/eagle.rom.addr.v6.ld
 LINKFLAGS += -nostdlib -lgcc -u ets_run -Wl,-gc-sections # -Wl,--print-gc-sections
-LINKFLAGS += -Wl,--warn-unresolved-symbols

>From the errors I see I understand where it comes from :)
The goal of this tracking pr is to try finding them and solving them if they are bugs.

The drawback I have with having this option, is I used "stdio_read" in a function and was able to generate a firmware that passed the compilation (with a warning but when there is one, why not 10) but failed at runtime. I was indeed missing a dependency but the build did not fail because of this. So we can link and generate invalid firmwares.

I however do not currently understand why the other platforms manage to link without providing errors for these modules. Maybe a difference in a flag but I did not investigate for the moment.

-- 
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#discussion_r269034436
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190326/346a4fa2/attachment-0001.html>


More information about the notifications mailing list