Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ESP32] Make error #5458

Closed
iha opened this issue Dec 21, 2017 · 2 comments · Fixed by #5459
Closed

[ESP32] Make error #5458

iha opened this issue Dec 21, 2017 · 2 comments · Fixed by #5459
Assignees
Labels
area: Build System bug The issue is a bug, or the PR is fixing a bug

Comments

@iha
Copy link
Contributor

iha commented Dec 21, 2017

@lpereira

Building hello_world sample, for example, I got this error:

xtensa-esp32-elf/5.2.0/../../../../xtensa-esp32-elf/bin/ld: unrecognized option '--print-memory-usage'

I've been using this toolchain version 1.22.0-73. And it worked until my last git rebase.

@SebastianBoe SebastianBoe self-assigned this Dec 21, 2017
@SebastianBoe SebastianBoe added area: Build System bug The issue is a bug, or the PR is fixing a bug labels Dec 21, 2017
@SebastianBoe
Copy link
Collaborator

Regression from dbdd722.

SebastianBoe added a commit to SebastianBoe/zephyr that referenced this issue Dec 21, 2017
-Wl,--print-memory-usage is a relatively new LD feature (2015) and is
not supported by the espressif toolchain. Unfortunately the toolchain
compatibility test was broken as it was not using the flag when
linking.

The root cause lies in how we test LD compatibility and this same fix
must be applied again in e.g. target_ld_options().

This patch fixes
zephyrproject-rtos#5458

Signed-off-by: Sebastian Bøe <sebastian.boe@nordicsemi.no>
@iha
Copy link
Contributor Author

iha commented Dec 21, 2017

@SebastianBoe thanks, it works.

nashif pushed a commit that referenced this issue Dec 21, 2017
-Wl,--print-memory-usage is a relatively new LD feature (2015) and is
not supported by the espressif toolchain. Unfortunately the toolchain
compatibility test was broken as it was not using the flag when
linking.

The root cause lies in how we test LD compatibility and this same fix
must be applied again in e.g. target_ld_options().

This patch fixes
#5458

Signed-off-by: Sebastian Bøe <sebastian.boe@nordicsemi.no>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Build System bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants