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

issues building #14

Closed
mzimmers opened this issue Feb 4, 2020 · 5 comments
Closed

issues building #14

mzimmers opened this issue Feb 4, 2020 · 5 comments
Assignees

Comments

@mzimmers
Copy link

mzimmers commented Feb 4, 2020

Hi -

I'm looking forward to getting this software to build. Currently, I'm attempting to build using esp-idf v4.1, and v8.2 of the toolchain. Target will be a proprietary board using the WROVER.

I'm getting two build errors:

  1. multiple definitions of if_indextoname(). This appears to be part of a workaround for an asio bug, but I wonder if the definition in lwip_udp_client.h may now be an anachronism.
  2. undefined references to several routines that appear to be C++ error handling routines, eg. _Unwind_Resume.

Thanks for any assistance on this.

chrta added a commit that referenced this issue Feb 4, 2020
This enables linking this project with an esp-idf version before and
after the fix of issue  espressif/esp-idf#3557

Commit to fix this in esp-idf
espressif/esp-idf@b7ef7fe

This was reported in issue #14
@chrta chrta self-assigned this Feb 4, 2020
@chrta
Copy link
Owner

chrta commented Feb 4, 2020

Thank you for reporting this.

See also espressif/esp-idf#4490 After upgrading to the current master and ensuring that you use a supported toolchain, you have to run:

idf.py fullclean

After that also your 2nd error should be gone. I have not tested this on hardware, yet.

@mzimmers
Copy link
Author

mzimmers commented Feb 4, 2020

Thank you Christian. I did a full clean on my project, and I believe my toolchain is supported:
toolchain
This eliminated the C++ error handler errors; I'm still getting the duplicate definition error, though.

@chrta
Copy link
Owner

chrta commented Feb 4, 2020

I'm still getting the duplicate definition error, though.

Have you pulled my latest change? It is working for me in linux.

@mzimmers
Copy link
Author

mzimmers commented Feb 4, 2020

You fixed this since yesterday? Anyway, yes, it builds now...thanks for the help. I'm sure I'll have more questions down the road.

@chrta
Copy link
Owner

chrta commented Feb 4, 2020

Yes, see the linked commit above. Glad to hear that it works now.

@chrta chrta closed this as completed Feb 4, 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

No branches or pull requests

2 participants