Skip to content

Fixes esp32_out.ld error #3694

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

Closed

Conversation

chopmann
Copy link

No description provided.

@chopmann
Copy link
Author

I have a working Demo with c++17 and the newer toolchain
https://github.com/HamburgChimps/ibis_wagenbus

@atanisoft
Copy link
Collaborator

@chopmann it isn't about having support for a newer version or not. The esp-idf build scripts default the gnu++11. I have also successfully overridden this to use c++14 without issues, even on esp-idf v3.x.

@chopmann
Copy link
Author

chopmann commented Feb 11, 2020

@chopmann it isn't about having support for a newer version or not. The esp-idf build scripts default the gnu++11. I have also successfully overridden this to use c++14 without issues, even on esp-idf v3.x.

I know that. Of course it works with c++14 with platformio. What im saying is:
the current package toolchain-xtensa32 wont work with this PR or #3670
As far as I understood it needs the newer toolchain with gcc8.2 and not the older one

esp-idf v3.x. -->https://docs.espressif.com/projects/esp-idf/en/v3.3.1/get-started/linux-setup.html (GCC 5.2)
esp-idf v4.x --> https://www.esp32.com/viewtopic.php?f=10&t=7400&p=31257#p41667

@atanisoft
Copy link
Collaborator

That is correct, PlatformIO can not use IDF v4 without a lot of major reworking in PlatformIO core.

I no longer use PlatformIO, but your example of using gnu++17 linked above is the only approach usable with PlatformIO. It will not work outside of that env and won't work for those using cmake based esp-idf builds that include arduino-esp32. For cmake there is a way to do it but it would be better if it was configurable from the esp-idf side instead.

@me-no-dev me-no-dev closed this Nov 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants