[riot-notifications] [RIOT-OS/RIOT] cpu/esp: treat undefined reference as errors (#11246)
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:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications